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
gsl: 2.5 -> 2.6 #67182
gsl: 2.5 -> 2.6 #67182
Conversation
See https://lists.gnu.org/archive/html/info-gnu/2019-08/msg00004.html for release announcement
@GrahamcOfBorg build gsl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are the two platform specific bug fixes mentioned alongside NIX_CFLAGS_COMPILE
and doCheck
still required?
I did not see in the changelog reasons to think those are not required anymore, but I have to admit I did not test without those… I’ll investigate further into this (given that I only have a x86_64 machine available, I will not be able to test for other platforms) |
If you want to test if the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The nix-review looks good. I can not see any build failures that I could attribute to the gsl update.
I removed the checks NIX_CFLAGS_COMPILE and doCheck instructions to see how it works (in a WIP independant commit). @GrahamcOfBorg build gsl |
@GrahamcOfBorg build gsl |
The We can properly build without disabling fma, but I’d prefer to leave it as it is for consistency. |
Thanks for double checking the fixes! |
See https://lists.gnu.org/archive/html/info-gnu/2019-08/msg00004.html
for release announcement
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @