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 playerAssignee: 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:
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 ***.