Opened 11 years ago

Closed 10 years ago

#6276 closed task (fixed)

MythTV shouldn't use cdrecord for drive info

Reported by: Nigel Owned by: Nigel
Priority: minor Milestone: unknown
Component: MythTV - General Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

If a frontend starts up while a CD drive is busy burning a disc, it will hang for several minutes on:

2009-02-18 18:41:04.021 MonitorRegisterExtensions(0x100, gif,jpg,png)
2009-02-18 18:41:05.808 Failed to run 'cdrecord --scanbus -dev=ATA'

before the main menu is drawn.

Attachments (4)

00-6276-cdwriter.patch (1.6 KB) - added by Matthew Wire <devel@…> 11 years ago.
Add defaultCDWriter function to mythmediamonitor
00-6276-mythmusic-cddevices.patch (5.1 KB) - added by Matthew Wire <devel@…> 11 years ago.
Switch mythmusic to use mythmediamonitor by default to detect cd devices
00-6276-mythvideo-dvddevice.patch (1.1 KB) - added by Matthew Wire <devel@…> 11 years ago.
Switch mythvideo to use mythmediamonitor by default to detect cd/dvd devices
00-6276-mythmusic-cddevices.2.patch (5.1 KB) - added by Matthew Wire <devel@…> 10 years ago.
Refresh against 20301. Includes in playlist.cpp had been rearranged

Download all attachments as: .zip

Change History (15)

comment:1 Changed 11 years ago by Dibblah

Owner: changed from Isaac Richards to Nigel
Status: newassigned
Type: defecttask

comment:2 Changed 11 years ago by EnderTheThird

I can confirm this for r20065 on 64-bit Ubuntu 8.10 with VDPAU. Very annoying bug. I wasn't burning a disc when Myth was started though. Let me know if you need any additional information.

comment:3 Changed 11 years ago by eric.bosch@…

Same here...

Changed 11 years ago by Matthew Wire <devel@…>

Attachment: 00-6276-cdwriter.patch added

Add defaultCDWriter function to mythmediamonitor

Changed 11 years ago by Matthew Wire <devel@…>

Switch mythmusic to use mythmediamonitor by default to detect cd devices

Changed 11 years ago by Matthew Wire <devel@…>

Switch mythvideo to use mythmediamonitor by default to detect cd/dvd devices

comment:4 Changed 11 years ago by Matthew Wire <devel@…>

Please try 00-6276-cdwriter.patch and 00-6276-mythmusic-cddevices.patch. The first adds a defaultCDWriter() function to mythmediamonitor, the second patch removes cdrecord device detection from mythmusic and changes it to use mythmediamonitor instead.

00-6276-mythvideo-dvddevice.patch just changes mythvideo to use mythmediamonitor by default for device detection and is not required to fix this bug but makes things consistent between plugin configurations

comment:5 Changed 11 years ago by Nigel

(In [20119]) The trivia part of Matthew Wire's cdrecord removal patches. See #6276.

comment:6 Changed 11 years ago by Nigel

(In [20232]) Make initial default VCD and DVD device an auto or user select one. See #6276

Changed 10 years ago by Matthew Wire <devel@…>

Refresh against 20301. Includes in playlist.cpp had been rearranged

comment:7 Changed 10 years ago by Nigel

Resolution: fixed
Status: assignedclosed

(In [20556]) Remove erroneous calls to cdrecord on plugin startup. Patch thanks to Matthew Wire. Fixes #6276

comment:8 Changed 10 years ago by Nigel

Resolution: fixed
Status: closednew

I should apply this to 0.21-fixes also.

comment:9 Changed 10 years ago by Nigel

(In [20557]) Backport [20119] and [20120] from trunk. Refs #6276

comment:10 Changed 10 years ago by Nigel

(In [20563]) Remove erroneous calls to cdrecord on plugin startup. Ported from trunk. Patch from to Matthew Wire. Refs #6276.

comment:11 Changed 10 years ago by Nigel

Resolution: fixed
Status: newclosed

OK, most uses of cdrecord are now removed from trunk and 21-fixes. MythMusic still uses it to blank and burn CDs, but that is mostly harmless.

Note: See TracTickets for help on using tickets.