Bug 188462 - Review Request: bmp-musepack - Mpegplus (mpc) playback plugin for BMP
Summary: Review Request: bmp-musepack - Mpegplus (mpc) playback plugin for BMP
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Thorsten Leemhuis (ignored mailbox)
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-04-10 10:14 UTC by Matthias Saou
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-04 11:50:46 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Matthias Saou 2006-04-10 10:14:09 UTC
Spec Name or Url: http://svn.rpmforge.net/svn/trunk/rpms/bmp-musepack/
SRPM Name or Url: http://freshrpms.net/rpm/bmp-musepack
Description:
This package contains an MPC playback plugin for BMP (Beep Media Player),
a media player that uses a skinned user interface based on Winamp 2.x skins,
and is based on ("forked off") XMMS.

Now that libmpcdec has been included into Extras, this plugin can be too.

Comment 1 Eric Tanguy 2006-04-12 06:10:47 UTC
Needs work:
* Source 0 is not available
(http://musepack.origean.net/files/linux/plugins/bmp-musepack-1.2.tar.bz2)
  (wiki: QAChecklist item 2)
* BuildRoot should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
  (wiki: PackagingGuidelines#BuildRoot)
* BuildRequires: gcc-c++ should not be included
  (wiki: PackagingGuidelines#Exceptions)

Comment 2 Michael Schwendt 2006-04-12 09:44:12 UTC
* A huge blocker is that it fails at run-time. The code formerly
taken from XMMS has not been updated to stay in sync with the
progress in BMP.

The plugin still uses plain fopen() and also libmpcdec's internal
stream I/O interface whereas BMP has advanced to implement VFS
features. The files/URLs passed on from BMP to the plugins have
prefixes like 'file://' or 'http://' and escaped special characters,
e.g. %20 for space. The plugin fails to read such input:

[xmms-musepack] mpcGetSongInfo is unable to open
file:///home/qa/files/audio/test/Paul%20Blackford%20-%20The%20Great%20Bath%20-%20Ambient%20Remake.mpc
: No such file or directory
[xmms-musepack] mpcGetSongInfo is unable to open http://localhost/test/test.mpc
: No such file or directory

While it would be trivial to patch the code and strip off a 'file://'
prefix to make it work with local files, it would be more work to
replace libmpcdec's stream I/O functions with something compatible
with the VFS features.

* Upstream download location has moved to:
http://files2.musepack.net/linux/plugins/bmp-musepack-1.2.tar.bz2


Comment 3 Matthias Saou 2006-05-03 20:07:58 UTC
I've contacted the author about this issue. Thanks a lot for spotting it, as I
thought I had some strange ape/id3 tag problem on the mpc files I was trying to
open with that plugin...

Comment 4 Matthias Saou 2006-07-04 11:50:46 UTC
Closing as WONTFIX since the author doesn't seem to have time or motivation to
fix the plugin, and the newer bmpx is gstreamer-based now anyway.


Note You need to log in before you can comment on or make changes to this bug.