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
nzbhydra2: init at 3.8.0 #99559
nzbhydra2: init at 3.8.0 #99559
Conversation
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.
The service is looking good. I have left a few comments that I think would improve it. I hope you find my comments helpful.
I generally only review modules, not packages, so we'll need to have someone review the package as well.
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 approve of the module.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
@JamieMagee Please resolve the merge conflcit. |
e89a43e
to
0e4ee76
Compare
@SuperSandro2000 Ready for review |
0e4ee76
to
c11c607
Compare
Pushed an update from 2.29.1 to 3.8.0. @aanderse I had to remove the OpenJDK 64-Bit Server VM warning: INFO: os::commit_memory(0x00007f9e8b2c3000, 2555904, 1) failed; error='Operation not permitted' (errno=1) |
c11c607
to
c9ebe19
Compare
c9ebe19
to
feb6351
Compare
@ofborg eval |
@SuperSandro2000 I propose merge - any outstanding issues from your point of view? |
@aanderse It appears there are no concerns? |
@JamieMagee sure. I'll assume @SuperSandro2000 is good with the package and merge once the bot runs your test. Thanks for contributing to nixpkgs! @GrahamcOfBorg test nzbhydra2 |
Motivation for this change
Closes #76878
Use the AUR package and jackett.nix as guidance on how to package NZBHydra2.
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)