[PATCH] udev-164: don't hardcode path to pidof

[PATCH] udev-164: don't hardcode path to pidof

[Andreas Oberritter]

* pidof lives in /bin, search it in $PATH.
* Assume pidof's presence.

Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
 meta/recipes-core/udev/udev/init   |    6 ++----
 meta/recipes-core/udev/udev_164.bb |    2 +-
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
index 78b5b1c..f9e9014 100644
--- a/meta/recipes-core/udev/udev/init
+++ b/meta/recipes-core/udev/udev/init
@@ -28,10 +28,8 @@ readfile () {
 }
 
 kill_udevd() {
-	if [ -x /sbin/pidof ]; then
-		pid=`/sbin/pidof -x udevd`
-		[ -n "$pid" ] && kill $pid
-	fi
+	pid=`pidof -x udevd`
+	[ -n "$pid" ] && kill $pid
 }
 
 export ACTION=add
diff --git a/meta/recipes-core/udev/udev_164.bb b/meta/recipes-core/udev/udev_164.bb
index b993a13..d661044 100644
--- a/meta/recipes-core/udev/udev_164.bb
+++ b/meta/recipes-core/udev/udev_164.bb
@@ -1,6 +1,6 @@
 include udev.inc
 
-PR = "r11"
+PR = "r12"
 
 SRC_URI += "file://udev-166-v4l1-1.patch"
 
-- 
1.7.5.4

Re: [PATCH] udev-164: don't hardcode path to pidof

[Otavio Salvador]

Acked-by: Otavio Salvador <otavio@ossystems.com.br>

On Thu, Mar 8, 2012 at 09:08, Andreas Oberritter <obi@opendreambox.org> wrote:
> * pidof lives in /bin, search it in $PATH.
> * Assume pidof's presence.
>
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>



-- 
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

Re: [PATCH] udev-164: don't hardcode path to pidof

[Denys Dmytriyenko]

On Thu, Mar 08, 2012 at 01:08:50PM +0100, Andreas Oberritter wrote:
> * pidof lives in /bin, search it in $PATH.

Nice catch! It does indeed live in /bin

> * Assume pidof's presence.
> 
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>

Acked-by: Denys Dmytriyenko <denys@ti.com>

> ---
>  meta/recipes-core/udev/udev/init   |    6 ++----
>  meta/recipes-core/udev/udev_164.bb |    2 +-
>  2 files changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index 78b5b1c..f9e9014 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -28,10 +28,8 @@ readfile () {
>  }
>  
>  kill_udevd() {
> -	if [ -x /sbin/pidof ]; then
> -		pid=`/sbin/pidof -x udevd`
> -		[ -n "$pid" ] && kill $pid
> -	fi
> +	pid=`pidof -x udevd`
> +	[ -n "$pid" ] && kill $pid
>  }
>  
>  export ACTION=add
> diff --git a/meta/recipes-core/udev/udev_164.bb b/meta/recipes-core/udev/udev_164.bb
> index b993a13..d661044 100644
> --- a/meta/recipes-core/udev/udev_164.bb
> +++ b/meta/recipes-core/udev/udev_164.bb
> @@ -1,6 +1,6 @@
>  include udev.inc
>  
> -PR = "r11"
> +PR = "r12"
>  
>  SRC_URI += "file://udev-166-v4l1-1.patch"
>  
> -- 
> 1.7.5.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Re: [PATCH] udev-164: don't hardcode path to pidof

[Richard Purdie]

On Thu, 2012-03-08 at 13:08 +0100, Andreas Oberritter wrote:
> * pidof lives in /bin, search it in $PATH.
> * Assume pidof's presence.
> 
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> ---
>  meta/recipes-core/udev/udev/init   |    6 ++----
>  meta/recipes-core/udev/udev_164.bb |    2 +-
>  2 files changed, 3 insertions(+), 5 deletions(-)

Merged to master, thanks.

Richard