-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
tahoe-lafs: 1.12.1 -> 1.13.0 #53873
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
tahoe-lafs: 1.12.1 -> 1.13.0 #53873
Conversation
@GrahamcOfBorg build tahoe-lafs |
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.
Thank you so much for doing this! I appreciate it.
@@ -56,7 +56,7 @@ pythonPackages.buildPythonApplication rec { | |||
propagatedBuildInputs = with pythonPackages; [ | |||
twisted foolscap nevow simplejson zfec pycryptopp darcsver | |||
setuptoolsTrial setuptoolsDarcs pycrypto pyasn1 zope_interface | |||
service-identity pyyaml | |||
service-identity pyyaml magic-wormhole hypothesis treq |
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 don't think that Hypothesis needs to be propagated; could you instead put it in checkInputs
or possibly buildInputs
?
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.
@MostAwesomeDude I added it to checkInputs, and everything seems to still work
6a6b0f1
to
0ce675a
Compare
I'd prefer +1 from @MostAwesomeDude prior to merging |
Let's merge it. I have, after chatting with upstream, learned that the breakage of |
Please merge this so I can use it as base for my Gridsync PR :) |
Motivation for this change
I've never used tahoe-lafs before, 1.13 is apparently needed for wormhole invitiations.
I bumped version and hash, and then added packages until the tests passed, but I don't
know if any changes to the
pre*/post*
are needed.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @exarkun @MostAwesomeDude