A few mpi files follow wrong syntax for list separators

Bug #632585 reported by Florian Boucault
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
media-player-info (Ubuntu)
Fix Released
Undecided
Martin Pitt
Lucid
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: media-player-info

The syntax for separating items in a list specified for media-player-info is to use a semicolumn ';'
Some mpi files use a comma ',' instead and that breaks some user applications (e.g. Banshee).

grep , *.mpi | grep -v Vendor | grep -v Product

reveals that 3 files have wrongly formatted OutputFormats:

nokia-n900.mpi
palm-pre.mpi
samsung-galaxy.mpi

Revision history for this message
Florian Boucault (fboucault) wrote :

I did not know where to send the patches upstream so I put the fixes in a bzr branch:

lp:~fboucault/media-player-info/samsung_galaxy_flac

revision 8 contains the fix

Sorry about that but revision 9 contains a fix for another bug whereby the Samsung Galaxy mpi file is missing an output format: audio/flac
Yes! The device supports FLAC out of the box :)

Revision history for this message
John Stowers (nzjrs) wrote :

This fixes n900 support for me, and is the source of this bug

http://ubuntuforums.org/showthread.php?t=1555514

Revision history for this message
Martin Pitt (pitti) wrote :
Changed in media-player-info (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
status: New → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :
Revision history for this message
Florian Boucault (fboucault) wrote :

Free software wins!

Revision history for this message
Martin Pitt (pitti) wrote :

media-player-info (8-1) unstable; urgency=low

  * New upstream bug fix release:
    - Fix OutputFormats syntax (LP: #632585)
    - Add audio/flac as output format for the Samsung Galaxy

 -- Martin Pitt <email address hidden> Mon, 13 Sep 2010 10:45:06 +0200

Changed in media-player-info (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted media-player-info into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in media-player-info (Ubuntu Lucid):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package media-player-info - 12-1~lucid1

---------------
media-player-info (12-1~lucid1) lucid-proposed; urgency=low

  * Backport current media player data to lucid, as per SRU exception.
    Representative SRU bugs:
    - Add HTC Desire (LP: #655185)
    - Fix OutputFormats syntax (LP: #632585)
 -- Martin Pitt <email address hidden> Sun, 09 Jan 2011 05:45:10 -0600

Changed in media-player-info (Ubuntu Lucid):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.