A few mpi files follow wrong syntax for list separators

Bug #632585 reported by Florian Boucault on 2010-09-07
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
media-player-info (Ubuntu)
Undecided
Martin Pitt
Lucid
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

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 :)

John Stowers (nzjrs) wrote :

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

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

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

Free software wins!

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

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
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  Edit
Everyone can see this information.

Other bug subscribers