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
pkgs/tools/compression: cleanup of various packages #104849
pkgs/tools/compression: cleanup of various packages #104849
Conversation
Result of 23 packages built:
|
sha256 = "0j2zm3z271x5aw63mwhr3vymzn45p2vvrlrpm9cz2nywna41b0hq"; | ||
}; | ||
|
||
buildInputs = [ bzip2 ]; | ||
patches = [ ./include-systypes.patch ]; |
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.
When and why was this patch added and why is it not needed anymore?
}; | ||
|
||
buildInputs = [ lzma ]; | ||
|
||
patches = [ ./_SC_ARG_MAX.patch ]; |
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.
Same question for this patch.
@@ -1,31 +1,27 @@ | |||
{ stdenv, fetchurl, zlib, util-linux }: | |||
{ stdenv, fetchurl, zlib, utillinux }: |
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 agree with the pname/version and fetchFromGitHub changes but a lot of the style changes just look like diff churn to me. I agree with your style choices but I don't think you should be forcing them on these packages' maintainers. To be clear, all the rebuild-causing changes look like a big improvement. It's just the whitespace changes and |
@@ -22,8 +22,7 @@ stdenv.mkDerivation rec { | |||
sha256 = "sOeXNVsCaBSD9i82GRUDrkyreGeQ7qaJWjjy/uLL0/0="; | |||
}); | |||
|
|||
cmakeFlags = [] | |||
++ stdenv.lib.optional staticOnly "-DBUILD_SHARED_LIBS=OFF"; | |||
cmakeFlags = stdenv.lib.optional staticOnly "-DBUILD_SHARED_LIBS=OFF"; |
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.
cmakeFlags = stdenv.lib.optional staticOnly "-DBUILD_SHARED_LIBS=OFF"; | |
cmakeFlags = lib.optional staticOnly "-DBUILD_SHARED_LIBS=OFF"; |
@@ -52,9 +51,7 @@ stdenv.mkDerivation rec { | |||
|
|||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
preInstall = '' | ||
makeFlagsArray+=("PREFIX=$out") | ||
''; | ||
makeFlags = [ "PREFIX=$(out)" ]; | ||
|
||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
''; | ||
|
||
meta = { | ||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
platforms = platforms.linux; | ||
}; | ||
} | ||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
installPhase = '' | ||
install -Dm755 pigz "$out/bin/pigz" | ||
install -Dm755 pigz.1 "$out/share/man/man1/pigz.1" | ||
install -Dm755 pigz.pdf "$out/share/doc/pigz/pigz.pdf" | ||
''; | ||
|
||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
license = stdenv.lib.licenses.bsd2; | ||
maintainers = [stdenv.lib.maintainers.raskin]; | ||
platforms = stdenv.lib.platforms.unix; | ||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
installPhase = '' | ||
mkdir -p $out/bin $out/share/man/man1 | ||
cp pxz $out/bin | ||
cp pxz.1 $out/share/man/man1 | ||
''; | ||
|
||
meta = { | ||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
@@ -16,10 +19,10 @@ stdenv.mkDerivation { | |||
}) | |||
]; | |||
|
|||
meta = { | |||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
sha256 = "1wjslvfy76szf0mgg2i9y9q30858xyjn6v2acc24zal76d1m778b"; | ||
}; | ||
|
||
makeFlags = [ "AR=${stdenv.cc.bintools.targetPrefix}ar" ]; | ||
|
||
meta = { | ||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
I marked this as stale due to inactivity. → More info |
I've rebased this here https://github.com/helsinki-systems/nixpkgs/tree/cleanup/compression, if you want to still have this merged @IvarWithoutBones |
Closed in favor of #130928 |
Thanks @ajs124! Totally forgot about this one. |
Motivation for this change
A ton of these packages are fairly outdated, with a lot of formatting issues. This fixes some of those.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)