-
-
Notifications
You must be signed in to change notification settings - Fork 13.3k
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
zlib: 1.2.12 -> 1.2.13 #196167
zlib: 1.2.12 -> 1.2.13 #196167
Conversation
@ofborg build pkgsCross.aarch64-multiplatform.zlib zlib |
@ofborg build pkgsCross.aarch64-embedded.zlib still fails (not a regression)
|
qt5.qtbase builds |
Successfully built |
Does it fetch for you successfully? When I try to fetch it in
|
Looks OK to me
However, on sourceforge I see only versions up to 1.2.11: https://sourceforge.net/projects/libpng/files/zlib/ |
Hm:
Looks like I'm catpcha'd:
Apologies for the noise. |
@@ -23,14 +23,14 @@ assert splitStaticOutput -> static; | |||
|
|||
stdenv.mkDerivation (rec { | |||
pname = "zlib"; | |||
version = "1.2.12"; | |||
version = "1.2.13"; | |||
|
|||
src = fetchurl { | |||
urls = | |||
[ "https://www.zlib.net/fossils/zlib-${version}.tar.gz" # stable archive path |
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.
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.
That's bad, as that URL disappears on every release. They only keep the last one there.
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.
It's why the "stable archive path" comment is there.
That's a bit of a problem. Could we maybe fallback to a mirror? or reach out to upstream? |
E.g. like this? #196910 |
On sourceforge I see only versions up to 1.2.11: https://sourceforge.net/projects/libpng/files/zlib/ and the order is switched to avoid captcha (I didn't encounter it): #196167 (comment)
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes