Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: McClintock Matthew-B29882 <B29882@freescale.com>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/6] Fix perl for powerpc64 build
Date: Fri, 07 Oct 2011 16:52:53 +0100	[thread overview]
Message-ID: <1318002781.8533.16.camel@ted> (raw)
In-Reply-To: <CAEsOVNdUTaVc1NR0hDH7PU1gVCXotJWEJquiz=EMQXQ231GmzA@mail.gmail.com>

On Wed, 2011-10-05 at 19:04 +0000, McClintock Matthew-B29882 wrote:
> On Wed, Oct 5, 2011 at 6:46 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >> --- a/meta/recipes-devtools/perl/perl_5.12.3.bb
> >> +++ b/meta/recipes-devtools/perl/perl_5.12.3.bb
> >> @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = "file://Copying;md5=2b4c6ffbcfcbdee469f02565f253d81a \
> >>  # We need gnugrep (for -I)
> >>  DEPENDS = "virtual/db grep-native"
> >>  DEPENDS += "gdbm zlib"
> >> -PR = "r5"
> >> +PR = "r6"
> >>
> >>  # 5.10.1 has Module::Build built-in
> >>  PROVIDES += "libmodule-build-perl"
> >> @@ -95,6 +95,8 @@ export LDDLFLAGS = "${LDFLAGS} -shared"
> >>  # We're almost Debian, aren't we?
> >>  CFLAGS += "-DDEBIAN"
> >>
> >> +export ARCH_powerpc64 = "powerpc64"
> >> +
> >
> > I'm finding it really hard to believe this makes any difference. I just
> > tried a powerpc64 build here and perl built fine. I can't find any
> > evidence for any part of the perl build using an ARCH variable
> > (TARGET_ARCH is used but that isn't ARCH).
> >
> > Can you give some further information about how/where this is used?
> 
> I'm trying to reproduce this again on 'edison' as well and it does not
> appear to be occurring anymore! The problem before had to do with a
> configuration script error:
> 
> config.sh-powerpc64-linux
> 
> This script was getting invoked as 'config.sh-powerpc-linux' on the
> 64bit build, then the file did not exist and the configure step failed
> out. Not sure if this was somehow fixed in a different manner.
> 
> Please drop this patch for now. I'll bring it up again if I see issues.

Do you still need this bit:

        case "${TARGET_ARCH}" in
-               x86_64 | powerpc | s390)
+               x86_64 | powerpc | powerpc64 | s390)
                        sed -i -e "s,\(need_va_copy=\)'undef',\1'define',g" \
                                config.sh-${TARGET_ARCH}-${TARGET_OS}
                        ;;

?

Cheers,

Richard




  reply	other threads:[~2011-10-07 15:58 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-04 22:03 [PATCH 1/6] Fix mdadm for powerpc64 Matthew McClintock
2011-10-04 22:03 ` [PATCH 2/6] Fix perl for powerpc64 build Matthew McClintock
2011-10-05 11:46   ` Richard Purdie
2011-10-05 19:04     ` McClintock Matthew-B29882
2011-10-07 15:52       ` Richard Purdie [this message]
2011-10-07 16:07         ` McClintock Matthew-B29882
2011-10-04 22:03 ` [PATCH 3/6] Add new IMAGE_CLASSES variable for classes for image generation Matthew McClintock
2011-10-05 22:52   ` McClintock Matthew-B29882
2011-10-05 23:53     ` Richard Purdie
2011-10-06  0:25       ` McClintock Matthew-B29882
2011-10-11 16:44         ` McClintock Matthew-B29882
2011-10-04 22:03 ` [PATCH 4/6] scripts/oe-buildenv-internal: Add SOCKS5_{USER, PASSWD} to BB_ENV_EXTRAWHITE Matthew McClintock
2011-10-05 13:25   ` Richard Purdie
2011-10-04 22:03 ` [PATCH 5/6] Fix sysprof for powerpc64 Matthew McClintock
2011-10-05  0:05   ` Khem Raj
2011-10-05  6:44     ` McClintock Matthew-B29882
2011-10-05 11:35     ` Richard Purdie
2011-10-05 16:46       ` Khem Raj
2011-10-05 16:48       ` McClintock Matthew-B29882
2011-10-05 17:06         ` Khem Raj
2011-10-05 17:10           ` Saul Wold
2011-10-05 18:30       ` Phil Blundell
2011-10-05 18:41         ` McClintock Matthew-B29882
2011-10-05 19:13         ` Khem Raj
2011-10-04 22:03 ` [PATCH 6/6] Bring libnl up to date with meta-openebedded Matthew McClintock

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1318002781.8533.16.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=B29882@freescale.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox