Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] udev: changed location of udevd to /sbin/udev/udevd
@ 2013-04-09 15:58 Alex DAMIAN
  2013-04-09 16:07 ` Saul Wold
  0 siblings, 1 reply; 4+ messages in thread
From: Alex DAMIAN @ 2013-04-09 15:58 UTC (permalink / raw)
  To: sgw, richard.purdie, openembedded-core; +Cc: Alexandru DAMIAN

From: Alexandru DAMIAN <alexandru.damian@intel.com>

This ammends the init script after changing the location from
/lib to /sbin.

Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
---
 meta/recipes-core/udev/udev/init |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
index 95d2672..ea87d64 100644
--- a/meta/recipes-core/udev/udev/init
+++ b/meta/recipes-core/udev/udev/init
@@ -15,7 +15,7 @@ export TZ=/etc/localtime
 
 [ -d /sys/class ] || exit 1
 [ -r /proc/mounts ] || exit 1
-[ -x /lib/udev/udevd ] || exit 1
+[ -x /sbin/udev/udevd ] || exit 1
 [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
 [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
 
@@ -71,7 +71,7 @@ case "$1" in
 
     # trigger the sorted events
     echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
-    /lib/udev/udevd -d
+    /sbin/udev/udevd -d
 
     /usr/bin/udevadm control --env=STARTUP=1
     if [ "$not_first_boot" != "" ];then
-- 
1.7.10.4




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] udev: changed location of udevd to /sbin/udev/udevd
  2013-04-09 15:58 [PATCH] udev: changed location of udevd to /sbin/udev/udevd Alex DAMIAN
@ 2013-04-09 16:07 ` Saul Wold
  2013-04-09 16:35   ` Otavio Salvador
  0 siblings, 1 reply; 4+ messages in thread
From: Saul Wold @ 2013-04-09 16:07 UTC (permalink / raw)
  To: Alex DAMIAN; +Cc: openembedded-core

On 04/09/2013 08:58 AM, Alex DAMIAN wrote:
> From: Alexandru DAMIAN <alexandru.damian@intel.com>
>
> This ammends the init script after changing the location from
> /lib to /sbin.
>
This only fixes one location and does not make it dynamic, I will have a 
patch shortly to fix this.

Sau!

> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
> ---
>   meta/recipes-core/udev/udev/init |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index 95d2672..ea87d64 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -15,7 +15,7 @@ export TZ=/etc/localtime
>
>   [ -d /sys/class ] || exit 1
>   [ -r /proc/mounts ] || exit 1
> -[ -x /lib/udev/udevd ] || exit 1
> +[ -x /sbin/udev/udevd ] || exit 1
>   [ -f /etc/default/udev-cache ] && . /etc/default/udev-cache
>   [ -f /etc/udev/udev.conf ] && . /etc/udev/udev.conf
>
> @@ -71,7 +71,7 @@ case "$1" in
>
>       # trigger the sorted events
>       echo -e '\000\000\000\000' > /proc/sys/kernel/hotplug
> -    /lib/udev/udevd -d
> +    /sbin/udev/udevd -d
>
>       /usr/bin/udevadm control --env=STARTUP=1
>       if [ "$not_first_boot" != "" ];then
>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] udev: changed location of udevd to /sbin/udev/udevd
  2013-04-09 16:07 ` Saul Wold
@ 2013-04-09 16:35   ` Otavio Salvador
  2013-04-09 17:06     ` Richard Purdie
  0 siblings, 1 reply; 4+ messages in thread
From: Otavio Salvador @ 2013-04-09 16:35 UTC (permalink / raw)
  To: Saul Wold; +Cc: Alex DAMIAN, Patches and discussions about the oe-core layer

On Tue, Apr 9, 2013 at 1:07 PM, Saul Wold <sgw@linux.intel.com> wrote:
> On 04/09/2013 08:58 AM, Alex DAMIAN wrote:
>>
>> From: Alexandru DAMIAN <alexandru.damian@intel.com>
>>
>> This ammends the init script after changing the location from
>> /lib to /sbin.
>>
> This only fixes one location and does not make it dynamic, I will have a
> patch shortly to fix this.

Your fix does not make it dynamic but staticly set at build time.
Please iterate over the possible alternatives, as I did in
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-core/initrdscripts?id=1b99640481882d23dc3ded41d9f2aef906f77e67

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] udev: changed location of udevd to /sbin/udev/udevd
  2013-04-09 16:35   ` Otavio Salvador
@ 2013-04-09 17:06     ` Richard Purdie
  0 siblings, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2013-04-09 17:06 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: layer, Alex DAMIAN, Patches

On Tue, 2013-04-09 at 13:35 -0300, Otavio Salvador wrote:
> On Tue, Apr 9, 2013 at 1:07 PM, Saul Wold <sgw@linux.intel.com> wrote:
> > On 04/09/2013 08:58 AM, Alex DAMIAN wrote:
> >>
> >> From: Alexandru DAMIAN <alexandru.damian@intel.com>
> >>
> >> This ammends the init script after changing the location from
> >> /lib to /sbin.
> >>
> > This only fixes one location and does not make it dynamic, I will have a
> > patch shortly to fix this.
> 
> Your fix does not make it dynamic but staticly set at build time.
> Please iterate over the possible alternatives, as I did in
> http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-core/initrdscripts?id=1b99640481882d23dc3ded41d9f2aef906f77e67

In the case of udev itself, we don't need to do this since it knows
where it installed the binaries to.

Cheers,

Richard





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-04-09 17:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-09 15:58 [PATCH] udev: changed location of udevd to /sbin/udev/udevd Alex DAMIAN
2013-04-09 16:07 ` Saul Wold
2013-04-09 16:35   ` Otavio Salvador
2013-04-09 17:06     ` Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox