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
gperftools: drop an option that seems no longer supported #109996
Conversation
> warning: unknown warning option '-Wno-aligned-allocation-unavailable'; > did you mean '-Wno-debug-compression-unavailable'? [-Wunknown-warning-option]
It's de-facto revert of 6a560f4 |
We were getting the warning even previously, but before it wasn't leading to this fatal problem. Most likely that was triggered by the autoconf update – default, we still keep 269 and could switch to it selectively. |
@@ -34,7 +34,7 @@ stdenv.mkDerivation rec { | |||
''; | |||
|
|||
NIX_CFLAGS_COMPILE = stdenv.lib.optionalString stdenv.isDarwin |
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.
NIX_CFLAGS_COMPILE = stdenv.lib.optionalString stdenv.isDarwin | |
NIX_CFLAGS_COMPILE = lib.optionalString stdenv.isDarwin |
stdenv.lib is deprecated.
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.
OK, I'd fix all the places in this file at once. Can you link the deprecation?
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.
You could have searched that yourself.
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.
Thanks :-)
I confirm that removing |
Only darwin gets rebuilt, but it needs to be tested.