[OE-core] [PATCH] rt-tests: fix dependency on eglibc, clean LDFLAGS passing
Darren Hart
dvhart at linux.intel.com
Tue Mar 20 16:22:37 UTC 2012
On 03/20/2012 09:09 AM, Denys Dmytriyenko wrote:
> On Tue, Mar 20, 2012 at 08:33:14AM -0700, Darren Hart wrote:
>>
>>
>> On 03/20/2012 08:28 AM, Denys Dmytriyenko wrote:
>>> On Tue, Mar 20, 2012 at 08:01:44AM -0700, Darren Hart wrote:
>>>>
>>>>
>>>> On 03/19/2012 05:22 PM, Denys Dmytriyenko wrote:
>>>>> From: Denys Dmytriyenko <denys at ti.com>
>>>>>
>>>>> Replace dependency on eglibc with virtual/libc
>>>>> Pass LDFLAGS to TARGET_CC_ARCH, instead of CFLAGS as Makefile overrides it
>>>>>
>>>>> Signed-off-by: Denys Dmytriyenko <denys at ti.com>
>>>>
>>>> Thanks Denys,
>>>>
>>>> Looking through the rt-tests sources I don't see TARGET_CC_ARCH
>>>> anywhere. Have you confirmed that changes made there make their way into
>>>> the actual build?
>>>
>>> Darren,
>>>
>>> TARGET_CC_ARCH is not an rt-tests specific variable, it's used in OE.
>>> Although, TARGET_LDFLAGS might be more appropriate for this purpose,
>>> historically TARGET_CC_ARCH was heavily used to pass distro LDFLAGS to the
>>> app's build process - grepping it would show many such cases in OE-Core and
>>> especially in meta-oe...
>>
>> OK, thanks for the context. Will you be resubmitting with TARGET_LDFLAGS?
>
> Nope, my bad, it's actually the other way around. From bitbake.conf:
>
> export LDFLAGS = "${TARGET_LDFLAGS}"
>
> So, appending to TARGET_CC_ARCH is the simplest way to pass flags, as that's
> embedded into $CC
>
> Another option would have been to call make with -e flag to let environment
> variables override the ones in the Makefile, but that's less controlled...
OK, from the context you've provided, the TARGET_CC_ARCH seems like the
most expedient option. Ultimately a patch to rt-tests to allow users to
provide LDFLAGS seems like the best approach. I can talk to Clark W.
about this.
So you have my ack:
Acked-by: Darren Hart <dvhart at linux.intel.com>
--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
More information about the Openembedded-core
mailing list