Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add flint support #157

Merged
merged 1 commit into from
Feb 20, 2024
Merged

Add flint support #157

merged 1 commit into from
Feb 20, 2024

Conversation

louisaslett
Copy link
Contributor

This adds support for FLINT (Fast Library for Number Theory).

Currently the library is in Debian, Ubuntu, OpenSUSE and Fedora, but not in RHEL type distros, including CentOS or Rocky.

The library is currently in testing branch of Alpine, so we will need to wait for release.

I've confirmed that make test-all RULES=rules/flint.json passes with no errors.

louisaslett added a commit to louisaslett/fhe that referenced this pull request Feb 20, 2024
…stem-requirements is merged (rstudio/r-system-requirements#157) then this should enable auto-install of dependencies on Linux.
Copy link
Contributor

@glin glin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! No problem if we don't have RHEL/Alpine support for now.

@glin glin merged commit 7cfee24 into rstudio:main Feb 20, 2024
21 checks passed
@gaborcsardi
Copy link
Collaborator

For the record, this is how to install from a non-default repo on Alpine:

"command": "apk add udunits-dev --repository=https://dl-cdn.alpinelinux.org/alpine/edge/community"

Assuming that it works together with the native packages of the Alpine version, of course.

louisaslett added a commit to louisaslett/fhe that referenced this pull request Mar 1, 2024
…e C++ headers). Also tweak supported rpm distros in line with PR submitted here: rstudio/r-system-requirements#157
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants