Bug 302597

Summary:	please add support for LibLastFm 1.0+
Product:	[Applications] amarok	Reporter:	Andrei Mihaila <andrei.mihaila>
Component:	Services/Last.fm	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	matej, rdieter, sam
Priority:	NOR
Version:	2.5.90 (2.6 beta)
Target Milestone:	2.7
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:	http://commits.kde.org/amarok/24b3feaf279199100f1a0a41e40e22a5408d7411	Version Fixed In:	2.7
Sentry Crash Report:

Description Andrei Mihaila 2012-06-26 20:28:13 UTC

Amarok doesn't detect LibLastFm 1.0+ (works fine with 0.3.3 for example). When I compiled it while having LibLastFm 1.0.1 installed the support for LastFm was disabled because the library was not found. Tried to hack the code that detects it (changed a line in the cmake/modules/FindLibLastFm.cmake) and the detection worked. I had a bit of hope ... but then the compilation failed on one of the files related to the support for LibLastFm.

Downgraded to LibLastFm 0.3.3 and it works as expected.

Anyway, good job on the rest of the features ... and when you can spare some time please add support for LibLastFm 1.0+.

Thank you!

Reproducible: Always

Steps to Reproduce:
1. install LibLastFm 1.0+
2. compile amarok 2.5.90
3.
Actual Results:  
the LastFm support is disabled

Expected Results:  
the LastFm support should be enabled and working

Comment 1 Sam Lade 2012-06-26 20:32:14 UTC

Unfortunately, liblastfm1 is API incompatible with the previous version. A port to liblastfm1 is available on our reviewboard here: https://git.reviewboard.kde.org/r/105276/ However, this has not yet been pushed to master for dependency reasons. You can apply the patch manually or just use the older liblastfm1. We'll increase the dependency once distros have had a chance to ship the new one.

Comment 2 Andrei Mihaila 2012-06-27 19:35:52 UTC

Thank you for the very quick answer!

Modified the Gentoo package (applied the patch, works nicely) and noticed the maintainers (https://bugs.gentoo.org/show_bug.cgi?id=419509).

Comment 3 Matěj Laitl 2012-07-03 13:14:30 UTC

(In reply to comment #2)
> Thank you for the very quick answer!
> 
> Modified the Gentoo package (applied the patch, works nicely) and noticed
> the maintainers (https://bugs.gentoo.org/show_bug.cgi?id=419509).

No, please don't do that. Amarok 2.5 and 2.6 wil require liblastfm 0.3* and Amarok 2.7 will be ported to liblastfm 1.0.

Comment 4 Sam Lade 2012-08-14 13:01:28 UTC

Reopened since 2.6 has been released. Liblastfm1 support will be merged shortly.

Comment 5 Matěj Laitl 2012-08-14 14:43:33 UTC

Git commit 24b3feaf279199100f1a0a41e40e22a5408d7411 by Matěj Laitl.
Committed on 14/08/2012 at 16:22.
Pushed by laitl into branch 'master'.

Merge branch 'liblastfm1'

Amarok developers, users building git: you'll need to upgrade your
liblastfm to >= 1.0.0 for Last.fm support to be included.

* liblastfm1:
  Update Last.fm info in CMakeLists, README, check its version in CMake
  fix port consistency as requested
  lastfm include cleanup
  port to liblastfm1 api

REVIEW: 105276
CCMAIL: amarok-devel@kde.org
CCMAIL: amarok@kde.org


http://commits.kde.org/amarok/24b3feaf279199100f1a0a41e40e22a5408d7411