Bug 147598 – sys-apps/hal-0.5.7.1 not recognizing usb cdrom

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description AleX Kiausch 2006-09-14 11:39:48 UTC]

Hello everybody and to whom it may concern,

excuse my stumbing english and maybe the unusual way of reporting.

I found out that 'not recognizing' USB-CDROM is a known issue of hal-0.5.7.1:

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=201533

I made a:
ebuild /usr/portage/sys-apps/hal/hal-0.5.7.1.ebuild unpack
nano /var/tmp/portage/hal-0.5.7.1/work/hal-0.5.7.1/hald/linux2/hotplug.c

applied this changes by hand (don't know the better way):

diff -ru hal-0.5.7.1.orig/hald/linux2/hotplug.c hal-0.5.7.1/hald/linux2/hotplug.c
--- hal-0.5.7.1.orig/hald/linux2/hotplug.c	2006-07-26 11:03:17.000000000 -0700
+++ hal-0.5.7.1/hald/linux2/hotplug.c	2006-08-30 19:50:22.000000000 -0700
@@ -282,7 +282,8 @@
 		size_t len;
 
 		len = strlen(hotplug_event->sysfs.sysfs_path);
-		is_partition = isdigit(hotplug_event->sysfs.sysfs_path[len - 1]) ||
+		is_partition = (isdigit(hotplug_event->sysfs.sysfs_path[len - 1])
+				&& ! strstr(hotplug_event->sysfs.sysfs_path, "/block/sr")) ||
 			       strstr (hotplug_event->sysfs.sysfs_path, "/fakevolume") ;
 
 		if (hotplug_event->action == HOTPLUG_ACTION_ADD) {

patch can be found here:
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=135616

then:
ebuild /usr/portage/sys-apps/hal/hal-0.5.7.1.ebuild compile
ebuild /usr/portage/sys-apps/hal/hal-0.5.7.1.ebuild install
and
ebuild /usr/portage/sys-apps/hal/hal-0.5.7.1.ebuild qmerge

/etc/init.d/hald restart
and now my USB-CDROM get recognized again

greetings!

[Comment 1 Jakub Moc (RETIRED) 2006-09-14 12:26:41 UTC]

This is not a Gentoo Bugzilla product bug, read the descriptions.

[Comment 2 Yaron Tausky 2006-09-16 00:59:37 UTC]

But Gentoo should issue a new ebuild with the relevant patch until upstream makes a new release, as this bug affects many users (anyone who uses the sr driver).

[Comment 3 Doug Goldstein (RETIRED) 2006-09-16 09:22:32 UTC]

Yaron: You're misunderstanding. The original reporter assigned the bug to the category that says "Bugzilla Bugs only!". And that's what Jakub was talking about. So basically the bug didn't get assigned to me like it should have. It got assigned to the guys that maintain the server that hosts Bugzilla.

[Comment 4 Doug Goldstein (RETIRED) 2006-09-16 10:35:37 UTC]

Commited as hal-0.5.7.1-r1. Please test.

[Comment 5 Yaron Tausky 2006-09-16 12:47:20 UTC]

Oh, sorry, my bad. Anyway, just emerged the new ebuild, and now HAL recognizes CDs as it should (a.k.a. "Works for me!"). Thanks!

[Comment 6 Doug Goldstein (RETIRED) 2006-09-16 13:59:04 UTC]

Fixed and masked.