[OE-core] [PATCH 5/6] Fix sysprof for powerpc64

Khem Raj raj.khem at gmail.com
Wed Oct 5 19:13:56 UTC 2011


On Wed, Oct 5, 2011 at 11:30 AM, Phil Blundell <philb at gnu.org> wrote:
> On Wed, 2011-10-05 at 12:35 +0100, Richard Purdie wrote:
>> On Tue, 2011-10-04 at 17:05 -0700, Khem Raj wrote:
>> > On 10/4/2011 3:03 PM, Matthew McClintock wrote:
>> > > sysprof will not build properly without this defined
>> > >
>> > > Signed-off-by: Matthew McClintock<msm at freescale.com>
>> > > ---
>> > > No comments on original patch sent
>> > >
>> > >   meta/recipes-kernel/sysprof/sysprof_git.bb |    2 ++
>> > >   1 files changed, 2 insertions(+), 0 deletions(-)
>> > >
>> > > diff --git a/meta/recipes-kernel/sysprof/sysprof_git.bb b/meta/recipes-kernel/sysprof/sysprof_git.bb
>> > > index 10bde04..271b5d8 100644
>> > > --- a/meta/recipes-kernel/sysprof/sysprof_git.bb
>> > > +++ b/meta/recipes-kernel/sysprof/sysprof_git.bb
>> > > @@ -16,6 +16,8 @@ SRC_URI_append_arm  = " file://rmb-arm.patch"
>> > >   SRC_URI_append_mips = " file://rmb-mips.patch"
>> > >   SRC_URI_append_powerpc = " file://ppc-macro-fix.patch"
>> > >
>> > > +export CFLAGS_append_powerpc64 = " -D__ppc64__"
>> > > +
>> >
>> > this is a gcc built-in define I wonder why you need to add it explicitly
>>
>> Are you sure? Is the built-in define not __powerpc64__ ?
>
> Yes, __powerpc64__ is the generic one.  I think __ppc64__ is only a
> predefine on Darwin.

__powerpc64__ is not generic. Its Linux specific.

>
> p.
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>




More information about the Openembedded-core mailing list