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
fcrackzip: init at 1.0 #21095
fcrackzip: init at 1.0 #21095
Conversation
forgot to commit all-packages.nix. Fixed |
patches = [ ./fcrackzip_forkexec.patch ]; | ||
|
||
buildInputs = [ ]; | ||
# installFlags = "PREFIX=\${out}"; |
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.
Please remove this line if it's not needed.
license = licenses.gpl2; | ||
maintainers = with maintainers; [ nico202 ]; | ||
platforms = with platforms; unix; | ||
priority = 10; |
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.
Why the priority? Which binaries clash with other packages?
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.
fcrackzip/zipinfo clash with unzip/zipinfo
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.
Should I rename zipinfo to fcrackzip-zipinfo instead?
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 sounds useful! Please do, thanks! 👍
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.
@fpletz renamed (line 14)
sha256 = "0l1qsk949vnz18k4vjf3ppq8p497966x4c7f2yx18x8pk35whn2a"; | ||
}; | ||
|
||
patches = [ ./fcrackzip_forkexec.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.
Did you have to modify that patch? If not, can you use fetchpatch?
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 had to modify it a bit because it was a patch over a patched file
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.
If possible could you add a comment above the patches
line explaining what the patch does?
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.
Done!
085f9a8
to
2a485a5
Compare
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 looks good to me, thanks! I can't merge though.
sha256 = "0l1qsk949vnz18k4vjf3ppq8p497966x4c7f2yx18x8pk35whn2a"; | ||
}; | ||
|
||
patches = [ ./fcrackzip_forkexec.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.
If possible could you add a comment above the patches
line explaining what the patch does?
patch adapted from https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=430387