[PATCH 0/1] gnu-config make sure we use perl-native

[PATCH 0/1] gnu-config make sure we use perl-native

[Saul Wold]

From: Saul Wold <sgw@linux.intel.com>

During a world build I was getting a perl version mismatch between
the perl-native and the host perl. This change ensures that 
perl-native is used always.

Pull URL: git://git.pokylinux.org/poky-contrib.git
  Branch: sgw/gnu-config
  Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=sgw/gnu-config

Thanks,
    Saul Wold <sgw@linux.intel.com>
---


Saul Wold (1):
  gnu-config: Ensure that we use perl-native to avoid version
    mis-matches

 .../gnu-config/gnu-config_20080123.bb              |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

[PATCH 1/1] gnu-config: Ensure that we use perl-native to avoid version mis-matches

[Saul Wold]

From: Saul Wold <sgw@linux.intel.com>

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 .../gnu-config/gnu-config_20080123.bb              |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/gnu-config/gnu-config_20080123.bb b/meta/recipes-devtools/gnu-config/gnu-config_20080123.bb
index 2629ecb..af34cee 100644
--- a/meta/recipes-devtools/gnu-config/gnu-config_20080123.bb
+++ b/meta/recipes-devtools/gnu-config/gnu-config_20080123.bb
@@ -26,9 +26,7 @@ do_install () {
 		sed -e 's,@gnu-configdir@,${datadir}/gnu-config,g' \
 		    -e 's,@autom4te_perllibdir@,${datadir}/autoconf,g' > ${D}${bindir}/gnu-configize
 	# In the native case we want the system perl as perl-native can't have built yet
-	if [ "${BUILD_ARCH}" != "${TARGET_ARCH}" ]; then
-		sed -i -e 's,/usr/bin/perl,${bindir}/perl,g' ${D}${bindir}/gnu-configize
-	fi
+	sed -i -e 's,/usr/bin/perl,${bindir}/perl,g' ${D}${bindir}/gnu-configize
 	chmod 755 ${D}${bindir}/gnu-configize
 	install -m 0644 config.guess config.sub ${D}${datadir}/gnu-config/
 }
-- 
1.7.1

Re: [PATCH 0/1] gnu-config make sure we use perl-native

[Saul Wold]

On 03/11/2011 05:15 PM, Saul Wold wrote:
> From: Saul Wold<sgw@linux.intel.com>
>
> During a world build I was getting a perl version mismatch between
> the perl-native and the host perl. This change ensures that
> perl-native is used always.
>

Never mind this patch, Saul's brain is not functioning well today,
I completely missed the comment!

Back to the drawing board on why the miss-match is occuring in Config.pm 
then.

Sau!
> Pull URL: git://git.pokylinux.org/poky-contrib.git
>    Branch: sgw/gnu-config
>    Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=sgw/gnu-config
>
> Thanks,
>      Saul Wold<sgw@linux.intel.com>
> ---
>
>
> Saul Wold (1):
>    gnu-config: Ensure that we use perl-native to avoid version
>      mis-matches
>
>   .../gnu-config/gnu-config_20080123.bb              |    4 +---
>   1 files changed, 1 insertions(+), 3 deletions(-)
>

Re: [poky] [PATCH 0/1] gnu-config make sure we use perl-native

[Khem Raj]

On (11/03/11 17:34), Saul Wold wrote:
> On 03/11/2011 05:15 PM, Saul Wold wrote:
> >From: Saul Wold<sgw@linux.intel.com>
> >
> >During a world build I was getting a perl version mismatch between
> >the perl-native and the host perl. This change ensures that
> >perl-native is used always.
> >
> 
> Never mind this patch, Saul's brain is not functioning well today,
> I completely missed the comment!

I discarded my reply :)
> 
> Back to the drawing board on why the miss-match is occuring in
> Config.pm then.

somewhere there is /usr/bin/perl referred may be a cross recipe ?

> 
> Sau!
> >Pull URL: git://git.pokylinux.org/poky-contrib.git
> >   Branch: sgw/gnu-config
> >   Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=sgw/gnu-config
> >
> >Thanks,
> >     Saul Wold<sgw@linux.intel.com>
> >---
> >
> >
> >Saul Wold (1):
> >   gnu-config: Ensure that we use perl-native to avoid version
> >     mis-matches
> >
> >  .../gnu-config/gnu-config_20080123.bb              |    4 +---
> >  1 files changed, 1 insertions(+), 3 deletions(-)
> >
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky

-- 
-Khem

Re: [poky] [PATCH 0/1] gnu-config make sure we use perl-native

[Paul Eggleton]

On Saturday 12 March 2011 02:31:40 Khem Raj wrote:
> > Back to the drawing board on why the miss-match is occuring in
> > Config.pm then.
> 
> somewhere there is /usr/bin/perl referred may be a cross recipe ?

I've hit this issue this morning, and have filed a bug report:

  http://bugzilla.pokylinux.org/show_bug.cgi?id=866

Not sure if it helps but at least it should be easy to reproduce.

Cheers,
Paul

Re: [poky] [PATCH 0/1] gnu-config make sure we use perl-native

[Koen Kooi]

Op 14 mrt 2011, om 12:51 heeft Paul Eggleton het volgende geschreven:

> On Saturday 12 March 2011 02:31:40 Khem Raj wrote:
>>> Back to the drawing board on why the miss-match is occuring in
>>> Config.pm then.
>> 
>> somewhere there is /usr/bin/perl referred may be a cross recipe ?
> 
> I've hit this issue this morning, and have filed a bug report:
> 
>  http://bugzilla.pokylinux.org/show_bug.cgi?id=866
> 
> Not sure if it helps but at least it should be easy to reproduce.

I haven't checked if the OE .dev and OE core gnu-config are similar, but have a look at:

http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=be21179c5321bd0afb9221f020ac12ad75c86a3b
http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=edcdefbf6e0675c1bcc1fc4f464f654223380e50

regards,

Koen

Re: [poky] [PATCH 0/1] gnu-config make sure we use perl-native

[Richard Purdie]

On Mon, 2011-03-14 at 13:18 +0100, Koen Kooi wrote:
> Op 14 mrt 2011, om 12:51 heeft Paul Eggleton het volgende geschreven:
> 
> > On Saturday 12 March 2011 02:31:40 Khem Raj wrote:
> >>> Back to the drawing board on why the miss-match is occuring in
> >>> Config.pm then.
> >> 
> >> somewhere there is /usr/bin/perl referred may be a cross recipe ?
> > 
> > I've hit this issue this morning, and have filed a bug report:
> > 
> >  http://bugzilla.pokylinux.org/show_bug.cgi?id=866
> > 
> > Not sure if it helps but at least it should be easy to reproduce.
> 
> I haven't checked if the OE .dev and OE core gnu-config are similar, but have a look at:
> 
> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=be21179c5321bd0afb9221f020ac12ad75c86a3b
> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=edcdefbf6e0675c1bcc1fc4f464f654223380e50

There are a few related patches to this floating around and I know Tom
did some work in OE.dev to make perl-native an early bootstrap
dependency.

I've been giving this some thought and given the various goals we have,
I think the long term best solution is going to be to place perl into an
isolated bin directory in sysroots, then for anything using really using
perl-native, add that directory to PATH. This is similar to what we now
do with the bootstrap toolchains in OE-Core. This way the dependencies
are deterministic and either perl is or isn't required but we don't
instantly need to build perl-native or run the risk of nasty loops in
the early bootstrap. Anything requiring perl-native would add the
directory to PATH, probably using a tiny class file.

The above patches still would make sense of course.

Cheers,

Richard

Re: [poky] [PATCH 0/1] gnu-config make sure we use perl-native

[Richard Purdie]

On Mon, 2011-03-14 at 13:18 +0100, Koen Kooi wrote:
> Op 14 mrt 2011, om 12:51 heeft Paul Eggleton het volgende geschreven:
> 
> > On Saturday 12 March 2011 02:31:40 Khem Raj wrote:
> >>> Back to the drawing board on why the miss-match is occuring in
> >>> Config.pm then.
> >> 
> >> somewhere there is /usr/bin/perl referred may be a cross recipe ?
> > 
> > I've hit this issue this morning, and have filed a bug report:
> > 
> >  http://bugzilla.pokylinux.org/show_bug.cgi?id=866
> > 
> > Not sure if it helps but at least it should be easy to reproduce.
> 
> I haven't checked if the OE .dev and OE core gnu-config are similar, but have a look at:
> 
> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=be21179c5321bd0afb9221f020ac12ad75c86a3b
> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=edcdefbf6e0675c1bcc1fc4f464f654223380e50
> 

I'm going to merge these into OE.core, thanks.

Richard