On Thursday 05 February 2015 at 17:50:23 +0000, Mike Crowe wrote:
> ${bindir} and ${base_bindir} may be the same. If they are don't try and
> move files onto themselves.
> 
> Signed-off-by: Mike Crowe <m...@mcrowe.com>
> ---
>  meta/recipes-extended/sed/sed_4.1.2.bb | 6 ++++--
>  meta/recipes-extended/sed/sed_4.2.2.bb | 6 ++++--
>  2 files changed, 8 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/recipes-extended/sed/sed_4.1.2.bb 
> b/meta/recipes-extended/sed/sed_4.1.2.bb
> index 4c0f345..40e3a53 100644
> --- a/meta/recipes-extended/sed/sed_4.1.2.bb
> +++ b/meta/recipes-extended/sed/sed_4.1.2.bb
> @@ -23,8 +23,10 @@ do_configure_prepend () {
>  do_install () {
>       autotools_do_install
>       install -d ${D}${base_bindir}
> -     mv ${D}${bindir}/sed ${D}${base_bindir}/sed
> -     rmdir ${D}${bindir}/
> +     if [ ! ${D}${bindir} -ef ${D}${base_bindir} ]; then
> +         mv ${D}${bindir}/sed ${D}${base_bindir}/sed
> +         rmdir ${D}${bindir}/
> +     fi
>  }
>  
>  ALTERNATIVE_${PN} = "sed"
> diff --git a/meta/recipes-extended/sed/sed_4.2.2.bb 
> b/meta/recipes-extended/sed/sed_4.2.2.bb
> index 72976de..bbfa238 100644
> --- a/meta/recipes-extended/sed/sed_4.2.2.bb
> +++ b/meta/recipes-extended/sed/sed_4.2.2.bb
> @@ -22,8 +22,10 @@ EXTRA_OECONF = "--disable-acl \
>  do_install () {
>       autotools_do_install
>       install -d ${D}${base_bindir}
> -     mv ${D}${bindir}/sed ${D}${base_bindir}/sed
> -     rmdir ${D}${bindir}/
> +     if [ ! ${D}${bindir} -ef ${D}${base_bindir} ]; then
> +         mv ${D}${bindir}/sed ${D}${base_bindir}/sed
> +         rmdir ${D}${bindir}/
> +     fi
>  }
>  
>  ALTERNATIVE_${PN} = "sed"
> -- 
> 2.1.4

It doesn't look like this patch has been merged. Is there something wrong
with it than I need to attend to?

Thanks.

Mike.
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to