[mythtv-commits] Ticket #7135: multiple [mythfrontend] <defunct>

[MythTV]

#7135: multiple [mythfrontend] <defunct>
---------------------------------------+------------------------------------
 Reporter:  simons.philippe@…          |       Owner:  ijr           
     Type:  defect                     |      Status:  infoneeded_new
 Priority:  minor                      |   Milestone:  unknown       
Component:  MythTV - General           |     Version:  head          
 Severity:  medium                     |     Mlocked:  0             
---------------------------------------+------------------------------------

Comment(by Bill Meek <llibkeem@…>):

 The attached changes work on a 9.04 mythbuntu distribution.
 If there are still distributions without udevadm, this 'fix'
 will give them the same problem we're seeing in this ticket.

 Point me in the right direction and give me a shove and I'd
 be happy to make a real fix.

 Details:

 trunk/mythtv/libs/libmyth/mediamonitor-unix.cpp executes
 udevinfo, which doesn't exist in mythubuntu 9.04 and
 ubuntu 9.10 (the two distributions I have.)


 {{{
 % type udevinfo
 -bash: type: udevinfo: not found

 % type udevadm
 udevadm is /sbin/udevadm
 }}}


 If the device is valid, both return the full path, as in:


 {{{
 udevinfo -q name -rp /sys/block/sdd      (existing code)
 udevadm info -q name -rp /sys/block/sdd  (proposed)
     /dev/sdd
 }}}


 In the error case, (... -q name -rp /sys/block/sd'''foo''')
 the existing 'udevinfo' code checks for a response of:


 {{{
 device not found in database
 }}}


 but udevadm returns:


 {{{
 device path not found
 }}}


 Also, if udevinfo is used but linked to udevadm, the following will appear
 in mythfrontend.log:


 {{{
 MMUnix::GetDeviceFile(/sys/block/sdd) - udevinfo error...
 the program '/usr/local/bin/mythfrontend' called 'udevinfo',
 it should use 'udevadm info <options>', this will stop working
 in a future release
 }}}

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/7135#comment:14>
MythTV <http://www.mythtv.org/>
MythTV