Re: [Rhythmbox-devel] Support for .mpc files? (new gstreamer musepack plugin)

[Matthias Saou <thias spam spam spam spam spam spam spam egg and spam freshrpms net>]

Christophe Fergeau wrote :

> Ok, I uploaded a new patch at the same address, I didn't test it at all
> though ;)

Looks much better, I'll try it right away :-)

Matthias

> Christophe
> 
> Le vendredi 10 décembre 2004 à 17:05 +0100, Matthias Saou a écrit :
> > Christophe Fergeau wrote :
> > 
> > > This patch http://testoo.no-ip.com/~teuf/hack/wma.diff (against
0.8.8)
> > > should be enough to add wma and mpc support to rhythmbox, but I'd
like
> > > to have some testers to know if it works properly.
> > 
> > I know nothing about rhythmbox's code, but gave a glance at the
patch...
> > and it looks weird to me :-)
> > 
> > - Why "MusePack" in the video/x-ms-asf ffdec section? Copy/paste typo?
> > - Why do you remove MonkeysAudio from the supported file types?
> > - How come having musepackdec triggers "application/x-apetag" support?
> > 
> > If it's the .mpc and .ape share the same application/x-apetag mime
type,
> > then isn't it a problem since musepack and monkeysaudio are supported
by
> > two different plugins, and that users may have only one of the two
> > installed? Just wondering.


-- 
Clean custom Red Hat Linux rpm packages : http://freshrpms.net/
Fedora Core release 3 (Heidelberg) - Linux kernel 2.6.9-1.681_FC3.r300
Load : 0.29 0.39 0.35