Opened 13 years ago

Closed 13 years ago

#2990 closed defect (fixed)

Eject CD Broken in CD Import

Reported by: devel@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythmusic Version: head
Severity: high Keywords: mythmusic import cd eject
Cc: Ticket locked: no

Description

If you insert a CD (of images) into the drive when in the Import CD dialog MythTV get stuck in an infinite loop.

  1. CD is detected as MEDIATYPE_MGALLERY.
  2. CD is mounted.
  3. MediaEvent? posted.
  4. CD unmounted.

The following error recurs infinitely: ioctl returned -1 Failed on cd_eject: Input/output error

A logfile is attached. I'm running MythTV latest SVN. I think this is only occurring since the cd import updates in Changeset 12582. Testing on a laptop with a DVD/CDWriter that cannot close the drive automatically.

Attachments (2)

mythmusic-cdimporterror.log (4.5 KB) - added by devel@… 13 years ago.
mythfrontend error log
cdrip.diff (1.8 KB) - added by paulh 13 years ago.
Patch to disable MediaMonitor? while ripping cds

Download all attachments as: .zip

Change History (7)

Changed 13 years ago by devel@…

Attachment: mythmusic-cdimporterror.log added

mythfrontend error log

Changed 13 years ago by paulh

Attachment: cdrip.diff added

Patch to disable MediaMonitor? while ripping cds

comment:1 Changed 13 years ago by paulh

Can you try the attached patch. It just disables the MediaMonitor? while ripping CDs.

comment:2 Changed 13 years ago by devel@…

Sorry for the delay in testing this.

The patch seems to make things work correctly for me here.

comment:3 Changed 13 years ago by Nigel

Patch looks good to me, except for the ifndef WIN32 stuff. The proper way to do this is to fix the MediaManager? so that it is created (even if it is a dummy class that does nothing) on that platform. I will try to modify mythmediamonitor.cpp so that it creates an empty class on Windows.

comment:4 in reply to:  3 Changed 13 years ago by paulh

Replying to nigel:

Patch looks good to me, except for the ifndef WIN32 stuff. The proper way to do this is to fix the MediaManager? so that it is created (even if it is a dummy class that does nothing) on that platform. I will try to modify mythmediamonitor.cpp so that it creates an empty class on Windows.

That's fair enough, I'll commit the patch with the if ndef's for now because without them it will break the WIN32 build although I don't think the WIN32 build is compilable yet anyway there's no need to make things worse.

comment:5 Changed 13 years ago by paulh

Resolution: fixed
Status: newclosed

(In [12709]) Disable the MediaMonitor? while ripping CD's. Fixes #2990.

Note: See TracTickets for help on using tickets.