Bug 186835

Summary:	copy to MTP device does not copy FLAC files
Product:	[Applications] amarok	Reporter:	herr-schneider.m
Component:	Collections/MTP player	Assignee:	Amarok Bugs <amarok-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aikawarazuni, oliver.henshaw, simonandric5
Priority:	NOR
Version First Reported In:	2.1-SVN
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	amarok --debug

Description herr-schneider.m 2009-03-11 08:27:16 UTC

Version:           2.1-SVN (using 4.2.1 (KDE 4.2.1) "release 103", KDE:KDE4:Factory:Desktop / openSUSE_11.1)
Compiler:          gcc
OS:                Linux (i686) release 2.6.27.19-3.2-trace

Whenever I do a copy of FLAC files to my MTP device (IAUDIO 7), nothing happens, files get not copied, no error raised. Copying MP3 works like a charm. Device supports FLAC.
suspicious amarok --debug attached

Comment 1 herr-schneider.m 2009-03-11 08:29:34 UTC

Created attachment 32017 [details]
amarok --debug

Comment 2 Mikko C. 2009-05-21 12:14:33 UTC

I suppose this is the problem:
amarok:   [MtpHandler] No filetype found by Amarok filetype                                                                         
amarok:   [MtpHandler] We do not support the extension . "flac"                                                                     

Is it still valid with current svn? Also, which version of libmtp are you using?

Comment 3 Myriam Schweingruber 2009-05-31 09:52:20 UTC

Waiting for info

Comment 4 Oliver Henshaw 2009-06-01 22:07:42 UTC

I see this. I just reproduced this with the fedora 11 preview with their build of amarok 2.1 final installed.

Packages:
amarok-2.1-1.fc11.i586
libmtp-0.3.7-1.fc11.i586

Comment 5 Alejandro Wainzinger 2009-06-02 00:10:01 UTC

SVN commit 976528 by awainzinger:

During port of MTP to 2.0, somehow code initializing supported filetypes got lost, but never noticed since MTP dev only really had mp3/ogg files.
Thanks to Marcus Schneider
~lart xevix
BUG:186835


 M  +2 -0      ChangeLog  
 M  +33 -0     src/collection/mtpcollection/handler/MtpHandler.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=976528

Comment 6 Oliver Henshaw 2009-06-23 21:51:38 UTC

I still have this problem with 2.1.1. Looks like http://websvn.kde.org/trunk/extragear/multimedia/amarok/src/collection/mtpcollection/handler/MtpHandler.cpp?r1=976528&r2=976527&pathrev=976528 fixed the bug for almost every filetype apart from flac.

Comment 7 Myriam Schweingruber 2009-06-23 21:54:20 UTC

This seems not to be solved for FLAC. Alejandro, could you have a look at that, please?

Comment 8 Alejandro Wainzinger 2009-06-24 02:51:18 UTC

SVN commit 986044 by awainzinger:

Added support for all conceivably useful filetypes for an MTP device to recognize according to libmtp, including flac.
BUG:186835

 M  +19 -7     MtpHandler.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=986044