Skip to content

fix electrum dependencies #26251

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

Merged
merged 7 commits into from
May 31, 2017
Merged

fix electrum dependencies #26251

merged 7 commits into from
May 31, 2017

Conversation

rnhmjoj
Copy link
Contributor

@rnhmjoj rnhmjoj commented May 30, 2017

Motivation for this change

See #26249

Things done
  • Tested using sandboxing
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using
  • Tested execution of all binary files
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

@mention-bot
Copy link

@rnhmjoj, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer.

@FRidh
Copy link
Member

FRidh commented May 30, 2017

Could you move each of these expressions to new files as described in the header of python-packages.nix. That will make future updating easier as well.

@FRidh FRidh added 6.topic: python 8.has: package (new) This PR adds a new package 8.has: package (update) This PR updates a package to a newer version labels May 30, 2017
@rnhmjoj
Copy link
Contributor Author

rnhmjoj commented May 30, 2017

Ah, sorry I missed it.

rnhmjoj added 7 commits May 30, 2017 21:36

Verified

This commit was signed with the committer’s verified signature. The key has expired.
LnL7 Daiderd Jordan

Verified

This commit was signed with the committer’s verified signature. The key has expired.
LnL7 Daiderd Jordan

Verified

This commit was signed with the committer’s verified signature. The key has expired.
LnL7 Daiderd Jordan

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@FRidh FRidh merged commit d4aecb3 into NixOS:master May 31, 2017
@FRidh
Copy link
Member

FRidh commented May 31, 2017

@rnhmjoj I just noticed there were evaluation errors so I reverted it. Furthermore, the expressions are supposed to be in pkgs/development/python-modules/<pname>/default.nix. Unfortunately there's a lot that aren't.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: python 8.has: package (new) This PR adds a new package 8.has: package (update) This PR updates a package to a newer version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants