Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 60697 - Patch for IVMAN to recognize some CD ROMS
Summary: Patch for IVMAN to recognize some CD ROMS
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Gentoo Linux bug wranglers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-17 11:33 UTC by Mario Doria
Modified: 2004-08-17 13:51 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mario Doria 2004-08-17 11:33:17 UTC
ivman reports the following after inserting a CDROM into my laptop's drive:
hal_interface.c:112 (hal_device_removed) Device removed: /org/freedesktop/Hal/devices/block_Fahrenheit
manager.c:250 (ivm_umount_if_needed) This device ((null)) wasn't mounted by me, ignoring...
hal_interface.c:42 (hal_device_added) New Device: /org/freedesktop/Hal/devices/block_Fahrenheit
hal_interface.c:80 (hal_device_added) Changed: /dev/hda
libhal.c 823 : Error sending msg: No property storage.drive_type on device with id /org/freedesktop/Hal/devices/block_Fahrenheit

** (process:10686): WARNING **: manager.c/223: cannot get storage.drive_type 

Reproducible: Always
Steps to Reproduce:
1. Insert a CD
2. Run ivman with debug on and fork off
3. Watch ivman




I made a patch to fix this, it checks if the CD inserted is a volume, and if it
is, it gets its data from the parent (?) block device.

--- src/manager.c       2004-06-19 22:02:41.000000000 -0500
+++ src/manager.c       2004-08-16 03:08:17.674413703 -0500
@@ -218,6 +218,16 @@

         media_type = hal_device_get_property_string(hal_ctx, udi,
                                                     "storage.drive_type");
+
+       if((hal_device_get_property_bool(hal_ctx, udi,"volume.is_disc") == TRUE)
&& (!media_type))
+       {
+               DEBUG("forcing media_type");
+
+               media_type = hal_device_get_property_string(hal_ctx,
+               hal_device_get_property_string(hal_ctx, udi,"block.storage_device"),
+                                                    "storage.drive_type");
+       }
+
         if (!media_type)
         {
                 warn("cannot get storage.drive_type\n");


Because this is patched on the ivman-cvs sources, you should also patch your ebuild.
Comment 1 Seemant Kulleen (RETIRED) gentoo-dev 2004-08-17 12:28:14 UTC
what package is this?
Comment 2 Stephen Bennett (RETIRED) gentoo-dev 2004-08-17 12:29:56 UTC
It's a userspace automounter in breakmygentoo's overlay.
Comment 3 Peter Johanson (RETIRED) gentoo-dev 2004-08-17 13:51:13 UTC
Yeah, we don't take bugs for BMG stuff. Push this to them. Marking INVALID.