Bug 139489

Summary:	Amarok does not compile when compiling with MTP support (libmtp version 0.1.2)
Product:	[Applications] amarok	Reporter:	Jose Negron <josenj.wa>
Component:	Collections/MTP player	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED FIXED
Severity:	normal	CC:	hydrogen, mueller
Priority:	NOR
Version:	1.4.4
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Hack to get it compile with libmtp 0.1.2 Against SVN for amarok 1.4.4

Description Jose Negron 2007-01-02 03:23:49 UTC

Version:           1.4.4 (using KDE KDE 3.5.5)
Installed from:    Compiled From Sources
Compiler:          gcc version 4.1.2 
OS:                Linux

When compiling Amarok v1.4.4 with MTP support, compile fails with the following message:

mtpmediadevice.cpp: In member function 'virtual bool MtpMediaDevice::getCapacity(KIO::filesize_t*, KIO::filesize_t*)':
mtpmediadevice.cpp:986: error: 'LIBMTP_Get_Storageinfo' was not declared in this scope
mtpmediadevice.cpp: In member function 'virtual void MtpMediaDevice::customClicked()':
mtpmediadevice.cpp:1025: error: 'LIBMTP_Get_Storageinfo' was not declared in this scope

Using version 0.1.2 of libmtp

Comment 1 Raul Metsma 2007-01-02 23:34:05 UTC

Created attachment 19083 [details]
Hack to get it compile with libmtp 0.1.2

Had same problem today, there has been some API changes.
I think there is still some work with patch to determine which 
version against amarok is building. Least it is wokring for me now

Comment 2 Raul Metsma 2007-01-02 23:36:11 UTC

Forgot to mention that last patch is against svn

Comment 3 Raul Metsma 2007-01-02 23:50:45 UTC

Created attachment 19084 [details]
Against SVN

Dang next time I need verify files before I upload

Comment 4 Raul Metsma 2007-01-02 23:52:10 UTC

Created attachment 19085 [details]
for amarok 1.4.4

Comment 5 Dirk Mueller 2007-01-12 12:16:49 UTC

Nice, I just came up with the same patch. However, the configure bits are still missing, because it seems those fields are not initialized with an old libmtp.

Comment 6 Dan Meltzer 2007-03-05 17:35:29 UTC

Is this still necessary in 1.4.5?

Comment 7 Kevin Funk 2007-06-23 19:00:58 UTC

*** Bug has been marked as fixed ***.