After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 593936 - Get rid of HAL
Get rid of HAL
Status: RESOLVED FIXED
Product: gnome-pilot
Classification: Other
Component: gpilotd
2.32.x
Other All
: Normal normal
: ---
Assigned To: gnome-pilot Maintainers
gnome-pilot Maintainers
Depends on:
Blocks: 593938
 
 
Reported: 2009-09-02 16:20 UTC by Javier Jardón (IRC: jjardon)
Modified: 2011-01-03 15:12 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Javier Jardón (IRC: jjardon) 2009-09-02 16:20:24 UTC
Please support devicekit, because hal is no longer in development and is
deprecated in favor of devicekit. 
See  http://www.freedesktop.org/wiki/Software/hal

DeiceKit home page: http://www.freedesktop.org/wiki/Software/DeviceKit
Comment 1 Javier Jardón (IRC: jjardon) 2009-09-02 16:36:44 UTC
For the record:

$ git grep libhal
gpilotd/gpilotd.c:#include <libhal.h>
gpilotd/gpilotd.c:      if ((platform = libhal_device_get_property_string (hal_ctx, udi, "pda.platform", NULL))) {
gpilotd/gpilotd.c:              libhal_free_string (platform);
gpilotd/gpilotd.c:          libhal_free_string (platform);
gpilotd/gpilotd.c:      } else if ((bus = libhal_device_get_property_string (hal_ctx, udi, "info.bus", NULL))) {
gpilotd/gpilotd.c:              libhal_free_string (bus);
gpilotd/gpilotd.c:          libhal_free_string (bus);
gpilotd/gpilotd.c:      vendor_id = libhal_device_get_property_int (hal_ctx, udi,
gpilotd/gpilotd.c:      product_id = libhal_device_get_property_int (hal_ctx, udi,
gpilotd/gpilotd.c:                      libhal_ctx_set_dbus_connection (hal_ctx, dbus_connection);
gpilotd/gpilotd.c:              libhal_ctx_free (hal_ctx);
gpilotd/gpilotd.c:      if (!(ctx = libhal_ctx_new ())) {
gpilotd/gpilotd.c:      libhal_ctx_set_dbus_connection (ctx, dbus_connection);
gpilotd/gpilotd.c:      libhal_ctx_set_device_added (ctx, hal_device_added);
gpilotd/gpilotd.c:      libhal_ctx_set_device_removed (ctx, hal_device_removed);
gpilotd/gpilotd.c:      if (!libhal_device_property_watch_all (ctx, &error)) {
gpilotd/gpilotd.c:              libhal_ctx_free (ctx);
gpilotd/gpilotd.c:      if (!libhal_ctx_init (ctx, &error)) {
gpilotd/gpilotd.c:              g_warning ("libhal_ctx_init failed: %s", error.message);
gpilotd/gpilotd.c:              libhal_ctx_free (ctx);
gpilotd/gpilotd.c:      if (!(devices = libhal_get_all_devices (ctx, &nr, &error))) {
gpilotd/gpilotd.c:              libhal_ctx_shutdown (ctx, NULL);
gpilotd/gpilotd.c:              libhal_ctx_free (ctx);
gpilotd/gpilotd.c:      libhal_free_string_array (devices);
Comment 2 Felipe Besoaín Pino 2010-09-27 14:23:11 UTC
This bug was reported against a version which is not supported any more. Developers are no longer working on this version so there will not be any bug fixes for it.
Can you please check again if the issue you reported here still happens in a recent version of GNOME and update this report by adding a comment and adjusting the 'Version' field?

Again thank you for reporting this and sorry that it could not be fixed for the version you originally used here.

Without feedback this report will be closed as INCOMPLETE after 6 weeks.
Comment 3 Javier Jardón (IRC: jjardon) 2010-09-27 14:27:09 UTC
Hello Felipe,

yes, this is still an issue in current gnome-pilot git master
Comment 4 Felipe Besoaín Pino 2010-09-27 16:41:29 UTC
Hi, thanks for replying Javier, I was just checking some old bugs!
Comment 5 Matt Davey 2010-11-04 12:09:49 UTC
I'm finally getting around to looking at this.  The libhal code is being used for detecting usb devices.  After googling to learn a bit about DBUS/HAL/DeviceKit status, I'm now presuming this should be migrated to use udev (gudev) directly, via GUdevClient and GUdevDevice.  No dbus required.  Does that sound right?
Comment 6 Javier Jardón (IRC: jjardon) 2010-11-04 12:46:09 UTC
Hello Matt,

yes, that is what others project are doing. See the blockers of tracker bug #593938
Comment 7 Matt Davey 2011-01-03 15:12:09 UTC
This should be resolved on master.  libhal remains for backwards compatibility, but it is not required and is not used unless specifically requested.  gudev is used by preference.