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
journalwatch: fix broken package, general maintenance #59424
Conversation
cd57b2d
to
d5e303c
Compare
d5e303c
to
60b9e33
Compare
See NixOS#53852 for details, related to the efforts in NixOS#56265
60b9e33
to
e916cdf
Compare
@aanderse Thanks for your review, it was very helpful! 🍻 |
@GrahamcOfBorg build python3Packages.journalwatch |
@aanderse Maybe we want to change other places referencing |
@flokli that sounds good. Hopefully at some point I'll circle back to that. |
Forgot the backporting, PR at #59424. |
Motivation for this change
The
journalwatch
package is broken on 19.03, as the now more strict dependency handling ofbuildPythonPackage
uncovered the error of havingpytest
asbuildInput
instead ofcheckInput
. This PR fixes that, along with some general maintenance:permissionsStartOnly
: replace deprecated usage of PermissionsStartOnly (part 2) #56265fetchFromGitHub
: GitHub unstable archive tracking issue #32997Addservice.journalwatch.package
option for easy change of used packageThe ee20ba8 fix pytest checks commit requires a backport to 19.03 to unbreak the package, the other commits are nice to have and should be backwards-compatible, but are not required. I'm sorry I did not notice the breakage prior release.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)