| Summary: | Patch for IVMAN to recognize some CD ROMS | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Mario Doria <mariodoria> |
| Component: | New packages | Assignee: | Gentoo Linux bug wranglers <bug-wranglers> |
| Status: | RESOLVED INVALID | ||
| Severity: | normal | ||
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
what package is this? It's a userspace automounter in breakmygentoo's overlay. Yeah, we don't take bugs for BMG stuff. Push this to them. Marking INVALID. |
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.