From 309c2a2ce95aae54879b4957d113f03608530c15 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Wed, 13 Jul 2011 20:11:23 +0200 Subject: [PATCH] loginctl: use new udev_device_has_tag() call from libudev where applicable --- configure.ac | 2 +- src/logind-dbus.c | 19 +------------------ src/logind.h | 2 -- src/sysfs-show.c | 21 ++------------------- 4 files changed, 4 insertions(+), 40 deletions(-) diff --git a/configure.ac b/configure.ac index 46d8eebb..787a0a82 100644 --- a/configure.ac +++ b/configure.ac @@ -117,7 +117,7 @@ AC_CHECK_HEADERS([sys/capability.h], [], [AC_MSG_ERROR([*** POSIX caps headers n # This makes sure pkg.m4 is available. m4_pattern_forbid([^_?PKG_[A-Z_]+$],[*** pkg.m4 missing, please install pkg-config]) -PKG_CHECK_MODULES(UDEV, [ libudev >= 160 ]) +PKG_CHECK_MODULES(UDEV, [ libudev >= 172 ]) AC_SUBST(UDEV_CFLAGS) AC_SUBST(UDEV_LIBS) diff --git a/src/logind-dbus.c b/src/logind-dbus.c index 16dbd366..e0855f4c 100644 --- a/src/logind-dbus.c +++ b/src/logind-dbus.c @@ -550,23 +550,6 @@ fail: return r; } -static bool device_has_tag(struct udev_device *d, const char *tag) { - struct udev_list_entry *first, *item; - - assert(d); - assert(tag); - - /* FIXME */ - udev_device_get_is_initialized(d); - - first = udev_device_get_tags_list_entry(d); - udev_list_entry_foreach(item, first) - if (streq(udev_list_entry_get_name(item), tag)) - return true; - - return false; -} - static int trigger_device(Manager *m, const char *prefix) { struct udev_enumerate *e; struct udev_list_entry *first, *item; @@ -628,7 +611,7 @@ static int attach_device(Manager *m, const char *seat, const char *sysfs) { if (!d) return -ENODEV; - if (!device_has_tag(d, "seat")) { + if (!udev_device_has_tag(d, "seat")) { r = -ENODEV; goto finish; } diff --git a/src/logind.h b/src/logind.h index db20bff1..cdb61280 100644 --- a/src/logind.h +++ b/src/logind.h @@ -38,8 +38,6 @@ * direct client API * * udev: - * drop redundant udev_device_get_is_initialized() use as soon as libudev is fixed - * use device_has_tag() as soon as it is available * enumerate recursively with libudev when triggering * use sysfs in device hash table, not sysname, when fb driver is fixed * diff --git a/src/sysfs-show.c b/src/sysfs-show.c index b456bf12..ab866a47 100644 --- a/src/sysfs-show.c +++ b/src/sysfs-show.c @@ -26,23 +26,6 @@ #include "util.h" #include "sysfs-show.h" -static bool device_has_tag(struct udev_device *d, const char *tag) { - struct udev_list_entry *first, *item; - - assert(d); - assert(tag); - - /* FIXME */ - udev_device_get_is_initialized(d); - - first = udev_device_get_tags_list_entry(d); - udev_list_entry_foreach(item, first) - if (streq(udev_list_entry_get_name(item), tag)) - return true; - - return false; -} - static int show_sysfs_one( struct udev *udev, const char *seat, @@ -77,7 +60,7 @@ static int show_sysfs_one( sn = "seat0"; /* fixme, also check for tag 'seat' here */ - if (!streq(seat, sn) || !device_has_tag(d, "seat")) { + if (!streq(seat, sn) || !udev_device_has_tag(d, "seat")) { udev_device_unref(d); *item = udev_list_entry_get_next(*item); continue; @@ -109,7 +92,7 @@ static int show_sysfs_one( if (isempty(lookahead_sn)) lookahead_sn = "seat0"; - found = streq(seat, lookahead_sn) && device_has_tag(lookahead_d, "seat"); + found = streq(seat, lookahead_sn) && udev_device_has_tag(lookahead_d, "seat"); udev_device_unref(lookahead_d); if (found) -- 2.39.5