* [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils
@ 2010-05-25 6:49 raj.khem
2010-05-25 7:22 ` Koen Kooi
0 siblings, 1 reply; 7+ messages in thread
From: raj.khem @ 2010-05-25 6:49 UTC (permalink / raw)
To: openembedded-devel
From: Khem Raj <raj.khem@gmail.com>
* Main purpose of the patch was to fix the problem where udev-utils was
needed to be in root file system just to get udevadm binary. So this
binary is moved into udev package instead.
* Use INC_PR
* Use udev.inc in all recipes.
* All udev recipes built udev_151 and udev_124 runs fine too.
Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
recipes/udev/udev-static_124.bb | 20 ++++++++-----
recipes/udev/udev.inc | 28 +++++++++---------
recipes/udev/udev_092.bb | 23 +++++++--------
recipes/udev/udev_097.bb | 15 +++++-----
recipes/udev/udev_100.bb | 21 +++++++-------
recipes/udev/udev_118.bb | 13 ++++-----
recipes/udev/udev_124.bb | 34 +++++++++++-----------
recipes/udev/udev_141.bb | 58 +++++++-------------------------------
recipes/udev/udev_151.bb | 43 ++++++++---------------------
9 files changed, 100 insertions(+), 155 deletions(-)
diff --git a/recipes/udev/udev-static_124.bb b/recipes/udev/udev-static_124.bb
index 1ea088a..3f07f5a 100644
--- a/recipes/udev/udev-static_124.bb
+++ b/recipes/udev/udev-static_124.bb
@@ -1,15 +1,19 @@
DESCRIPTION = "Static version of udev for devices with an old (e.g. <2.6.27) kernel"
-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://mtd-exclude-persistent.patch;patch=1 \
+require udev.inc
+
+SRC_URI += "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://mtd-exclude-persistent.patch;patch=1 \
"
-require udev.inc
+
+PR = "${INC_PR}.0"
+
+LD = "${CC}"
S = "${WORKDIR}/udev-${PV}"
diff --git a/recipes/udev/udev.inc b/recipes/udev/udev.inc
index 14d10b4..943fdfc 100644
--- a/recipes/udev/udev.inc
+++ b/recipes/udev/udev.inc
@@ -1,8 +1,9 @@
DESCRIPTION ?= "udev is a program which dynamically creates and removes device nodes from \
/dev/. It responds to /sbin/hotplug device events and requires a 2.6 kernel."
+
LICENSE = "GPLv2+"
-SRC_URI += " \
+SRC_URI = "http://kernel.org/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
file://udev.rules \
file://devfs-udev.rules \
file://links.conf \
@@ -12,6 +13,8 @@ SRC_URI += " \
file://local.rules \
file://init"
+INC_PR = "r26"
+
inherit update-rc.d autotools pkgconfig
PARALLEL_MAKE = ""
@@ -19,18 +22,19 @@ UDEV_DEVFS_RULES ?= "0"
PKG_libvolume-id-dev = "libvolume-id-dev"
INITSCRIPT_NAME = "udev"
-INITSCRIPT_PARAMS = "start 04 S ."
+INITSCRIPT_PARAMS = "start 03 S ."
export CROSS = "${TARGET_PREFIX}"
export HOSTCC = "${BUILD_CC}"
export udevdir ?= "/dev"
+# Put stuff in /lib and /sbin
+export bindir := "${base_bindir}"
+export sbindir := "${base_sbindir}"
export usrbindir := "${bindir}"
export usrsbindir := "${sbindir}"
-export etcdir = "${sysconfdir}"
-LD = "${CC}"
-bindir = "/bin"
-sbindir = "/sbin"
+export etcdir := "${sysconfdir}"
+TARGET_CC_ARCH += "${LDFLAGS}"
UDEV_EXTRAS = "<override me>"
EXTRA_OEMAKE = "-e \
'EXTRAS=${UDEV_EXTRAS}' \
@@ -56,17 +60,13 @@ do_install () {
install -m 0755 ${WORKDIR}/mount.sh ${D}${sysconfdir}/udev/scripts/mount.sh
install -m 0755 ${WORKDIR}/network.sh ${D}${sysconfdir}/udev/scripts
-}
-
-do_stage () {
- autotools_stage_all
- install -m 0644 ${S}/extras/volume_id/lib/libvolume_id.h ${STAGING_INCDIR}
- oe_libinstall -C extras/volume_id/lib -so libvolume_id ${STAGING_LIBDIR}
+ install -m 0644 ${S}/extras/volume_id/lib/libvolume_id.h ${D}${includedir}
+ oe_libinstall -C extras/volume_id/lib -so libvolume_id ${D}${libdir}
}
PACKAGES =+ "udev-utils libvolume-id libvolume-id-dev"
FILES_libvolume-id-dev = "${includedir}/libvolume_id.h ${libdir}/libvolume_id.a ${libdir}/libvolume_id.so ${libdir}/pkgconfig/libvolume_id.pc"
-FILES_udev-utils = "${usrbindir}/udevinfo ${sbindir}/udevadm ${usrbindir}/udevtest"
+FILES_udev-utils = "${usrbindir}/udevinfo ${usrbindir}/udevtest"
FILES_libvolume-id = "${base_libdir}/libvolume_id.so.*"
-FILES_${PN} += "${usrbindir}/* ${usrsbindir}/udevd"
+FILES_${PN} += "${usrbindir}/* ${usrsbindir}/udevd ${sbindir}/udevadm"
FILES_${PN}-dbg += "${usrbindir}/.debug ${usrsbindir}/.debug"
diff --git a/recipes/udev/udev_092.bb b/recipes/udev/udev_092.bb
index 6e4d025..3ed05d1 100644
--- a/recipes/udev/udev_092.bb
+++ b/recipes/udev/udev_092.bb
@@ -3,21 +3,20 @@ 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 = "r25"
-
-SRC_URI = "${KERNELORG_MIRROR}/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
- file://noasmlinkage.patch;patch=1 \
- file://flags.patch;patch=1 \
- file://udevsynthesize.patch;patch=1 \
- file://udevsynthesize.sh \
- file://arm_inotify_fix.patch;patch=1 \
- file://mtd-exclude-persistent.patch;patch=1 \
- file://mount.blacklist \
- "
-
require udev.inc
-INITSCRIPT_PARAMS = "start 03 S ."
+LD = "${CC}"
+
+PR = "${INC_PR}.0"
+
+SRC_URI += "file://noasmlinkage.patch;patch=1 \
+ file://flags.patch;patch=1 \
+ file://udevsynthesize.patch;patch=1 \
+ file://udevsynthesize.sh \
+ file://arm_inotify_fix.patch;patch=1 \
+ file://mtd-exclude-persistent.patch;patch=1 \
+ file://mount.blacklist \
+ "
FILES_${PN} += "${base_libdir}/udev/*"
FILES_${PN}-dbg += "${base_libdir}/udev/.debug"
diff --git a/recipes/udev/udev_097.bb b/recipes/udev/udev_097.bb
index 4e49e4a..8757794 100644
--- a/recipes/udev/udev_097.bb
+++ b/recipes/udev/udev_097.bb
@@ -8,17 +8,16 @@ used to detect the type of a file system and read its metadata."
DESCRIPTION_libvolume-id-dev = "libvolume_id development headers, \
needed to link programs with libvolume_id."
-PR = "r13"
+require udev.inc
-SRC_URI = "${KERNELORG_MIRROR}/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
- file://noasmlinkage.patch;patch=1 \
- file://flags.patch;patch=1 \
- file://mount.blacklist \
- "
+PR = "${INC_PR}.0"
-require udev.inc
+LD = "${CC}"
-INITSCRIPT_PARAMS = "start 03 S ."
+SRC_URI += "file://noasmlinkage.patch;patch=1 \
+ file://flags.patch;patch=1 \
+ file://mount.blacklist \
+ "
FILES_${PN} += "${base_libdir}/udev/*"
FILES_${PN}-dbg += "${base_libdir}/udev/.debug"
diff --git a/recipes/udev/udev_100.bb b/recipes/udev/udev_100.bb
index b74ae69..7261483 100644
--- a/recipes/udev/udev_100.bb
+++ b/recipes/udev/udev_100.bb
@@ -9,19 +9,18 @@ used to detect the type of a file system and read its metadata."
DESCRIPTION_libvolume-id-dev = "libvolume_id development headers, \
needed to link programs with libvolume_id."
-PR = "r16"
-
-SRC_URI = "${KERNELORG_MIRROR}/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
- file://noasmlinkage.patch;patch=1 \
- file://flags.patch;patch=1 \
- file://mtd-exclude-persistent.patch;patch=1 \
- file://mount.blacklist \
- file://mount.sh \
- "
-
require udev.inc
-INITSCRIPT_PARAMS = "start 03 S ."
+PR = "${INC_PR}.0"
+
+LD = "${CC}"
+
+SRC_URI += "file://noasmlinkage.patch;patch=1 \
+ file://flags.patch;patch=1 \
+ file://mtd-exclude-persistent.patch;patch=1 \
+ file://mount.blacklist \
+ file://mount.sh \
+ "
FILES_${PN} += "${base_libdir}/udev/*"
FILES_${PN}-dbg += "${base_libdir}/udev/.debug"
diff --git a/recipes/udev/udev_118.bb b/recipes/udev/udev_118.bb
index d4f6fe2..d2abd8f 100644
--- a/recipes/udev/udev_118.bb
+++ b/recipes/udev/udev_118.bb
@@ -3,12 +3,15 @@ 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 = "r6"
+require udev.inc
+
+PR = "${INC_PR}.0"
+
+LD = "${CC}"
DEFAULT_PREFERENCE = "-118"
-SRC_URI = "\
- http://kernel.org/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
+SRC_URI += "\
file://flags.patch;patch=1 \
file://vol_id_ld.patch;patch=1 \
file://udevtrigger_add_devname_filtering.patch;patch=1 \
@@ -17,10 +20,6 @@ SRC_URI = "\
"
TARGET_CC_ARCH += "${LDFLAGS}"
-require udev.inc
-
-INITSCRIPT_PARAMS = "start 03 S ."
-
FILES_${PN} += "${base_libdir}/udev/*"
FILES_${PN}-dbg += "${base_libdir}/udev/.debug"
UDEV_EXTRAS = "extras/firmware/ extras/scsi_id/ extras/volume_id/"
diff --git a/recipes/udev/udev_124.bb b/recipes/udev/udev_124.bb
index ed40594..bdbe5e0 100644
--- a/recipes/udev/udev_124.bb
+++ b/recipes/udev/udev_124.bb
@@ -2,28 +2,28 @@ DESCRIPTION = "udev is a daemon which dynamically creates and removes device nod
/dev/, handles hotplug events and loads drivers at boot time. It replaces \
the hotplug package and requires a kernel not older than 2.6.12."
RPROVIDES_${PN} = "hotplug"
-PR = "r19"
-
-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://mtd-exclude-persistent.patch;patch=1 \
- file://mount.blacklist \
- file://run.rules \
- file://default \
- file://local.rules \
+
+require udev.inc
+
+PR = "${INC_PR}.0"
+
+LD = "${CC}"
+
+SRC_URI += "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://mtd-exclude-persistent.patch;patch=1 \
+ file://mount.blacklist \
+ file://run.rules \
+ file://default \
+ file://local.rules \
"
SRC_URI_append_h2200 = " file://50-hostap_cs.rules "
PACKAGE_ARCH_h2200 = "h2200"
-require udev.inc
-
-INITSCRIPT_PARAMS = "start 03 S ."
-
FILES_${PN} += "${base_libdir}/udev/*"
FILES_${PN}-dbg += "${base_libdir}/udev/.debug"
UDEV_EXTRAS = "extras/firmware/ extras/scsi_id/ extras/volume_id/"
diff --git a/recipes/udev/udev_141.bb b/recipes/udev/udev_141.bb
index 4be7ae5..27055be 100644
--- a/recipes/udev/udev_141.bb
+++ b/recipes/udev/udev_141.bb
@@ -1,33 +1,19 @@
DESCRIPTION = "udev is a daemon which dynamically creates and removes device nodes from \
/dev/, handles hotplug events and loads drivers at boot time. It replaces \
the hotplug package and requires a kernel not older than 2.6.12."
-LICENSE = "GPLv2+"
# Untested
DEFAULT_PREFERENCE = "-1"
-PR = "r21"
+require udev.inc
-# needed for init.d script
-RDEPENDS_${PN} += "udev-utils"
+PR = "${INC_PR}.0"
-SRC_URI = "http://kernel.org/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
- file://mount.blacklist \
- file://run.rules \
+SRC_URI += "file://mount.blacklist \
+ file://run.rules \
+ file://default \
+ file://cache \
"
-SRC_URI += " \
- file://udev.rules \
- file://devfs-udev.rules \
- file://links.conf \
- file://permissions.rules \
- file://mount.sh \
- file://network.sh \
- file://local.rules \
- file://default \
- file://init \
- file://cache \
-"
-
SRC_URI_append_h2200 = " file://50-hostap_cs.rules "
PACKAGE_ARCH_h2200 = "h2200"
@@ -39,26 +25,16 @@ SRC_URI_append_bug = " \
PACKAGE_ARCH_bug = "bug"
-inherit update-rc.d autotools_stage
-
-# Put stuff in /lib and /sbin
-export sbindir="${base_sbindir}"
-export exec_prefix=""
EXTRA_OECONF += " --with-udev-prefix= \
--with-libdir-name=${base_libdir} \
"
-INITSCRIPT_NAME = "udev"
-INITSCRIPT_PARAMS = "start 03 S ."
+UDEV_EXTRAS = "extras/firmware/ extras/scsi_id/ extras/volume_id/"
-PACKAGES =+ "udev-utils libvolume-id libvolume-id-dev"
-
-FILES_libvolume-id-dev = "${includedir}/libvolume_id.h ${libdir}/libvolume_id.a ${libdir}/libvolume_id.so ${libdir}/pkgconfig/libvolume_id.pc"
-FILES_udev-utils = "${usrbindir}/udevinfo ${sbindir}/udevadm ${usrbindir}/udevtest"
-FILES_libvolume-id = "${base_libdir}/libvolume_id.so.*"
+LEAD_SONAME = "libudev.so.0"
RPROVIDES_${PN} = "hotplug"
-FILES_${PN} += "${usrbindir}/* ${usrsbindir}/udevd"
+FILES_${PN} += "${usrbindir}/* ${usrsbindir}/udevd ${sbindir}/udevadm"
FILES_${PN}-dbg += "${usrbindir}/.debug ${usrsbindir}/.debug"
# udev installs binaries under $(udev_prefix)/lib/udev, even if ${libdir}
@@ -103,6 +79,8 @@ do_install () {
install -m 0755 ${WORKDIR}/mount.sh ${D}${sysconfdir}/udev/scripts/mount.sh
install -m 0755 ${WORKDIR}/network.sh ${D}${sysconfdir}/udev/scripts
+ oe_libinstall -C udev/lib -so libudev ${D}${libdir}
+ install ${S}/udev/lib/libudev.h ${D}${includedir}
}
do_install_append_h2200() {
@@ -132,19 +110,5 @@ if [ -d $D/lib/udev/rules.d ] ; then
fi
}
-do_stage_append() {
- install -m 0644 ${S}/extras/volume_id/lib/libvolume_id.h ${STAGING_INCDIR}
- oe_libinstall -C extras/volume_id/lib -so libvolume_id ${STAGING_LIBDIR}
- oe_libinstall -C udev/lib -so libudev ${STAGING_LIBDIR}
- # Since we change exec_prefix above, autotools_stage_all will not see the .pc files
- # When we upgrade to 145 with the path bugs fixed we can drop all this (see poky)
- install -d ${STAGING_DIR_TARGET}${prefix}${libdir}/pkgconfig/
- install ${S}/extras/volume_id/lib/libvolume_id.pc ${STAGING_DIR_TARGET}${prefix}${libdir}/pkgconfig/
- install ${S}/udev/lib/libudev.pc ${STAGING_DIR_TARGET}${prefix}${libdir}/pkgconfig/
- install -d ${STAGING_DIR_TARGET}${prefix}${includedir}/
- install ${S}/extras/volume_id/lib/libvolume_id.h ${STAGING_DIR_TARGET}${prefix}${includedir}/
- install ${S}/udev/lib/libudev.h ${STAGING_DIR_TARGET}${prefix}${includedir}/
-}
-
SRC_URI[md5sum] = "86382b7bbc64459e714c65a2a4e10916"
SRC_URI[sha256sum] = "001be4a8963c48e7debc82c6078562d087a2979da63e558a5e3613c03725c377"
diff --git a/recipes/udev/udev_151.bb b/recipes/udev/udev_151.bb
index d7df81a..35b43b0 100644
--- a/recipes/udev/udev_151.bb
+++ b/recipes/udev/udev_151.bb
@@ -1,9 +1,10 @@
DESCRIPTION = "udev is a daemon which dynamically creates and removes device nodes from \
/dev/, handles hotplug events and loads drivers at boot time. It replaces \
the hotplug package and requires a kernel not older than 2.6.12."
-LICENSE = "GPLv2+"
-PR = "r9"
+require udev.inc
+
+PR = "${INC_PR}.0"
# Untested
DEFAULT_PREFERENCE = "-1"
@@ -11,25 +12,11 @@ DEFAULT_PREFERENCE = "-1"
# Needed for udev-extras
DEPENDS = "gperf-native usbutils acl glib-2.0"
-# needed for init.d script
-RDEPENDS_${PN} += "udev-utils"
-
-SRC_URI = "http://kernel.org/pub/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
- file://mount.blacklist \
- file://run.rules \
+SRC_URI += "file://mount.blacklist \
+ file://run.rules \
+ file://default \
+ file://cache \
"
-SRC_URI += " \
- file://udev.rules \
- file://devfs-udev.rules \
- file://links.conf \
- file://permissions.rules \
- file://mount.sh \
- file://network.sh \
- file://local.rules \
- file://default \
- file://init \
- file://cache \
-"
SRC_URI_append_h2200 = " file://50-hostap_cs.rules "
PACKAGE_ARCH_h2200 = "h2200"
@@ -42,8 +29,6 @@ SRC_URI_append_bug = " \
PACKAGE_ARCH_bug = "bug"
-inherit update-rc.d autotools
-
EXTRA_OECONF += " --with-udev-prefix= \
--with-libdir-name=${base_libdir} \
--disable-introspection \
@@ -53,27 +38,23 @@ EXTRA_OECONF += " --with-udev-prefix= \
--sbindir=${base_sbindir} \
--libexecdir=${base_libdir}/udev \
--with-rootlibdir=${base_libdir} \
-"
+ "
-INITSCRIPT_NAME = "udev"
-INITSCRIPT_PARAMS = "start 03 S ."
-
-PACKAGES =+ "libudev libgudev udev-utils"
+LEAD_SONAME = "libudev.so.0"
+PACKAGES =+ "libudev libgudev"
FILES_libudev = "${libdir}/libudev.so.*"
FILES_libgudev = "${libdir}/libgudev*.so.*"
-FILES_udev-utils = "${bindir}/udevinfo ${base_sbindir}/udevadm ${bindir}/udevtest"
-
RPROVIDES_${PN} = "hotplug"
-FILES_${PN} += "${usrbindir}/* ${usrsbindir}/udevd"
-FILES_${PN}-dbg += "${usrbindir}/.debug ${usrsbindir}/.debug"
# udev installs binaries under $(udev_prefix)/lib/udev, even if ${libdir}
# is ${prefix}/lib64
FILES_${PN} += "/lib/udev* ${libdir}/ConsoleKit"
FILES_${PN}-dbg += "/lib/udev/.debug"
+UDEV_EXTRAS = "extras/firmware/ extras/scsi_id/ extras/volume_id/"
+
do_install () {
install -d ${D}${usrsbindir} \
${D}${sbindir}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils
2010-05-25 6:49 [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils raj.khem
@ 2010-05-25 7:22 ` Koen Kooi
2010-05-25 9:11 ` Grzegorz Toporek
2010-05-25 14:50 ` Khem Raj
0 siblings, 2 replies; 7+ messages in thread
From: Koen Kooi @ 2010-05-25 7:22 UTC (permalink / raw)
To: openembedded-devel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 25-05-10 08:49, raj.khem@gmail.com wrote:
> From: Khem Raj <raj.khem@gmail.com>
>
> * Main purpose of the patch was to fix the problem where udev-utils was
> needed to be in root file system just to get udevadm binary. So this
> binary is moved into udev package instead.
>
> * Use INC_PR
>
> * Use udev.inc in all recipes.
>
> * All udev recipes built udev_151 and udev_124 runs fine too.
I explicitly didn't use udev.inc in udev 151 because it made the recipe
very hard to read. It would make more sense to do a udev-1xx.inc based
on udev 151 and use that for future recipes.
The udevadm packaging change looks good, but it's only needed for udev
>124 if I read the udev changelogs correctly.
regards,
Koen
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
iD8DBQFL+3rPMkyGM64RGpERAreVAJ9ONwqKo/pWx9zBQ+v3YAQcFrzTZACglliD
Z9mfuLp0SJ+/AuburyQIeaQ=
=h54a
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils
2010-05-25 7:22 ` Koen Kooi
@ 2010-05-25 9:11 ` Grzegorz Toporek
2010-05-25 9:25 ` Koen Kooi
2010-05-25 9:55 ` Henri Bragge
2010-05-25 14:50 ` Khem Raj
1 sibling, 2 replies; 7+ messages in thread
From: Grzegorz Toporek @ 2010-05-25 9:11 UTC (permalink / raw)
To: openembedded-devel
>
> > * Main purpose of the patch was to fix the problem where udev-utils was
> > needed to be in root file system just to get udevadm binary. So this
> > binary is moved into udev package instead.
> >
> > * Use INC_PR
> >
> > * Use udev.inc in all recipes.
> >
> > * All udev recipes built udev_151 and udev_124 runs fine too.
>
> I explicitly didn't use udev.inc in udev 151 because it made the recipe
> very hard to read. It would make more sense to do a udev-1xx.inc based
> on udev 151 and use that for future recipes.
>
> The udevadm packaging change looks good, but it's only needed for udev
> >124 if I read the udev changelogs correctly.
>
> regards,
>
> Koen
>
Could you upload this patch to the git respiratory? Or this is just
non-official change?
I'm trying now to compile base-image on angstrom distro and I got error from
udev_151.bb recipe. May be your patch will help with this...
This is what I got:
Does anyone met with this error? Sorry if it's not a right place to post.
Kind Regards,
Grzegorz
#########################################
| checking for GLIB... configure: error: Package requirements (glib-2.0 >=
2.7.0 gobject-2.0 >= 2.7.0) were not met:
|
| Package file
'/home/grzesiek/tele/tmp/sysroots/armv5te-angstrom-linux-gnueabi/usr/lib/pkgconfig/glib-2.0.pc'
appears to be empty
| Package 'glib-2.0' has no Name: field
|
| Consider adjusting the PKG_CONFIG_PATH environment variable if you
| installed software in a non-standard prefix.
|
| Alternatively, you may set the environment variables GLIB_CFLAGS
| and GLIB_LIBS to avoid the need to call pkg-config.
| See the pkg-config man page for more details.
|
NOTE: Task failed:
/home/grzesiek/tele/tmp/work/armv5te-angstrom-linux-gnueabi/udev-151-r9/temp/log.do_configure.18550
ERROR: TaskFailed event exception, aborting
ERROR: Build of
/home/grzesiek/tele/openembedded/recipes/udev/udev_151.bbdo_configure
failed
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils
2010-05-25 9:11 ` Grzegorz Toporek
@ 2010-05-25 9:25 ` Koen Kooi
2010-05-25 10:04 ` Grzegorz Toporek
2010-05-25 9:55 ` Henri Bragge
1 sibling, 1 reply; 7+ messages in thread
From: Koen Kooi @ 2010-05-25 9:25 UTC (permalink / raw)
To: openembedded-devel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 25-05-10 11:11, Grzegorz Toporek wrote:
> '/home/grzesiek/tele/tmp/sysroots/armv5te-angstrom-linux-gnueabi/usr/lib/pkgconfig/glib-2.0.pc'
> appears to be empty
That seems to be the problem, does rebuilding glib-2.0 help?
regards,
Koen
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
iD8DBQFL+5d/MkyGM64RGpERAmBSAJ9bJhUwVftZY/vN8jPmyCVE3TSYUQCfeHcl
rCpt16O0fh4CGOv2b50lja8=
=A8Vo
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils
2010-05-25 9:11 ` Grzegorz Toporek
2010-05-25 9:25 ` Koen Kooi
@ 2010-05-25 9:55 ` Henri Bragge
1 sibling, 0 replies; 7+ messages in thread
From: Henri Bragge @ 2010-05-25 9:55 UTC (permalink / raw)
To: openembedded-devel
Grzegorz Toporek wrote:
> Could you upload this patch to the git respiratory? Or this is just
> non-official change?
>
> I'm trying now to compile base-image on angstrom distro and I got error from
> udev_151.bb recipe. May be your patch will help with this...
>
> This is what I got:
>
> Does anyone met with this error? Sorry if it's not a right place to post.
>
>
I've another issue concerning acl and attr that udev-151 requires, and
I'm hoping that this patch would help me too. Btw I didn't have this
problem with bitbake 1.8.18, but 1.10 with newest oe fails with output:
| make[4]: Entering directory
`.../tmp/work/armv5te-angstrom-linux-gnueabi/attr-2.4.44-r3/attr-2.4.44/attr'
| ../include/install-sh -o hbragge -g 1002 -m 755 -d /usr/bin
| chmod: changing permissions of `//usr/bin': Operation not permitted
| make[4]: *** [install] Error 1
| make[4]: Leaving directory
`.../tmp/work/armv5te-angstrom-linux-gnueabi/attr-2.4.44-r3/attr-2.4.44/attr'
| make[3]: *** [attr-install] Error 2
| make[3]: Leaving directory
`.../tmp/work/armv5te-angstrom-linux-gnueabi/attr-2.4.44-r3/attr-2.4.44'
| FATAL: oe_runmake failed
NOTE: package attr-2.4.44-r3: task do_install: Failed
ERROR: TaskFailed event exception, aborting
ERROR: Build of .../openembedded/recipes/udev/attr_2.4.44.bb do_install
failed
ERROR: Task 1641 (.../openembedded/recipes/udev/attr_2.4.44.bb,
do_install) failed with 256
Building acl does the same thing. Something is wrong with environment
variables exported by acl/attr recipes, which should point in ${D} but
are now pointing into root. I'm executing bitbake from a makefile, which
also seems to have and effect on this (starting bitbake directly
preserves those variables).
- Henri
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils
2010-05-25 7:22 ` Koen Kooi
2010-05-25 9:11 ` Grzegorz Toporek
@ 2010-05-25 14:50 ` Khem Raj
1 sibling, 0 replies; 7+ messages in thread
From: Khem Raj @ 2010-05-25 14:50 UTC (permalink / raw)
To: openembedded-devel
On (25/05/10 09:22), Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 25-05-10 08:49, raj.khem@gmail.com wrote:
> > From: Khem Raj <raj.khem@gmail.com>
> >
> > * Main purpose of the patch was to fix the problem where udev-utils was
> > needed to be in root file system just to get udevadm binary. So this
> > binary is moved into udev package instead.
> >
> > * Use INC_PR
> >
> > * Use udev.inc in all recipes.
> >
> > * All udev recipes built udev_151 and udev_124 runs fine too.
>
> I explicitly didn't use udev.inc in udev 151 because it made the recipe
> very hard to read. It would make more sense to do a udev-1xx.inc based
> on udev 151 and use that for future recipes.
On the contrary I thought it was more confusing to include udev.inc in some
and not in others.I find that the recipes are quite similar. infact do_install function could
be also commonized properly and extra bits put into install_append in given
recipes. I did not spend time doing that yet.
Then the description field is also weirdly overridden in recipes probably
that needs fixing as well.
Any specific reason to have udev-1xx.inc ?
I will leave udev_151 as it is and only commit the udevadm change for now
-Khem
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2010-05-25 14:55 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-25 6:49 [PATCH] udev: Use udev.inc and add udevadm to udev pacakge instead of udev-utils raj.khem
2010-05-25 7:22 ` Koen Kooi
2010-05-25 9:11 ` Grzegorz Toporek
2010-05-25 9:25 ` Koen Kooi
2010-05-25 10:04 ` Grzegorz Toporek
2010-05-25 9:55 ` Henri Bragge
2010-05-25 14:50 ` Khem Raj
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.