Bug 263103

Summary:	[PATCH] Support APE format in phonon
Product:	[Frameworks and Libraries] phonon-backend-gstreamer	Reporter:	Weng Xuetian <wengxt>
Component:	general	Assignee:	Harald Sitter <sitter>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	martin.sandsmark, myriam, tdfischer
Priority:	NOR
Version:	4.4.3
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	phonon-gstream patch for ape

Description Weng Xuetian 2011-01-14 06:31:38 UTC

Version:           4.4.3 (KDE 4.6.x) (using KDE 4.5.95) 
OS:                Linux

Actually ffmpeg and mplayer can play ape format without problem, but phonon declare not support it yet.

Reproducible: Always

Steps to Reproduce:
1. Open amarok
2. Browse ape file

Actual Results:  
Ape file don't show up.

Expected Results:  
Ape file show correctly.

I found some older patch for phonon and author says it has been tested on magiclinux.

And maybe some other format that backend actually support is not included too.

http://ftp.magiclinux.org.cn/nihui/kde4/patch/phonon-4.3.80-gstreamer-ape_mediatype.patch
http://ftp.magiclinux.org.cn/nihui/kde4/patch/phonon-4.3.80-mplayer-ape_mediatype.patch

Comment 1 Harald Sitter 2011-01-15 22:33:43 UTC

Those URLs do not load for me. Please attach the relevant patches to this bug report.
Thank you.

Comment 2 Harald Sitter 2011-01-15 22:58:35 UTC

Also, please file individual bug reports for the affected backends, I am claiming this one into, though the gstreamer backend does dynamic mimetype listing, so I would expect that gstreamer itself maps a wrong mimetype causing this behaviour.

Comment 3 Weng Xuetian 2011-01-22 12:44:13 UTC

Created attachment 56313 [details]
phonon-gstream patch for ape

Comment 4 Myriam Schweingruber 2011-04-27 21:18:58 UTC

reassigning to the new bugzilla product for better bug tracing of the various backends. Sorry for the noise.

Comment 5 Myriam Schweingruber 2011-04-29 00:09:38 UTC

Sorry, we seem to have missed your patch. Could you please submit it to http://reviewboard.kde.org? Thank you.

Comment 6 Torrie Fischer 2011-04-29 02:23:50 UTC

They load fine to me. Applying...

Comment 7 Torrie Fischer 2011-04-29 02:47:24 UTC

Git commit ada3aa003a5185870e6f3c191f4577defb7ace92 by Trever Fischer.
Committed on 29/04/2011 at 02:38.
Pushed by tdfischer into branch 'master'.

Monkeypatch to fix gstreamer-ffmpeg not reporting ape

Patch contributed by Weng Xuetian.

BUG: 263103

M  +1    -0    gstreamer/backend.cpp     
M  +1    -1    gstreamer/gstreamer.desktop.cmake     

http://commits.kde.org/phonon-gstreamer/ada3aa003a5185870e6f3c191f4577defb7ace92