Re: [PATCH] cpan.bbclass: Fix CCFLAGS.

From: Pascal Ouyang <xin.ouyang@windriver.com>
To: Khem Raj <raj.khem@gmail.com>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] cpan.bbclass: Fix CCFLAGS.
Date: Fri, 6 Jul 2012 13:33:20 +0800	[thread overview]
Message-ID: <4FF678A0.4040200@windriver.com> (raw)
In-Reply-To: <CAMKF1sqqiaXyxxzrreU2+00mmdCAmtpSNXnrXVgF1F5a4b07_g@mail.gmail.com>

On 2012年07月05日 22:57, Khem Raj wrote:
> On Thu, Jul 5, 2012 at 4:17 AM, Pascal Ouyang <xin.ouyang@windriver.com> wrote:
>>
>> So, I still don't know what is your opinion of this patch.
>
> I think we still need the CC and CFLAGS to appear in CCFLAGS since
> they sometimes decide ABI etc. that we use and without that
> the modules will be incompatible. So may be
> appending to CCFLAGS would work instead
>

Exactly in my patch, this just appends CFLAGS to CCFLAGS.

@@ -26,13 +26,14 @@ cpan_do_configure () {
  			test -f $f2 || continue
  			sed -i -e "s:\(PERL_ARCHLIB = \).*:\1${PERL_ARCHLIB}:" \
  				-e 's/perl.real/perl/' \
+				-e "s/^\(CCFLAGS =.*\)/\1 ${CFLAGS}/" \
  				$f2
  		done
  	fi
  }

Thanks.
-- 
- Pascal




