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
dcraw: fix build on Darwin #44912
dcraw: fix build on Darwin #44912
Conversation
@@ -8,15 +8,18 @@ stdenv.mkDerivation rec { | |||
sha256 = "1fdl3xa1fbm71xzc3760rsjkvf0x5jdjrvdzyg2l9ka24vdc7418"; | |||
}; | |||
|
|||
buildInputs = [ libjpeg lcms2 gettext jasper ]; | |||
buildInputs = [ libjpeg lcms2 gettext jasper ] | |||
++ stdenv.lib.optional stdenv.isDarwin libiconv; |
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.
I think this doesn't need to be conditional anymore.
@GrahamcOfBorg build dcraw |
Success on aarch64-linux (full log) Attempted: dcraw Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: dcraw Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: dcraw Partial log (click to expand)
|
1ceff36
to
32b0817
Compare
iconv in this case is used only for character conversions at build time, which means I should make it a |
32b0817
to
85c4367
Compare
Motivation for this change
Noticed it was broken.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)