[patch] Port to DBUS 0.30

John (J5) Palmieri johnp at redhat.com
Thu Feb 10 16:41:42 PST 2005


Ok, ran into a huge problem with the way HAL handles devices.  It will
take the UDI and use that as a object path in order to be able to query
those device.  Big problem is that object paths can not have any
characters besides [a-z][A-Z][0-9]_.  Some of the udi's have - and : in
them.  Not sure how we should proceed.

On Wed, 2005-02-09 at 19:08 -0500, John (J5) Palmieri wrote:
> Here is the port to dbus-0.30.  It passes all the test cases but
> requires a patch to dbus.  Attached is the dbus patch for testing only.
> I think the fix is a bit deeper so I want to discuss it on the dbus list
> before committing.  I didn't include the ChangeLog in the patch so here
> it is:
> 
> 
> 2005-02-09  John (J5) Palmieri  <johnp at redhat.com>
> 
> 	* hald/device.c, hald/device_info.c, hald/property.c, hald/property.h,
> 	tools/hal_set_property.c, tools/hal_get_property.c: 
> 	s/HAL_PROPERTY_TYPE_NIL/HAL_PROPERTY_TYPE_INVALID
> 
> 	* hald/hald_dbus.c: ported to new dbus-0.30 API
> 	(foreach_property_append): implemented real string lists 
> 	and got rid of the \tval\tval\tval\t hack
> 
> 	* libhal/libhal.c: ported to new dbus-0.30 API
> 	s/HAL_PROPERTY_TYPE_NIL/HAL_PROPERTY_TYPE_INVALID
> 	(libhal_get_string_array_from_iter): new helper function to create
> 	string arrays from dbus arrays
> 	(libhal_property_fill_value_from_variant): new helper function
> 	that fills in properties from variants.  Used when getting
> 	a hash of properties from hald
> 	
> 	* libhal/libhal.h: s/HAL_PROPERTY_TYPE_NIL/HAL_PROPERTY_TYPE_INVALID
> 	HAL_PROPERTY_TYPE_INVALID = DBUS_TYPE_INVALID
> 
> 	* hald/hald_test_libhal.c: ported to new dbus-0.30 API
> 	Added success messages and more detailed failed messages
> 
> 
> _______________________________________________
> hal mailing list
> hal at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/hal
-- 
John (J5) Palmieri
Associate Software Engineer
Desktop Group
Red Hat, Inc.
Blog: http://martianrock.com

_______________________________________________
hal mailing list
hal at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/hal



More information about the Hal mailing list