iPod video mpi file looks incorrect (and hence udev rules is incorrect)

Bug #576738 reported by John Stowers
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
media-player-info
Fix Released
Undecided
Unassigned
media-player-info (Ubuntu)
Fix Released
High
Martin Pitt

Bug Description

Binary package hint: media-player-info

I am trying to add iPod video support to Conduit using python-gudev. After some research (note: i dont have one of these devices to test with) I believe the media-player-info file for these devices is incorrect and hence the /lib/udev/rules.d/40-usb-media-player.rules file is also incorrect (as it is generated from the mpi files)

It has the following lines

ATTRS{vendor}=="*Apple*" , ATTRS{model}=="*iPod*" , ENV{ID_MEDIA_PLAYER}="apple-ipod" ENV{UDISKS_PRESENTATION_ICON_NAME}="multimedia-player-apple-ipod"

ATTRS{vendor}=="*Apple*" , ENV{ID_MEDIA_PLAYER}="apple-1261_1262"

AFAICT the second rule needs to be more specific, and include ATTRS{idProduct}=="1261|1262"

This is not included in the generated rules file because the mpi line reads

ProductID=1261;1262

It is the only line that has a ProductID= stanza. I believe that (following the convention of other mpi files) the mpi should instead read like

DeviceMatch=usb:1261:1262;

that means the next time the udev rules is generated using mpi2udev.py things should work correctly

Related branches

Revision history for this message
Jonathan Ernst (jonathan.ernst) wrote :

I have created a patch that fixes this issue (and the missing icon issue as well) in the freedesktop.org bug tracker : https://bugs.freedesktop.org/show_bug.cgi?id=30253

description: updated
Revision history for this message
Charlie Kravetz (cjkgeek) wrote :

Upstream bug added by mistake. This does not apply to this bug report.

Changed in media-player-info:
importance: Unknown → Undecided
status: Unknown → New
Revision history for this message
Martin Pitt (pitti) wrote :

Indeed it is meant to translate "ProductID", but this somehow doesn't work. Thanks for pointing out!

Changed in media-player-info (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
importance: Undecided → High
status: New → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :
Changed in media-player-info (Ubuntu):
status: In Progress → Fix Committed
Changed in media-player-info:
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package media-player-info - 9-1

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

  * New upstream release:
    - New players: Samsung YP-M1 (LP: #573219), iRiver E30 (LP: #601718),
      Motorola Dext/Cliq (LP: #642944), Motorola Droid debugging ID
      (LP: #642014)
    - Fix IDs for Sony Ericsson Xperia X10 (LP: #616910)
    - Fix IDs for video iPods (LP: #576738)
  * Add debian/watch.
 -- Martin Pitt <email address hidden> Mon, 20 Sep 2010 15:01:28 +0200

Changed in media-player-info (Ubuntu):
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.