-
-
Notifications
You must be signed in to change notification settings - Fork 17.1k
flint, ntl: Fix pkgsStatic builds, clean-up derivations #445050
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
Conversation
|
Hm seems pacakges in |
e9c8594 to
344b33a
Compare
|
I'm investigating the Sage failure. |
|
Hm Sage builds fine on my machine |
|
Can see here: |
|
@qweered Is the msolve test failure reproducible on your aarch64-linux machine? Two of the three authors of msolve work right next to my office, and I'm sure they would be very interested in cases where the computation fails. |
1e70ce9 to
795537f
Compare
795537f to
7e74695
Compare
|
I messed up the rebases so the github compare button doesn't work well, but the full diff between what was before vendoring the patch and after is here: |
I don't have aarch64 machines, I only rely on nixpkgs-review-gha which I think might be incorrect sometimes? |
|
Just have to keep an eye here and https://hydra.nixos.org/job/nixpkgs/trunk/msolve.aarch64-linux |
|
Hydra built msolve for aarch64 and sage for x86 so seems fine |

Things done
ntl:
dontAddStaticConfigureFlags = trueso that autotools flags aren't appended to the perl configure scriptwith lib;flint:
with lib;make checkcan actually link the testsI tested both normally and with
pkgsStatic.flintpassthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.