This patch still stands.<br><br>I took a look at meta-oe. IMHO, the way the build works here is the right way.<br><br>I had no problems rebasing it to the latest HEAD, df5f9a3e6dfcf8e8f57132843a024b44c0bd90fc,<br>and rebuilding.<br>
<br>Tested as follows:<br>- build core-image-sato<br>- running qemux86 image<br>- udevd is started correctly. the image works normally under usage smoke tests<br>- udevd is verified to be version 182<br><br>If you'll see problems building, please let me have a log.<br>
<br>Alex<br><br><div class="gmail_quote">On Wed, Sep 19, 2012 at 9:02 PM, Khem Raj <span dir="ltr"><<a href="mailto:raj.khem@gmail.com" target="_blank">raj.khem@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Tue, Sep 11, 2012 at 1:53 PM, Saul Wold <<a href="mailto:sgw@linux.intel.com">sgw@linux.intel.com</a>> wrote:<br>
> On 09/06/2012 02:27 AM, Alex DAMIAN wrote:<br>
>><br>
>> From: Alexandru DAMIAN <<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>><br>
>><br>
>> This is the final upgrade of udev. Futher upgrades will only<br>
>> come in conjunction with systemd.<br>
>><br>
>> The v4l1 removal patch is deprecated as the bug is fixed inside udev.<br>
>> There is a new patch fixing the path for default sh interpreter.<br>
>> New debug binaries are generated, and udev.inc is modified to package<br>
>> those correctly.<br>
>> The install locations changed for udevd and udevadm, so the scripts<br>
>> are updated accordingly.<br>
>><br>
>> Signed-off-by: Alexandru DAMIAN <<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>><br>
>><br>
><br>
> Alex,<br>
><br>
> I have had some problems with this patch trying to build it. It was looking<br>
> on my host for usbutils version .82 or greater, but we have the new numbered<br>
> 006 usbutils and it failed.<br>
><br>
> Also, why did you not take the meta-oe version, did you compare your changes<br>
> with the version is meta-oe? This was mentioned once before?<br>
><br>
> Thanks<br>
> Sau!<br>
><br>
><br>
<br>
</div>Alex<br>
<br>
whats status of this patch is there is a v4 expected soon<br>
<div><div class="h5"><br>
><br>
>> Conflicts:<br>
>><br>
>> meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
>> ---<br>
>> .../initscripts/initscripts-1.0/volatiles | 2 +-<br>
>> meta/recipes-core/udev/udev.inc | 12 +++--<br>
>> ...yboard_force_release.sh-shell-script-path.patch | 35 ++++++++++++++<br>
>> meta/recipes-core/udev/udev/include_resource.patch | 31 ------------<br>
>> meta/recipes-core/udev/udev/init | 14 +++---<br>
>> meta/recipes-core/udev/udev/udev-166-v4l1-1.patch | 50<br>
>> --------------------<br>
>> meta/recipes-core/udev/udev/udev-cache | 2 +-<br>
>> meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a> | 9 ----<br>
>> meta/recipes-core/udev/<a href="http://udev_182.bb" target="_blank">udev_182.bb</a> | 9 ++++<br>
>> 9 files changed, 61 insertions(+), 103 deletions(-)<br>
>> create mode 100644<br>
>> meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch<br>
>> delete mode 100644 meta/recipes-core/udev/udev/include_resource.patch<br>
>> delete mode 100644 meta/recipes-core/udev/udev/udev-166-v4l1-1.patch<br>
>> delete mode 100644 meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
>> create mode 100644 meta/recipes-core/udev/<a href="http://udev_182.bb" target="_blank">udev_182.bb</a><br>
>><br>
>> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles<br>
>> b/meta/recipes-core/initscripts/initscripts-1.0/volatiles<br>
>> index b2ae279..e0741aa 100644<br>
>> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles<br>
>> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles<br>
>> @@ -36,4 +36,4 @@ f root root 0664 /var/log/wtmp none<br>
>> f root root 0664 /var/run/utmp none<br>
>> l root root 0644 /etc/resolv.conf /var/run/resolv.conf<br>
>> f root root 0644 /var/run/resolv.conf none<br>
>> -<br>
>> +l root root 0755 /run /var/run<br>
>> diff --git a/meta/recipes-core/udev/udev.inc<br>
>> b/meta/recipes-core/udev/udev.inc<br>
>> index 9cc00e8..fafd9ce 100644<br>
>> --- a/meta/recipes-core/udev/udev.inc<br>
>> +++ b/meta/recipes-core/udev/udev.inc<br>
>> @@ -6,9 +6,10 @@ LICENSE = "GPLv2.0+ & LGPLv2.1+"<br>
>> LICENSE_${PN} = "GPLv2.0+"<br>
>> LICENSE_libudev = "LGPLv2.1+"<br>
>> LICENSE_libgudev = "LGPLv2.1+"<br>
>> -LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \<br>
>> -<br>
>> file://libudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343 \<br>
>> -<br>
>> file://extras/gudev/COPYING;md5=a6f89e2100d9b6cdffcea4f398e37343"<br>
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \<br>
>> +<br>
>> file://src/COPYING;md5=17c4e5fb495e6707ac92a3864926f979 \<br>
>> +<br>
>> file://src/gudev/COPYING;md5=fb494485a7d0505308cb68e4997cc266"<br>
>> +<br>
>><br>
>> DEPENDS = "acl glib-2.0 libusb usbutils pciutils gperf-native<br>
>> libxslt-native"<br>
>> RPROVIDES_${PN} = "hotplug"<br>
>> @@ -16,6 +17,7 @@ RRECOMMENDS_${PN} += "udev-extraconf usbutils-ids<br>
>> pciutils-ids"<br>
>> RDEPENDS_libudev = "${PN} (= ${EXTENDPKGV})"<br>
>><br>
>> SRC_URI =<br>
>> "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \<br>
>> +<br>
>> file://0001-Fixing-keyboard_force_release.sh-shell-script-path.patch \<br>
>> file://run.rules \<br>
>> file://udev.rules \<br>
>> file://devfs-udev.rules \<br>
>> @@ -32,7 +34,7 @@ inherit autotools pkgconfig update-rc.d<br>
>><br>
>> # udevd/udevadm -> /sbin/, libudev.so.* -> /lib/<br>
>> sbindir = "${base_sbindir}"<br>
>> -libexecdir = "${base_libdir}/udev"<br>
>> +libexecdir = "${base_libdir}"<br>
>> EXTRA_OECONF = "--disable-introspection --with-rootlibdir=${base_libdir}<br>
>> \<br>
>> --with-pci-ids-path=${datadir}/pci.ids"<br>
>><br>
>> @@ -50,6 +52,8 @@ FILES_${PN} += "${libexecdir} ${libdir}/ConsoleKit"<br>
>> RRECOMMENDS_${PN} += "udev-utils"<br>
>><br>
>> FILES_${PN}-dbg += "${libexecdir}/.debug"<br>
>> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"<br>
>> +FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"<br>
>> FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"<br>
>> FILES_libudev = "${base_libdir}/libudev.so.*"<br>
>> FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"<br>
>> diff --git<br>
>> a/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch<br>
>> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch<br>
>> new file mode 100644<br>
>> index 0000000..41deafa<br>
>> --- /dev/null<br>
>> +++<br>
>> b/meta/recipes-core/udev/udev/0001-Fixing-keyboard_force_release.sh-shell-script-path.patch<br>
>> @@ -0,0 +1,35 @@<br>
>> +From 0f8290c943da298abd269ca60fd8375dfb219971 Mon Sep 17 00:00:00 2001<br>
>> +From: Alexandru DAMIAN <<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>><br>
>> +Date: Thu, 12 Jul 2012 12:54:48 +0300<br>
>> +Subject: [PATCH] Fixing keyboard_force_release.sh shell script path<br>
>> +<br>
>> +With the introduction of rootprefix, the <a href="http://keyboard-force-release.sh.in" target="_blank">keyboard-force-release.sh.in</a><br>
>> +was modified to be executed with @rootprefix@/bin/sh, which is wrong<br>
>> +because @rootprefix@ defaults to /usr (which is correct), but the<br>
>> +shell is always at /bin/sh (IEEE Std 1003.2-1992).<br>
>> +<br>
>> +Therefore the interpreter for shell scripts needs to be /bin/sh at all<br>
>> times.<br>
>> +<br>
>> +The upstream moved to configurable root prefix, this patch taclkes a<br>
>> +transition bug, and will not be applied upstream.<br>
>> +<br>
>> +Upstream-Status: Inappropriate [legacy version]<br>
>> +<br>
>> +Signed-off-by: Alexandru DAMIAN <<a href="mailto:alexandru.damian@intel.com">alexandru.damian@intel.com</a>><br>
>> +---<br>
>> + src/keymap/<a href="http://keyboard-force-release.sh.in" target="_blank">keyboard-force-release.sh.in</a> | 2 +-<br>
>> + 1 file changed, 1 insertion(+), 1 deletion(-)<br>
>> +<br>
>> +diff --git a/src/keymap/<a href="http://keyboard-force-release.sh.in" target="_blank">keyboard-force-release.sh.in</a><br>
>> b/src/keymap/<a href="http://keyboard-force-release.sh.in" target="_blank">keyboard-force-release.sh.in</a><br>
>> +index dd040ce..597a3a6 100755<br>
>> +--- a/src/keymap/<a href="http://keyboard-force-release.sh.in" target="_blank">keyboard-force-release.sh.in</a><br>
>> ++++ b/src/keymap/<a href="http://keyboard-force-release.sh.in" target="_blank">keyboard-force-release.sh.in</a><br>
>> +@@ -1,4 +1,4 @@<br>
>> +-#!@rootprefix@/bin/sh -e<br>
>> ++#!/bin/sh -e<br>
>> + # read list of scancodes, convert hex to decimal and<br>
>> + # append to the atkbd force_release sysfs attribute<br>
>> + # $1 sysfs devpath for serioX<br>
>> +--<br>
>> +1.7.9.5<br>
>> +<br>
>> diff --git a/meta/recipes-core/udev/udev/include_resource.patch<br>
>> b/meta/recipes-core/udev/udev/include_resource.patch<br>
>> deleted file mode 100644<br>
>> index a46ce59..0000000<br>
>> --- a/meta/recipes-core/udev/udev/include_resource.patch<br>
>> +++ /dev/null<br>
>> @@ -1,31 +0,0 @@<br>
>> -Fixes errors like below with eglibc 2.16<br>
>> -<br>
>> -libudev/libudev-util-private.c:348:16: error: 'PRIO_PROCESS' undeclared<br>
>> (first use in this function)<br>
>> -<br>
>> -Signed-off-by: Khem Raj <<a href="mailto:raj.khem@gmail.com">raj.khem@gmail.com</a>><br>
>> -Upstream-Status: Pending<br>
>> -<br>
>> -Index: udev-164/libudev/libudev-util-private.c<br>
>> -===================================================================<br>
>> ---- udev-164.orig/libudev/libudev-util-private.c 2010-08-03<br>
>> 02:42:26.000000000 -0700<br>
>> -+++ udev-164/libudev/libudev-util-private.c 2012-07-03<br>
>> 10:53:35.082346684 -0700<br>
>> -@@ -21,6 +21,7 @@<br>
>> - #include <grp.h><br>
>> - #include <sys/wait.h><br>
>> - #include <sys/param.h><br>
>> -+#include <sys/resource.h><br>
>> -<br>
>> - #include "libudev.h"<br>
>> - #include "libudev-private.h"<br>
>> -Index: udev-164/udev/udevd.c<br>
>> -===================================================================<br>
>> ---- udev-164.orig/udev/udevd.c 2010-10-13 04:08:29.000000000 -0700<br>
>> -+++ udev-164/udev/udevd.c 2012-07-03 10:53:19.214345914 -0700<br>
>> -@@ -43,6 +43,7 @@<br>
>> - #include <sys/ioctl.h><br>
>> - #include <sys/inotify.h><br>
>> - #include <sys/utsname.h><br>
>> -+#include <sys/resource.h><br>
>> -<br>
>> - #include "udev.h"<br>
>> - #include "sd-daemon.h"<br>
>> diff --git a/meta/recipes-core/udev/udev/init<br>
>> b/meta/recipes-core/udev/udev/init<br>
>> index d521685..ca02abb 100644<br>
>> --- a/meta/recipes-core/udev/udev/init<br>
>> +++ b/meta/recipes-core/udev/udev/init<br>
>> @@ -13,7 +13,7 @@ export TZ=/etc/localtime<br>
>><br>
>> [ -d /sys/class ] || exit 1<br>
>> [ -r /proc/mounts ] || exit 1<br>
>> -[ -x /sbin/udevd ] || exit 1<br>
>> +[ -x /lib/udev/udevd ] || exit 1<br>
>> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache<br>
>> [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf<br>
>><br>
>> @@ -70,15 +70,15 @@ kill_udevd > "/dev/null" 2>&1<br>
>><br>
>> # trigger the sorted events<br>
>> echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug<br>
>> -/sbin/udevd -d<br>
>> +/lib/udev/udevd -d<br>
>><br>
>> -/sbin/udevadm control --env=STARTUP=1<br>
>> +/usr/bin/udevadm control --env=STARTUP=1<br>
>> if [ "$not_first_boot" != "" ];then<br>
>> - /sbin/udevadm trigger --action=add --subsystem-nomatch=tty<br>
>> --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole<br>
>> --subsystem-nomatch=misc --subsystem-nomatch=dcon<br>
>> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics<br>
>> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux<br>
>> --subsystem-nomatch=platform<br>
>> - (/sbin/udevadm settle --timeout=3; /sbin/udevadm control<br>
>> --env=STARTUP=)&<br>
>> + /usr/bin/udevadm trigger --action=add --subsystem-nomatch=tty<br>
>> --subsystem-nomatch=mem --subsystem-nomatch=vc --subsystem-nomatch=vtconsole<br>
>> --subsystem-nomatch=misc --subsystem-nomatch=dcon<br>
>> --subsystem-nomatch=pci_bus --subsystem-nomatch=graphics<br>
>> --subsystem-nomatch=backlight --subsystem-nomatch=video4linux<br>
>> --subsystem-nomatch=platform<br>
>> + (/usr/bin/udevadm settle --timeout=3; /usr/bin/udevadm control<br>
>> --env=STARTUP=)&<br>
>> else<br>
>> - /sbin/udevadm trigger --action=add<br>
>> - /sbin/udevadm settle<br>
>> + /usr/bin/udevadm trigger --action=add<br>
>> + /usr/bin/udevadm settle<br>
>> fi<br>
>><br>
>> exit 0<br>
>> diff --git a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch<br>
>> b/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch<br>
>> deleted file mode 100644<br>
>> index 2086fe1..0000000<br>
>> --- a/meta/recipes-core/udev/udev/udev-166-v4l1-1.patch<br>
>> +++ /dev/null<br>
>> @@ -1,50 +0,0 @@<br>
>> -Upstream-Status: Backport<br>
>> -<br>
>> -Submitted By: Matt Burgess<br>
>> <matthew_at_linuxfromscratch_dot_org><br>
>> -Date: 2011-03-26<br>
>> -Initial Package Version: 166<br>
>> -Upstream Status: From upstream<br>
>> -Origin:<br>
>> <a href="http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b" target="_blank">http://git.kernel.org/?p=linux/hotplug/udev.git;a=commitdiff;h=4ace8a43ac2cbbd4d6f5c29fc461c3caa8f8545b</a><br>
>> -Description: Fixes a compilation error caused by the removal<br>
>> of the<br>
>> - Video for Linux 1 API from Linux kernels from<br>
>> 2.6.38<br>
>> - onwards.<br>
>> -<br>
>> -diff -Naur udev-166.orig/extras/v4l_id/v4l_id.c<br>
>> udev-166/extras/v4l_id/v4l_id.c<br>
>> ---- udev-166.orig/extras/v4l_id/v4l_id.c 2009-12-03<br>
>> 12:45:03.000000000 +0000<br>
>> -+++ udev-166/extras/v4l_id/v4l_id.c 2011-03-25 20:26:33.000000000<br>
>> +0000<br>
>> -@@ -28,7 +28,6 @@<br>
>> - #include <sys/types.h><br>
>> - #include <sys/time.h><br>
>> - #include <sys/ioctl.h><br>
>> --#include <linux/videodev.h><br>
>> - #include <linux/videodev2.h><br>
>> -<br>
>> - int main (int argc, char *argv[])<br>
>> -@@ -39,7 +38,6 @@<br>
>> - };<br>
>> - int fd;<br>
>> - char *device;<br>
>> -- struct video_capability v1cap;<br>
>> - struct v4l2_capability v2cap;<br>
>> -<br>
>> - while (1) {<br>
>> -@@ -82,19 +80,6 @@<br>
>> - if ((v2cap.capabilities & V4L2_CAP_RADIO) > 0)<br>
>> - printf("radio:");<br>
>> - printf("\n");<br>
>> -- } else if (ioctl (fd, VIDIOCGCAP, &v1cap) == 0) {<br>
>> -- printf("ID_V4L_VERSION=1\n");<br>
>> -- printf("ID_V4L_PRODUCT=%s\n", <a href="http://v1cap.name" target="_blank">v1cap.name</a>);<br>
>> -- printf("ID_V4L_CAPABILITIES=:");<br>
>> -- if ((v1cap.type & VID_TYPE_CAPTURE) > 0)<br>
>> -- printf("capture:");<br>
>> -- if ((v1cap.type & VID_TYPE_OVERLAY) > 0)<br>
>> -- printf("video_overlay:");<br>
>> -- if (v1cap.audios > 0)<br>
>> -- printf("audio:");<br>
>> -- if ((v1cap.type & VID_TYPE_TUNER) > 0)<br>
>> -- printf("tuner:");<br>
>> -- printf("\n");<br>
>> - }<br>
>> -<br>
>> - close (fd);<br>
>> diff --git a/meta/recipes-core/udev/udev/udev-cache<br>
>> b/meta/recipes-core/udev/udev/udev-cache<br>
>> index 9c24e76..0f3d7cd 100644<br>
>> --- a/meta/recipes-core/udev/udev/udev-cache<br>
>> +++ b/meta/recipes-core/udev/udev/udev-cache<br>
>> @@ -12,7 +12,7 @@<br>
>> export TZ=/etc/localtime<br>
>><br>
>> [ -r /proc/mounts ] || exit 1<br>
>> -[ -x /sbin/udevd ] || exit 1<br>
>> +[ -x /lib/udev/udevd ] || exit 1<br>
>> [ -d /sys/class ] || exit 1<br>
>><br>
>> [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache<br>
>> diff --git a/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
>> b/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
>> deleted file mode 100644<br>
>> index 4d22bc8..0000000<br>
>> --- a/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
>> +++ /dev/null<br>
>> @@ -1,9 +0,0 @@<br>
>> -include udev.inc<br>
>> -<br>
>> -PR = "r16"<br>
>> -<br>
>> -SRC_URI += "file://udev-166-v4l1-1.patch \<br>
>> - file://include_resource.patch \<br>
>> - "<br>
>> -SRC_URI[md5sum] = "fddac2d54761ea34865af9467377ca9f"<br>
>> -SRC_URI[sha256sum] =<br>
>> "c12e66280b5e1465f6587a8cfa47d7405c4caa7e52ce5dd13478d04f6ec05e5c"<br>
>> diff --git a/meta/recipes-core/udev/<a href="http://udev_182.bb" target="_blank">udev_182.bb</a><br>
>> b/meta/recipes-core/udev/<a href="http://udev_182.bb" target="_blank">udev_182.bb</a><br>
>> new file mode 100644<br>
>> index 0000000..6678fe1<br>
>> --- /dev/null<br>
>> +++ b/meta/recipes-core/udev/<a href="http://udev_182.bb" target="_blank">udev_182.bb</a><br>
>> @@ -0,0 +1,9 @@<br>
>> +include udev.inc<br>
>> +<br>
>> +PR = "r0"<br>
>> +<br>
>> +# module-init-tools from kmod_git will provide libkmod runtime<br>
>> +DEPENDS = "module-init-tools"<br>
>> +<br>
>> +SRC_URI[md5sum] = "1b964456177fbf48023dfee7db3a708d"<br>
>> +SRC_URI[sha256sum] =<br>
>> "7857ed19fafd8f3ca8de410194e8c7336e9eb8a20626ea8a4ba6449b017faba4"<br>
>><br>
><br>
</div></div>> _______________________________________________<br>
> Openembedded-core mailing list<br>
> <a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a><br>
> <a href="http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core" target="_blank">http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core</a><br>
</blockquote></div><br>