Bug 3290

Summary: mpd.tainted does not play .m4a files
Product: Mageia Reporter: Derek Jennings <derekjenn>
Component: RPM PackagesAssignee: QA Team <qa-bugs>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal CC: davidwhodgins, jquelin, sysadmin-bugs, tmb
Version: 1Keywords: Triaged, validated_update
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: mpd-0.16.2-3.1.mga1.tainted.rpm CVE:
Status comment:

Description Derek Jennings 2011-11-08 01:19:42 CET
Description of problem:
During validation of bug 1630 it was discovered that the tainted version of mpd would not play .m4a files or add them to the database. The tainted version has a dependency for libfaad and should play .m4a files.

Version-Release number of selected component (if applicable):
mpd-0.16.2-2.mga1.tainted and mpd-0.16.2-3.1.mga1.tainted.rpm

How reproducible:


Steps to Reproduce:
1. add an .m4a file to the music directory of mpd
2. Instruct mpd to update database using for example gnome music client
3. Observe .m4a  file is not included in database. Confirm by looking in /var/log/mpd/mpd.log

Tested on x86_64.

The same .m4a files were observed to play OK in vlc and clementine.
Comment 1 Manuel Hiebel 2011-11-08 23:15:07 CET
Hi, thanks for reporting this bug.
Assigned to the package maintainer.

Keywords: (none) => Triaged
Assignee: bugsquad => jquelin
Source RPM: mpd-0.16.2-2.mga1.tainted and mpd-0.16.2-3.1.mga1.tainted.rpm => mpd-0.16.2-3.1.mga1.tainted.rpm

Comment 2 Derek Jennings 2011-11-09 02:20:52 CET
I think I see the problem.
mpd has no requirement for libffmpeg-devel

# mpd --version
mpd (MPD: Music Player Daemon) 0.16.2 

Copyright (C) 2003-2007 Warren Dukes <warren.dukes@gmail.com>
Copyright (C) 2008-2010 Max Kellermann <max@duempel.org>
This is free software; see the source for copying conditions.  There is NO
warranty; not even MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Supported decoders:

[mad] mp3 mp2
[vorbis] ogg oga
[oggflac] ogg oga
[flac] flac
[sndfile] wav aiff aif au snd paf iff svx sf voc w64 pvf xi htk caf sd2
[audiofile] wav au aiff aif
[faad] aac
[mpcdec] mpc

Supported outputs:

shout null fifo alsa ao oss pulse jack httpd recorder 

Supported encoders:

null vorbis wave flac 

Supported protocols:

file:// http:// 



If I add
BuildRequires:      libffmpeg-devel
to the spec file and rebuild the rpm then I get

# mpd --version
mpd (MPD: Music Player Daemon) 0.16.3 

Copyright (C) 2003-2007 Warren Dukes <warren.dukes@gmail.com>
Copyright (C) 2008-2010 Max Kellermann <max@duempel.org>
This is free software; see the source for copying conditions.  There is NO
warranty; not even MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Supported decoders:

[mad] mp3 mp2
[vorbis] ogg oga
[oggflac] ogg oga
[flac] flac
[sndfile] wav aiff aif au snd paf iff svx sf voc w64 pvf xi htk caf sd2
[audiofile] wav au aiff aif
[faad] aac
[mpcdec] mpc
[ffmpeg] 16sv 3g2 3gp 4xm 8svx aa3 aac ac3 afc aif aifc aiff al alaw amr anim apc ape asf atrac au aud avi avm2 avs bap bfi c93 cak cin cmv cpk daud dct divx dts dv dvd dxa eac3 film flac flc fli fll flx flv g726 gsm gxf iss m1v m2v m2t m2ts m4a m4b m4v mad mj2 mjpeg mjpg mka mkv mlp mm mmf mov mp+ mp1 mp2 mp3 mp4 mpc mpeg mpg mpga mpp mpu mve mvi mxf nc nsv nut nuv oga ogm ogv ogx oma ogg omg psp pva qcp qt r3d ra ram rl2 rm rmvb roq rpl rvc shn smk snd sol son spx str swf tgi tgq tgv thp ts tsp tta xa xvid uv uv2 vb vid vob voc vp6 vmd wav wma wmv wsaud wsvga wv wve

Supported outputs:

shout null fifo alsa ao oss pulse jack httpd recorder 

Supported encoders:

null vorbis lame wave flac 

Supported protocols:

file:// http:// gopher:// rtp:// rtsp:// rtmp:// rtmpt:// rtmps:// 


Now I can add .m4a files to the database and they play OK
Comment 3 Jerome Quelin 2011-11-09 09:28:17 CET
derek - do you want to commit your fix or should i go ahead?
Comment 4 Derek Jennings 2011-11-09 12:04:59 CET
As I do not have commit rights could you do it please Jerome  .
Comment 5 Jerome Quelin 2011-11-10 15:28:01 CET
mpd-0.16.2-3.2.mga1 now available in core/updates_testing and
tainted/updates_testing
please test & push to updates if that's ok.

CC: (none) => jquelin
Assignee: jquelin => qa-bugs

Comment 6 Derek Jennings 2011-11-10 15:58:50 CET
Tested OK on x86_64 both tainted and non tainted.
mpd-0.16.2-3.2.mga1.tainted and mpd-0.16.2-3.2.mga1

Test procedure
-------------
Set up music directory containing .mp3. .ogg, and .m4a files
Install mpd from core_updates_testing with
urpmi --media "Core Updates Testing" mpd
Using mpd client such as Gnome Music Client observe that only mp3 and ogg files appear in database
Upgrade mpd to tainted with 'urpmi --media "Tainted Updates Testing" mpd
Observe database now reports .m4a files and they play OK
Comment 7 Dave Hodgins 2011-11-10 19:44:11 CET
Testing complete on i586 for both the core and tainted versions of mpd.

Could someone from the sysadmin team push the srpm
mpd-0.16.2-3.2.mga1.src.rpm
from Core Updates Testing to Core Updates and the srpm
mpd-0.16.2-3.2.mga1.tainted.src.rpm
from Tainted Updates Testing to Tainted Updates

Keywords: (none) => validated_update
CC: (none) => davidwhodgins, sysadmin-bugs

Comment 8 Thomas Backlund 2011-11-10 22:29:18 CET
Advisory ?

CC: (none) => tmb

Comment 9 Dave Hodgins 2011-11-10 23:16:24 CET
Sorry, forgot to add it.

Advisory:  This update to mpd adds a BuildRequires, so that
libffmpeg-devel is used during the tainted package build,
which is required to allow mpd to access .m4a music files.
Comment 10 Thomas Backlund 2011-11-10 23:25:59 CET
Update pushed.

Status: NEW => RESOLVED
Resolution: (none) => FIXED