[PATCH V2] systemd: set the location of the kill binary

[PATCH V2] systemd: set the location of the kill binary

[Ross Burton]

Otherwise systemd uses AC_PROG_PATH and finds it in the sysroot, which won't
work on the target.

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-core/systemd/systemd_197.bb |    1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/systemd/systemd_197.bb b/meta/recipes-core/systemd/systemd_197.bb
index c765c18..001d111 100644
--- a/meta/recipes-core/systemd/systemd_197.bb
+++ b/meta/recipes-core/systemd/systemd_197.bb
@@ -63,6 +63,7 @@ EXTRA_OECONF = " --with-rootprefix=${base_prefix} \
                  --disable-microhttpd \
                  --without-python \
                  --with-sysvrcnd-path=${sysconfdir} \
+                 ac_cv_path_KILL=/bin/kill \
                "
 # uclibc does not have NSS
 EXTRA_OECONF_append_libc-uclibc = " --disable-myhostname "
-- 
1.7.10.4

Re: [PATCH V2] systemd: set the location of the kill binary

[Otavio Salvador]

On Tue, Feb 26, 2013 at 2:53 PM, Ross Burton <ross.burton@intel.com> wrote:
> Otherwise systemd uses AC_PROG_PATH and finds it in the sysroot, which won't
> work on the target.
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>

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

-- 
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 V2] systemd: set the location of the kill binary

[Otavio Salvador]

On Tue, Feb 26, 2013 at 4:52 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> On Tue, Feb 26, 2013 at 2:53 PM, Ross Burton <ross.burton@intel.com> wrote:
>> Otherwise systemd uses AC_PROG_PATH and finds it in the sysroot, which won't
>> work on the target.
>>
>> Signed-off-by: Ross Burton <ross.burton@intel.com>
>
> Acked-by: Otavio Salvador <otavio@ossystems.com.br>

Now I saw the Khem suggestion to use /usr/bin/env kill; seems good.

-- 
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 V2] systemd: set the location of the kill binary

[Koen Kooi]

On Tue, 2013-02-26 at 17:53 +0000, Ross Burton wrote:
> Otherwise systemd uses AC_PROG_PATH and finds it in the sysroot, which won't
> work on the target.
> 
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  meta/recipes-core/systemd/systemd_197.bb |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-core/systemd/systemd_197.bb b/meta/recipes-core/systemd/systemd_197.bb
> index c765c18..001d111 100644
> --- a/meta/recipes-core/systemd/systemd_197.bb
> +++ b/meta/recipes-core/systemd/systemd_197.bb
> @@ -63,6 +63,7 @@ EXTRA_OECONF = " --with-rootprefix=${base_prefix} \
>                   --disable-microhttpd \
>                   --without-python \
>                   --with-sysvrcnd-path=${sysconfdir} \
> +                 ac_cv_path_KILL=/bin/kill \

${base_bindir} ?

Re: [PATCH V2] systemd: set the location of the kill binary

[Koen Kooi]

On Tue, 2013-02-26 at 17:53 +0000, Ross Burton wrote:
> Otherwise systemd uses AC_PROG_PATH and finds it in the sysroot, which won't
> work on the target.
> 
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  meta/recipes-core/systemd/systemd_197.bb |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-core/systemd/systemd_197.bb b/meta/recipes-core/systemd/systemd_197.bb
> index c765c18..001d111 100644
> --- a/meta/recipes-core/systemd/systemd_197.bb
> +++ b/meta/recipes-core/systemd/systemd_197.bb
> @@ -63,6 +63,7 @@ EXTRA_OECONF = " --with-rootprefix=${base_prefix} \
>                   --disable-microhttpd \
>                   --without-python \
>                   --with-sysvrcnd-path=${sysconfdir} \
> +                 ac_cv_path_KILL=/bin/kill \

Like V1 this hardcodes ${base_bindir} to /bin, this breaks when your
DISTRO sets $base_prefix to /usr. So please change that to:

	ac_cv_path+KILL=${base_bindir}/kill

Re: [PATCH V2] systemd: set the location of the kill binary

[Phil Blundell]

On Tue, 2013-02-26 at 16:53 -0300, Otavio Salvador wrote:
> On Tue, Feb 26, 2013 at 4:52 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
> > On Tue, Feb 26, 2013 at 2:53 PM, Ross Burton <ross.burton@intel.com> wrote:
> >> Otherwise systemd uses AC_PROG_PATH and finds it in the sysroot, which won't
> >> work on the target.
> >>
> >> Signed-off-by: Ross Burton <ross.burton@intel.com>
> >
> > Acked-by: Otavio Salvador <otavio@ossystems.com.br>
> 
> Now I saw the Khem suggestion to use /usr/bin/env kill; seems good.

If it wants a target path then "/usr/bin/env kill" would fail for
targets that either don't have /usr or don't have "env" installed.  That
might not be so good.

/usr/bin/env would be fine if we were talking about a build host path
but that doesn't appear to be the case here.

p.

Re: [PATCH V2] systemd: set the location of the kill binary

[Burton, Ross]

On 26 February 2013 20:00, Koen Kooi <koen@dominion.thruhere.net> wrote:
> Like V1 this hardcodes ${base_bindir} to /bin, this breaks when your
> DISTRO sets $base_prefix to /usr. So please change that to:
>
>         ac_cv_path+KILL=${base_bindir}/kill

Yeah, not sure why I did that.  :/ V2 coming shortly.

Ross