Skip to content
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

mopidy-gmusic: 1.0.0 -> 2.0.0 #20675

Merged
merged 5 commits into from
Nov 25, 2016
Merged

Conversation

carlsverre
Copy link
Contributor

@carlsverre carlsverre commented Nov 24, 2016

Motivation for this change

Gmusic api changed - 2.x version is required to connect to google music.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@carlsverre, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jgillich and @FRidh to be potential reviewers.

@@ -0,0 +1,18 @@
{ stdenv, fetchurl, python, buildPythonPackage, gmp }:

buildPythonPackage rec {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can move this into the top level python packages file - I put it here to match pycrypto since pycryptodome is a fork of pycrypto. Open to thoughts on this.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, it's good to leave it here, since python-packages.nix is getting way too big and having a function is more flexible as well.

@carlsverre
Copy link
Contributor Author

Note: this is based on channels/nixos-unstable which I am running on my machine.

@carlsverre
Copy link
Contributor Author

Anyone know why digikam was built in the travis-ci build? I don't see how my changes affected it. Seems like nox tried to rebuild everything that depended on any python package or something? Not sure whats going on here.

@carlsverre
Copy link
Contributor Author

@anderspapitto maybe you might know?

@FRidh
Copy link
Member

FRidh commented Nov 24, 2016

Each package you modify needs to be in a commit of its own.

@carlsverre
Copy link
Contributor Author

@FRidh like that?

@FRidh FRidh merged commit 51645f8 into NixOS:master Nov 25, 2016
@FRidh
Copy link
Member

FRidh commented Nov 25, 2016

yep, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants