[PATCH] gnu-config: removed absolute perl binary paths from gnu-configize.

[PATCH] gnu-config: removed absolute perl binary paths from gnu-configize.

[Filip Zyzniewski]

Related bugzilla entry: http://bugs.openembedded.org/show_bug.cgi?id=5510
---
 recipes/gnu-config/files/gnu-configize.in |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes/gnu-config/files/gnu-configize.in b/recipes/gnu-config/files/gnu-configize.in
index be8580c..adf9c0e 100755
--- a/recipes/gnu-config/files/gnu-configize.in
+++ b/recipes/gnu-config/files/gnu-configize.in
@@ -1,7 +1,7 @@
-#! /usr/bin/perl -w
+#! /usr/bin/env perl
 # -*- perl -*-
 
-eval 'case $# in 0) exec /usr/bin/perl -S "$0";; *) exec /usr/bin/perl -S "$0" "$@";; esac'
+eval 'case $# in 0) exec perl -S "$0";; *) exec perl -S "$0" "$@";; esac'
     if 0;
 
 # gnu-configize - install the GNU config.guess / config.sub in a directory tree
-- 
1.7.1

Re: [PATCH] gnu-config: removed absolute perl binary paths from gnu-configize.

[Khem Raj]

On Sat, Feb 5, 2011 at 9:26 AM, Filip Zyzniewski
<filip.zyzniewski@gmail.com> wrote:
> Related bugzilla entry: http://bugs.openembedded.org/show_bug.cgi?id=5510
> ---
>  recipes/gnu-config/files/gnu-configize.in |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes/gnu-config/files/gnu-configize.in b/recipes/gnu-config/files/gnu-configize.in
> index be8580c..adf9c0e 100755
> --- a/recipes/gnu-config/files/gnu-configize.in
> +++ b/recipes/gnu-config/files/gnu-configize.in
> @@ -1,7 +1,7 @@
> -#! /usr/bin/perl -w
> +#! /usr/bin/env perl
>  # -*- perl -*-
>
> -eval 'case $# in 0) exec /usr/bin/perl -S "$0";; *) exec /usr/bin/perl -S "$0" "$@";; esac'
> +eval 'case $# in 0) exec perl -S "$0";; *) exec perl -S "$0" "$@";; esac'
>     if 0;
>
>  # gnu-configize - install the GNU config.guess / config.sub in a directory tree
> --
> 1.7.1
>

Looks ok however signoff it properly and resend. See
http://openembedded.org/index.php/How_to_submit_a_patch_to_OpenEmbedded
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>

Re: [PATCH] gnu-config: removed absolute perl binary paths from gnu-configize.

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1321 bytes --]

On Sat, Feb 05, 2011 at 06:50:16PM -0800, Khem Raj wrote:
> On Sat, Feb 5, 2011 at 9:26 AM, Filip Zyzniewski
> <filip.zyzniewski@gmail.com> wrote:
> > Related bugzilla entry: http://bugs.openembedded.org/show_bug.cgi?id=5510
> > ---
> >  recipes/gnu-config/files/gnu-configize.in |    4 ++--
> >  1 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/recipes/gnu-config/files/gnu-configize.in b/recipes/gnu-config/files/gnu-configize.in
> > index be8580c..adf9c0e 100755
> > --- a/recipes/gnu-config/files/gnu-configize.in
> > +++ b/recipes/gnu-config/files/gnu-configize.in
> > @@ -1,7 +1,7 @@
> > -#! /usr/bin/perl -w
> > +#! /usr/bin/env perl
> >  # -*- perl -*-
> >
> > -eval 'case $# in 0) exec /usr/bin/perl -S "$0";; *) exec /usr/bin/perl -S "$0" "$@";; esac'
> > +eval 'case $# in 0) exec perl -S "$0";; *) exec perl -S "$0" "$@";; esac'
> >     if 0;
> >
> >  # gnu-configize - install the GNU config.guess / config.sub in a directory tree
> > --
> > 1.7.1
> >
> 
> Looks ok however signoff it properly and resend. See
> http://openembedded.org/index.php/How_to_submit_a_patch_to_OpenEmbedded

And please rebase it on top of master, as first line was changed few
days ago, so this won't apply anymore.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH] gnu-config: removed absolute perl binary paths from gnu-configize.

[Filip Zyzniewski]

On 2/6/11, Martin Jansa <martin.jansa@gmail.com> wrote:

> And please rebase it on top of master, as first line was changed few
> days ago, so this won't apply anymore.

Well first line is fixed now and the others are if 0'ed anyway, so I
don't think there is a point in applying it anymore.

I've marked this patch in patchwork as superseded.

bye,
Filip