[oe] [RFC] Fix udev pulling in libvolume-id-dev
Denys Dmytriyenko
denis at denix.org
Thu Jan 15 06:14:18 UTC 2009
Hi,
I was debugging the strange behavior of one of my boxes pulling in libc6-dev
into the filesystem image and it happened to be because udev pulled in
libvolume-id-dev instead of libvolume-id0... It worked as expected on the
other box, though.
Several people on IRC have seen this lingering bug before, but no fix was
available. Koen mentioned it got fixed for him by rebuilding udev...
After some investigation I found out that both libvolume-id packages (-dev and
regular) are registered to provide libvolume_id.so.0 and the one appearing
last in os.walk() gets pulled in by udev. The order of files provided by
os.walk() depends on parallel builds and phase of the Moon.
libvolume-id-dev does not provide the actual .so library, but it contains a
symlink in /usr/lib, which points to /lib/libvolume.so.0 (the host one!)
instead of ../../lib/ one...
The first patch attached fixes this in udev and should be a no brainer. Unless
someone objects, I'll push it in in a day or two.
Since I was debugging the code which actually does this shlib magic in
package.bbclass, I thought it might be useful to prevent this kind of issues
in the future by skipping symlinks while looking for SONAME/NEEDED in the
objdump - please see the second patch attached and provide feedback.
--
Denys
-------------- next part --------------
diff --git a/packages/udev/udev-124/libvolume-id-soname.patch b/packages/udev/udev-124/libvolume-id-soname.patch
new file mode 100644
index 0000000..56365a4
--- /dev/null
+++ b/packages/udev/udev-124/libvolume-id-soname.patch
@@ -0,0 +1,12 @@
+diff -uNr udev-124-orig/extras/volume_id/lib/Makefile udev-124/extras/volume_id/lib/Makefile
+--- udev-124-orig/extras/volume_id/lib/Makefile 2008-06-12 01:24:30.000000000 -0400
++++ udev-124/extras/volume_id/lib/Makefile 2009-01-14 23:32:42.000000000 -0500
+@@ -113,7 +113,7 @@
+ ifeq ($(libdir),$(usrlibdir))
+ ln -sf $(SHLIB) $(DESTDIR)$(usrlibdir)/libvolume_id.so
+ else
+- ln -sf $(libdir)/$(SHLIB) $(DESTDIR)$(usrlibdir)/libvolume_id.so
++ ln -sf ../..$(libdir)/$(SHLIB) $(DESTDIR)$(usrlibdir)/libvolume_id.so
+ endif
+ $(INSTALL) -d $(DESTDIR)$(usrlibdir)/pkgconfig
+ $(INSTALL_DATA) libvolume_id.pc $(DESTDIR)$(usrlibdir)/pkgconfig/libvolume_id.pc
diff --git a/packages/udev/udev_124.bb b/packages/udev/udev_124.bb
index 515debe..aa156a1 100644
--- a/packages/udev/udev_124.bb
+++ b/packages/udev/udev_124.bb
@@ -3,13 +3,14 @@ DESCRIPTION = "udev is a daemon which dynamically creates and removes device nod
the hotplug package and requires a kernel not older than 2.6.12."
RPROVIDES_${PN} = "hotplug"
-PR = "r11"
+PR = "r12"
SRC_URI = "http://kernel.org/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
file://noasmlinkage.patch;patch=1 \
file://flags.patch;patch=1 \
file://vol_id_ld.patch;patch=1 \
file://udevtrigger_add_devname_filtering.patch;patch=1 \
+ file://libvolume-id-soname.patch;patch=1 \
file://mount.blacklist \
file://run.rules \
"
-------------- next part --------------
diff --git a/classes/package.bbclass b/classes/package.bbclass
index 8b7d649..a7cdb5f 100644
--- a/classes/package.bbclass
+++ b/classes/package.bbclass
@@ -615,7 +615,7 @@ python package_do_shlibs() {
for file in files:
soname = None
path = os.path.join(root, file)
- if os.access(path, os.X_OK) or lib_re.match(file):
+ if (os.access(path, os.X_OK) or lib_re.match(file)) and not os.path.islink(path):
cmd = bb.data.getVar('OBJDUMP', d, 1) + " -p " + path + " 2>/dev/null"
cmd = "PATH=\"%s\" %s" % (bb.data.getVar('PATH', d, 1), cmd)
fd = os.popen(cmd)
More information about the Openembedded-devel
mailing list