Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.
Bug 85224 - (IT_69202) magicdev does not automount DVDs
magicdev does not automount DVDs
Product: Red Hat Enterprise Linux 3
Classification: Red Hat
Component: magicdev (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: John (J5) Palmieri
David Lawrence
Depends On:
  Show dependency treegraph
Reported: 2003-02-26 14:56 EST by Mike McLean
Modified: 2013-03-13 00:45 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2005-10-14 14:48:34 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
mount DVDs as well (1.19 KB, patch)
2003-02-27 07:39 EST, Bastien Nocera
no flags Details | Diff
only launch the dvd player if we have a video dvd (2.48 KB, patch)
2003-03-29 06:14 EST, Bastien Nocera
no flags Details | Diff
updated patch (4.45 KB, patch)
2003-05-16 10:57 EDT, Bastien Nocera
no flags Details | Diff
yet another patch (5.56 KB, patch)
2003-05-16 13:05 EDT, Bastien Nocera
no flags Details | Diff

  None (edit)
Description Mike McLean 2003-02-26 14:56:08 EST
* -re0225....
* magicdev-1.1.4-3

CDs are automounted, but DVDs are not.
Comment 1 Owen Taylor 2003-02-26 15:27:10 EST
Not expected to, actually. Might be nice, but we'd have to figure
out the interaction with play-a-dvd, since you can't distinguish
a Video DVD from a data DVD.
Comment 2 Bastien Nocera 2003-02-27 07:38:49 EST
I *think* we could also mount the DVD and still be able to access it via the DVD
program. We could then add something to Nautilus to show that it's a DVD, and
probably get the name of the film if it's a Video DVD.

Comment 3 Bastien Nocera 2003-02-27 07:39:23 EST
Created attachment 90398 [details]
mount DVDs as well
Comment 4 Bastien Nocera 2003-03-29 05:53:59 EST
Owen, I finally had time to test this patch, and it does what one would expect.
Can you apply the patch ?
Comment 5 Bastien Nocera 2003-03-29 06:14:21 EST
Created attachment 90783 [details]
only launch the dvd player if we have a video dvd

2003-03-29  Bastien Nocera  <hadess@hadess.net>

	* daemon.c: (is_video_dvd), (app_device_activate_cdrom),
	(app_device_mount): mount DVDs all the time, but only launch the
	DVD player if it's a video DVD, mount the media synchronously so we
	know that it will be mounted when we return, check if mount was
	successful before saying it is mounted
Comment 6 Bastien Nocera 2003-05-16 10:57:35 EDT
Created attachment 91724 [details]
updated patch

Hmm, I don't remember what you told me was wrong with the previous patch,
But I updated it not to change dev->did_mount, and check if the dvd was mounted
before running the DVD player program.
Comment 7 Owen Taylor 2003-05-16 11:53:15 EDT
I seem to remember that what we discussed was that if we mount DVD's,
then playing DVDs should be handled in the same location as 
autorun, not in the same place we handle playing CD's.
Comment 8 Bastien Nocera 2003-05-16 13:05:31 EDT
Created attachment 91734 [details]
yet another patch
Comment 9 Bastien Nocera 2003-12-14 19:13:25 EST
Merged upstream

2003-12-15  Bastien Nocera  <hadess@hadess.net>
        * .cvsignore: upd
        * daemon.c: (is_video_dvd), (app_device_activate_cdrom),
        (app_device_mount), (run_me), (app_device_autorun):
        make magicdev automount data DVDs, and double-check if a DVD is
        a video one before launching the application
Comment 10 John (J5) Palmieri 2004-08-25 13:43:46 EDT
Comment 11 John (J5) Palmieri 2005-04-05 11:28:46 EDT
New packages going through the errata process for RHEL3-QU6 release. 
Comment 12 John (J5) Palmieri 2005-10-14 14:48:34 EDT
This package did not fit the update guidelines (specificly the one about not
adding new functionality) so it was dropped from the QU6 release and is now
being closed WONTFIX.

Note You need to log in before you can comment on or make changes to this bug.