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
libpng: 1.6.35 -> 1.6.36, license v2 #51447
Conversation
So... they do a licensing change in a patch version, to begin with.
Oh come on... Just choose one of the two licenses already defined and accepted as free instead of (not) reinventing the wheel by composing parts of others' licenses... |
@@ -25,14 +25,14 @@ in stdenv.mkDerivation rec { | |||
|
|||
propagatedBuildInputs = [ zlib ]; | |||
|
|||
doCheck = stdenv.hostPlatform == stdenv.buildPlatform; | |||
doCheck = true; |
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 will be still skipped for cross-builds, right?
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.
Yes, that's my understanding! :)
@GrahamcOfBorg eval |
1 similar comment
@GrahamcOfBorg eval |
240017f
to
d37b48d
Compare
Sorry for the delay! :) |
Okay to merge IMO, any objections? |
Bikeshedding: maybe we don't need the definition in |
Cross-links: pnggroup/libpng#266 98be12a |
Motivation for this change
https://sourceforge.net/p/png-mng/mailman/message/36483059/
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)