Re: [oe] [meta-oe][PATCH v2] minicoredumper: relocate admin-binaries to sbin

[Koen Kooi]

Op 10-04-18 om 17:39 schreef Christophe PRIOUZEAU:
> Change path of minicoredumper on init/service script to /usr/sbin.
> 
> Since the commit "relocate admin-binaries to sbin" on minicoredumper,
> minicoredumper binaries are installed on /usr/sbin directory instead
> of /usr/bin.
> src/minicoredumper/Makefile.am: sbin_PROGRAMS = minicoredumper
> 
> Signed-off-by: Christophe Priouzeau 
> ---
>  meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init| 2 +-
>  meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init 
> b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init
> index 90ef7fa..9c96e31 100644
> --- a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init
> +++ b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init
> @@ -22,7 +22,7 @@
>  
>  case "$1" in
>  start)
> -echo "|/usr/bin/minicoredumper %p %u %g %s %t %h %e" > 
> /proc/sys/kernel/core_pattern
> +echo "|/usr/sbin/minicoredumper %p %u %g %s %t %h %e" > 
> /proc/sys/kernel/core_pattern
>  ;;
>  stop)
>  echo "core" > /proc/sys/kernel/core_pattern
> diff --git 
> a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service 
> b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service
> index 851b54c..45d098f 100644
> --- a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service
> +++ b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service
> @@ -4,7 +4,7 @@ Description=Enable minicoredumper.
>  [Service]
>  Type=oneshot
>  RemainAfterExit=yes
> -ExecStart=/bin/sh -c '/bin/echo "|/usr/bin/minicoredumper %%p %%u %%g %%s 
> %%t %%h %%e" > /proc/sys/kernel/core_pattern'
> +ExecStart=/bin/sh -c '/bin/echo "|/usr/sbin/minicoredumper %%p %%u %%g %%s 
> %%t %%h %%e" > /proc/sys/kernel/core_pattern'
>  ExecStop=/bin/sh -c '/bin/echo "core" > /proc/sys/kernel/core_pattern'
>  
>  [Install]


You're trading one hardcoded entry for another, a bettter way would be to do 
something like

do_install_append() {
sed -i -e s:/usr/bin/minicoredumper:${sbindir}/minicoredumper:g 
}

That would fix the situation for DISTROs where $prefix = "".

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[oe] [meta-oe][PATCH v2] minicoredumper: relocate admin-binaries to sbin

[Christophe PRIOUZEAU]

Change path of minicoredumper on init/service script to /usr/sbin.

Since the commit "relocate admin-binaries to sbin" on minicoredumper,
minicoredumper binaries are installed on /usr/sbin directory instead
of /usr/bin.
src/minicoredumper/Makefile.am: sbin_PROGRAMS = minicoredumper

Signed-off-by: Christophe Priouzeau 
---
 meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init| 2 +-
 meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init 
b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init
index 90ef7fa..9c96e31 100644
--- a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init
+++ b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.init
@@ -22,7 +22,7 @@
 
 case "$1" in
 start)
-echo "|/usr/bin/minicoredumper %p %u %g %s %t %h %e" > 
/proc/sys/kernel/core_pattern
+echo "|/usr/sbin/minicoredumper %p %u %g %s %t %h %e" > 
/proc/sys/kernel/core_pattern
 ;;
 stop)
 echo "core" > /proc/sys/kernel/core_pattern
diff --git a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service 
b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service
index 851b54c..45d098f 100644
--- a/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service
+++ b/meta-oe/recipes-kernel/minicoredumper/files/minicoredumper.service
@@ -4,7 +4,7 @@ Description=Enable minicoredumper.
 [Service]
 Type=oneshot
 RemainAfterExit=yes
-ExecStart=/bin/sh -c '/bin/echo "|/usr/bin/minicoredumper %%p %%u %%g %%s %%t 
%%h %%e" > /proc/sys/kernel/core_pattern'
+ExecStart=/bin/sh -c '/bin/echo "|/usr/sbin/minicoredumper %%p %%u %%g %%s %%t 
%%h %%e" > /proc/sys/kernel/core_pattern'
 ExecStop=/bin/sh -c '/bin/echo "core" > /proc/sys/kernel/core_pattern'
 
 [Install]
-- 
2.7.4
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel