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
gdbm: avoid a warning on Darwin #44875
Conversation
@GrahamcOfBorg build gdbm |
Success on x86_64-linux (full log) Attempted: gdbm Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: gdbm Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: gdbm Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: gdbm Partial log (click to expand)
|
@@ -2,6 +2,8 @@ | |||
|
|||
stdenv.mkDerivation rec { | |||
name = "gdbm-1.17"; | |||
# FIXME: remove on update to > 1.17 | |||
NIX_CFLAGS_COMPILE = if stdenv.isDarwin then "-Wno-error=return-type" else null; |
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.
This is nitpicking but technically this should be stdenv.cc.isClang
. This works too but just because no one else is using clang.
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.
Oh, right, I was thinking of build vs. host and didn't realize that the compiler vendor is more likely to be the significant difference.
This is just to minimize rebuilds. The actual bugfix doesn't seem to hurry, as the function has been returning void until now, so if the int returned isn't a meaningful value in some cases, nothing should happen yet.
Success on x86_64-darwin (full log) Attempted: gdbm Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: gdbm Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gdbm Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gdbm Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gdbm Partial log (click to expand)
|
This is just to minimize rebuilds. The actual bugfix doesn't seem to
hurry, as the function has been returning void until now, so if the int
returned isn't a meaningful value in some cases, nothing should happen
yet.
No rebuild except on Darwin (where it was failing).
I wasn't sure when this is required on cross-compilation, so the condition might be imprecise.