Modify ↓

Ticket #6276 (closed task: fixed)

Opened 3 years ago

Last modified 3 years ago

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

00-6276-cdwriter.patch​ (1.6 KB) - added by Matthew Wire <devel@…> 3 years ago.
Add defaultCDWriter function to mythmediamonitor
00-6276-mythmusic-cddevices.patch​ (5.1 KB) - added by Matthew Wire <devel@…> 3 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@…> 3 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@…> 3 years ago.
Refresh against 20301. Includes in playlist.cpp had been rearranged

Change History

comment:1 Changed 3 years ago by Dibblah

  • Owner changed from ijr to nigel
  • Status changed from new to assigned
  • Type changed from defect to task

comment:2 Changed 3 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 3 years ago by eric.bosch@…

Same here...

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

Add defaultCDWriter function to mythmediamonitor

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

Switch mythmusic to use mythmediamonitor by default to detect cd devices

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

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

comment:4 Changed 3 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 3 years ago by nigel

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

comment:6 Changed 3 years ago by nigel

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

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

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

comment:7 Changed 3 years ago by nigel

  • Status changed from assigned to closed
  • Resolution set to fixed

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

comment:8 Changed 3 years ago by nigel

  • Status changed from closed to new
  • Resolution fixed deleted

I should apply this to 0.21-fixes also.

comment:9 Changed 3 years ago by nigel

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

comment:10 Changed 3 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 3 years ago by nigel

  • Status changed from new to closed
  • Resolution set to fixed

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.

View ↑

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.