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
amarok: add last.fm support, bump version #88016
Conversation
3bec51f
to
084e27b
Compare
e801b1d
to
2e1a6dc
Compare
2e1a6dc
to
85b5b20
Compare
64dc678
to
ac5ce94
Compare
5c4a0fc
to
72b7d64
Compare
Let's drop Qt 4 version of liblastfm #89821 |
@veprbl Do you want me to rebase on your branch? It looks like your branch subsumes most of what I did in this PR. All that's left is the bumping of Amarok and the addition of Your approach to porting |
@kini With the patch would be also fine, but preferably need to fetch from the upstream commit. I'm just doing the minimal change to fix things. Interested parties could look into bumping |
72b7d64
to
52d1963
Compare
@veprbl One of the patches is from a PR on the upstream repo that has been open for a couple of months (the maintainer doesn't seem to be responding to it). Is it OK to fetch from this PR? |
@kini Fetching from an unstable url like |
@veprbl Thanks. I pushed a commit that updates liblastfm to liblastfm-unstable-2019-08-23 (the latest master) and then applies that open PR's patch. One more question -- should the |
That seems reasonable to me. |
This commit bumps amarok to the most recent commit in master and adds liblastfm as a dependency to make use of the last.fm integration that has been re-enabled in upstream master (it was disabled for a while due to breakage on Qt5). I also updated the package name and version to match [the stipulations in the Nixpxgs manual](https://nixos.org/nixpkgs/manual/#sec-package-naming).
b31e097
to
640eba8
Compare
OK, thanks -- made that change too. |
640eba8
to
557f56d
Compare
Thank you! |
Motivation for this change
This PR bumps amarok to the most recent commit in master and adds liblastfm as a dependency to make use of the last.fm integration that has been re-enabled in upstream master (it was disabled for a while due to breakage on Qt5).
Things done
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)