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-mpd: init at version 3.0.0 #80000
Conversation
Probably overlaps with #78361 :/ |
You're right. I'll close this one. |
@mmilata @Tomahna with the rate of progress of #78361 I think it's more important that mopidy remains usable with mpd client in the meantime. I'm following unstable, and without this PR I am left to wait for the other PR to go through, and cannot use mopidy anymore (!) This PR is easy to review and merge comparatively. Please reopen. |
Done. |
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.
LGTM
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.
otherwise LGTM
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.
Is this supposed to be a python package? I don't see any applications being published:
$ tree -L 3 ./results/mopidy-mpd
./results/mopidy-mpd
├── lib
│ └── python3.7
│ └── site-packages
└── nix-support
└── propagated-build-inputs
It is a python library that you bundle with mopidy to add the mpd server functionality. The important part is located in the site-packages folder. It is not a standalone application. |
Sounds like they should have been included in |
looks like my concerns are being addressed in #82651 |
Thank you, i'll look into it to update a few extensions accordingly |
Motivation for this change
It looks like that since version 3.0.0, Mopidy has separated it's mpd functionality to a separate library.
Things done
Add mopidy-mpd package.
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)