Bug 117964

Summary:	hald-addon-acpi: does not react and forward ACPI events to HAL
Product:	[openSUSE] SUSE LINUX 10.0	Reporter:	Danny Al-Gaaf <dalgaaf>
Component:	Basesystem	Assignee:	Danny Al-Gaaf <dalgaaf>
Status:	VERIFIED FIXED	QA Contact:	E-mail List <qa-bugs>
Severity:	Blocker
Priority:	P5 - None	CC:	ast, kontakt, lmuelle
Version:	Final
Target Milestone:	---
Hardware:	Other
OS:	All
Whiteboard:
Found By:	Other	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

Description Danny Al-Gaaf 2005-09-20 09:06:55 UTC

There is a bug in the hald-addon-acpi. The addon correct read the incomming 
event from the acpid socket, but never call HAL to change/rescan the related 
devices. 

This is a bug in the latest patch to reconnect if connection to acpid socket 
lost. 'if (errno) ...' does not work. Because of this the addon go into codepath 
to reconnect to the socket and not to analyse event and call HAL. We need also 
check if !event (now: if (errno && !event)...).

I also changed the timeout to try to reconnect from 1 second to 3.

Comment 3 Anja Stock 2005-10-06 10:10:50 UTC

released

Comment 4 Danny Al-Gaaf 2005-10-06 13:02:10 UTC

I can't see a updated version of HAL, I maked no patch, so I think this is not 
fixed.

Comment 5 Danny Al-Gaaf 2005-10-06 13:11:24 UTC

*** Bug 120154 has been marked as a duplicate of this bug. ***

Comment 7 Danny Al-Gaaf 2005-10-12 12:15:53 UTC

New Package submitted. Close the bug if the package is available via YOU.

Comment 8 Anja Stock 2005-10-12 12:59:22 UTC

ok, please use a new SWAMPID: 2583 - I will close the bug, when I released this
additional patch

Comment 9 Danny Al-Gaaf 2005-10-12 22:02:38 UTC

@Anja: I would include a patch for #121913 (add missing addon to HAL package) to 
this update/SWAMPID. I could submitt tomorrow a package, Is this o.k?

Comment 10 Anja Stock 2005-10-13 07:28:11 UTC

Yes, use the ID for both of them and adjust the patchinfo file please.
Thank you

Comment 11 Danny Al-Gaaf 2005-10-17 14:09:16 UTC

*** Bug 114612 has been marked as a duplicate of this bug. ***

Comment 12 Danny Al-Gaaf 2005-10-18 10:08:09 UTC

Anja: I saw the YOU update is available. I close the bug now.