Skip to contentSkip to conversation

systemd: remove usbutils dependancy

PATCH Superseded [email protected] 3 participants 3 messages Jan 24, 2013

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 systemd: remove usbutils dependancy

From: Jack Mitchell <[email protected]>

No where in the systemd readme does it specify that usbutils is a dependancy of systemd. I can only guess that it was added due to an issue elsewhere in the chain or it was brought in when udev was merged and the dependancies never checked.

I have build tested this with no issues.

---
 meta/recipes-core/systemd/systemd_196.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta/recipes-core/systemd/systemd_196.bb +1 -1
index a78a8a6..9eca25b 100644
@@ -8,7 +8,7 @@ LIC_FILES_CHKSUM = "file://LICENSE.GPL2;md5=751419260aa954499f7abaabaa882bbe \
 
 PROVIDES = "udev"
 
-DEPENDS = "xz kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup tcp-wrappers usbutils glib-2.0 libgcrypt"
+DEPENDS = "xz kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup tcp-wrappers glib-2.0 libgcrypt"
 DEPENDS += "${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
 
 SECTION = "base/shell"
Signature · 1 line
1.8.1.1
#1 lore raw
Quoted 6 lines · On 24 January 2013 10:44, <[email protected]> wrote:

No where in the systemd readme does it specify that usbutils is a dependancy of systemd. I can only guess that it was added due to an issue elsewhere in the chain or it was brought in when udev was merged and the dependancies never checked.

I have build tested this with no issues.

Build-tested from a clean sysroot?

Ross

#2 lore raw
In reply to [email protected]
Quoted 6 lines · On Thu, Jan 24, 2013 at 10:44:38AM +0000, [email protected] wrote:

From: Jack Mitchell <[email protected]>

No where in the systemd readme does it specify that usbutils is a dependancy of systemd. I can only guess that it was added due to an issue elsewhere in the chain or it was brought in when udev was merged and the dependancies never checked.

It was needed for older systemd revisions:
http://git.openembedded.org/meta-openembedded/commit/?id=ec2459251c68bb090c8b6f0d6cbf5ea0c2486337

http://cgit.freedesktop.org/systemd/systemd/tree/configure.ac?id=cd96b3b86abb4a88cac2722bdfb6e5d4413f6831

And removed between 195 and 196:
http://cgit.freedesktop.org/systemd/systemd/commit/configure.ac?id=796b06c21b62d13c9021e2fbd9c58a5c6edb2764

Please update commit message.

Quoted 28 lines

I have build tested this with no issues.

---
 meta/recipes-core/systemd/systemd_196.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta/recipes-core/systemd/systemd_196.bb +1 -1
index a78a8a6..9eca25b 100644
@@ -8,7 +8,7 @@ LIC_FILES_CHKSUM = "file://LICENSE.GPL2;md5=751419260aa954499f7abaabaa882bbe \
 
 PROVIDES = "udev"
 
-DEPENDS = "xz kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup tcp-wrappers usbutils glib-2.0 libgcrypt"
+DEPENDS = "xz kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup tcp-wrappers glib-2.0 libgcrypt"
 DEPENDS += "${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
 
 SECTION = "base/shell"
Signature · 7 lines
1.8.1.1


_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]