[mythtv] Ticket #6822: win32 patch to display drive letter instead of the devicePath when having multiple optical drives

Nigel Pearson nigel at ind.tansu.com.au
Thu Aug 13 05:04:30 UTC 2009


> Comment(by Jonathan Martens <jonathan at snetram.nl>):
>
>  Replying to [comment:1 nigel]:
>> I think it would be better to patch the code in the Windows  
>> subclass of
>> the media monitor that is incorrectly filling in the volumeID from
>>  GetVolumeNameForVolumeMountPoint().
>
>  Is it really as the drivedetection algorithm seems to spit out the  
> proper
>  labels in the MythFrontend logfile:
>
>  {{{
>  2009-08-08 17:50:26.328 Creating inactive MediaMonitor and static  
> device
>  list
>  2009-08-08 17:50:26.328 IgnoreDevices=
>  2009-08-08 17:50:26.328 Found CD Drive : E:\
>  2009-08-08 17:50:26.359 Found CD Drive : F:\
>  2009-08-08 17:50:26.359 Initial device list: E:\ (E:\), F:\ (F:\)

The standard code outputs m_RealDevice (m_DeviceModel).

m_VolumeID is only used later, when we know what media is mounted in  
that device.
I suspect the strings like '\\.\{8213csad-3422dda-etc}' could be used  
for m_KeyID.
m_VolumeID would be something like "Audio CD" or "THEITALIANJOB2006".



P.S. I assume you have patches for other media monitor parts?
At the moment for me, the standard code does:

2009-08-13 14:48:24.703 Creating MediaMonitor, SendEvents=false
2009-08-13 14:48:24.703 IgnoreDevices=
2009-08-13 14:48:24.734 Error. Couldn't create MythMediaDevice.
2009-08-13 14:48:24.734 Starting MediaMonitor
2009-08-13 14:48:24.750 Executing '/bin/mount A:\'
2009-08-13 14:48:24.750 performMountCmd() attempted to find mounted  
media, but failed?

--
Nigel Pearson, nigel at ind.tansu.com.au|Excuse me, have you seen a     |
Telstra Net. Eng., Sydney, Australia | blowfish driving a sports car?|
Office: 9202 3900    Fax:  9261 3912 |      (points ... mutters)     |
Mobile: 0408 664435  Home: 9792 6998 |             Bloody Torchwood! |






More information about the mythtv-dev mailing list