Use MythMediaMonitor in plugins

[Nigel]

MythMusic, and mtd (the transcoding part of MythDVD) use active polling to detect CD/DVD insertion. To get these working with multiple drives, or on OS X, they need to use MythMediaMonitor. Fairly simple - see [11333]

[Nigel]

(In [11602]) Tidyup of DVD/VCD/SVCD directory detection. Tested on OS X, but have only tried compiling on Linux. Includes Mac OS X audio detection hack which will be needed for plugin MythMediaMonitor upgrade. See #2598

[awk@…]

Patches to mtd to use DiskArbitration? Framework on Mac OS X

[awk@…]

mtd-darwin.cpp - Darwin subclass of mtd disk checking thread uses DiskArbitration?

[awk@…]

mtd-darwin.h - Darwin subclass of mtd disk checking thread uses DiskArbitration?

[awk@…]

Using the MythMediaMonitor classes directly looks really rather complicated - since the Media Monitor posts custom events to the main thread for handling in a window whenever the media changes (disks inserted etc).

Since mtd doesn't (and wouldn't I think) ever have a window to handle such events it seemed more straight forward to leverage the DiskArbitration? changes from MythMediaMonitor and create a darwin/disk arbitration specific thread to handle the OS disk insertion/removal events.

[awk@…]

I should add that my patches and new files are only for mtd - I don't use mythmusic myself so I've not tackled whatever media 'sniffing' it does.

[myth@…]

I've actually been working on making MythMusic use MythMediaMonitor to look for CDs. I'm nearly done, just need to fix PlaybackBoxMusic? to it can actually play the audio tracks when detected. I hope I'll have a submittable patch sometime next week.

[myth@…]

redo cd monitoring to use mediamonitor

[myth@…]

Description of cdstore.patch ;

Introduce two new classes, StoreManager? and Store; StoreManager creates Stores when given a MythMediaDevice? via an event from MediaMonitor? (passed to it from whoever is registered as the event handler). The returned store is a subclass that can handle the actual store. Currently only a subclass for CDs is implemented, but I've also got an iPod store in the works.

Store is responsible for creating the metadata objects that represent the store, as well as it's name. The intent here is that this object will later be the one that can add/remove tracks from a portable player (iPod...).

Add a call to CdDecoder? so you can override the configured device. If there's an event on a CD device, it should be up to MediaMonitor to decide whether to issue an event or not, and if, CdDecoder should just accept the device it's given.

Give GenericTree? two templated methods findNode and isNodeInPath that traverses a path and looks for a specific in a path respectively. findNode replaces recursiveNodeFinder and isNodeInPath replaces checkNode. recursiveNoderFinder seemed overly complex and didn't even look like it was correct.

Change UIManagedTreeListType to use these and to not return a pointer to the path when getRouteToActive is called. All calls would dereference the pointer and copy it anyway.

Change MediaMonitor to post events for OPEN and UNPLUGGED as well for USEABLE and MOUNTED. Otherwise listeners couldn't detect that a CD had been removed.

Added some helpers (setPathTo/getPathTo) to playbackbox.cpp to replace the many many places where it would construct a path to "Active Queue".

Make DatabaseBox? and PlaybackBoxMusic? use the StoreManager, and kinda prepare PlaybackBoxMusic to be able to handle multiple connected devices. For DatabaseBox?, this also kills off ReadCDThread, as this is now handled by StoreManager, which does the loading in a thread.

Change MusicNode::writeTree to assign the given counter as the int component of the GenericTree node, also return the created node so that a caller (PlaybackBoxMusic) can remember which node was added. This lets PlaybackBoxMusic easily insert new nodes at the root for new devices, and when the device is removed, move out from that branch if necessary and remove it from the tree.

[myth@…]

updated patch, fixes race condition

[myth@…]

updated to latest svn and fixes a wrong include.

[Nigel]

(In [13336]) Don't lookup CD device in decoder, set it in the parent. See #2598. See #3077

[Nigel]

(In [13362]) Multiple CD drive with media manager in Music Plugin. See #3077. See #2598

[Nigel]

(In [13364]) Multiple DVD drive with media manager in Video Plugin. See #3077. See #2598. Not real happy about the structure here (TitleDialog? and DVDRipBox need to share the same dvd_device, and it the needs to get passed to mtd for use?), but that will have to wait for some extra, um, "brave pills". Until then, setting DVDDeviceLocation to "default" should let you play from multiple drives, but will break ripping.

[Nigel]

(In [13829]) Add commands to allow setting (and forgetting) DVD device path. See #2598. Should enable multiple-drive support. Changes in the caller (DVDRipBox) will be next, and when the MediaMonitor? chain is modified to also send OPEN/UNPLUGGED/NOTMOUNTED messages, the disc checking thread can be removed

[Nigel]

(In [13838]) Add code so MythMusic tells the MTD what drive to monitor/rip. Users with multiple DVD drives should now be able to rip from all of them. Either through the MythMediaMonitor, or by setting DVDDeviceLocation to default. See #2598

[Nigel]

(In [13877]) Now also send eject/unplug events. See #2598. Also reduces VERBOSE output

[Nigel]

(In [13879]) Device ignore list - see #3663. Send eject events - see #2598. Also changes the way events are send (now as a side-effect of status change, same as Unix)

[Nigel]

(In [13937]) A few missing parts in [13879]. See #3662. See #2598.

[Nigel]

(In [14177]) Add another method for plugins to use, tidy some login and comments. See #2598

[stuartm]

Nigel, still working on this or can the ticket now be closed?

[robertm]

Appears fixed, can be re-opened as necessary if not. No change in three years.