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
rar2fs: init at 1.28.0 #78189
rar2fs: init at 1.28.0 #78189
Conversation
554d6a7
to
ac80d22
Compare
Woops. Thought I finally understood git history. Apparently not. |
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.
Hi @kraem, welcome to Nixpkgs and thanks for your contribution :)
I found a few things that should be improved, but overall it looks good.
Finally managed to build it without the unrar source-code, wasn't used to debugging autoconf scripts anymore.. :o :D @kraem could you test if everything still works as expected? cc: @ehmry as the |
@primeos Works perfectly. Thanks! |
This is e.g. required for the rar2fs build [0], which needs at least version.hpp, rar.hpp, dllext.hpp, dll.hpp, and headers5.hpp. At least Gentoo does this as well [1] but most other distributions only install dll.hpp or no header files at all. [0]: NixOS#78189 [1]: https://gitweb.gentoo.org/repo/gentoo.git/tree/app-arch/unrar/unrar-5.8.5.ebuild
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.
LGTM now, thanks for the PR :)
@ehmry I'll merge it tomorrow if you don't have any complaints (just in case I missed something) ;)
Edit: Forgot to mention: nix-review
was successful:
6 package were build:
pdfread rar2fs rarcrack sabnzbd unrar zod
This is e.g. required for the rar2fs build [0], which needs at least version.hpp, rar.hpp, dllext.hpp, dll.hpp, and headers5.hpp. At least Gentoo does this as well [1] but most other distributions only install dll.hpp or no header files at all. [0]: NixOS#78189 [1]: https://gitweb.gentoo.org/repo/gentoo.git/tree/app-arch/unrar/unrar-5.8.5.ebuild (cherry picked from commit 983f39c)
Motivation for this change
rar2fs
uses fuse to mount rar archives, or directories containing rar archives, and makes the files available as if they were unrar:ed.rar2fs Github
rar2fs web
Things done
Init package at most recent version 1.28.0
Added the unfree license even though
rar2fs
is licensed using GPLv3.The source from
RARLAB
is used during compilation though. The license ofRARLAB
is free to distribute if only theUnRAR
sources are used (source). I'm not sure if anything else is used during the compilation ofrar2fs
. I'm super new to licenses and if anyone has any idea as to what the situation is here they are very welcome to have a say in this :)Other
This is my first package added so added myself to the maintainers-list. I'm more than happy to receive suggestions for improvement too :)
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)