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 <[email protected]> >>>>> >>>>> 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 <[email protected]> >>>> >>>> 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... >
How about the follow patch against rt-tests? Clark W. said he's fine taking a patch that allows such an override. Does this meet your needs? >From 9f1944b0751ccb398a47fc4c3652c459aab44f38 Mon Sep 17 00:00:00 2001 Message-Id: <9f1944b0751ccb398a47fc4c3652c459aab44f38.1332268655.git.dar...@dvhart.com> From: Darren Hart <[email protected]> Date: Tue, 20 Mar 2012 11:34:08 -0700 Subject: [PATCH] rt-tests: Support user supplied CFLAGS and LDFLAGS Accept user supplied CFLAGS and LDFLAGS, overwriting the Makefile supplied versions. This can cause the build to fail if the user does not provide at least what the Makefile defines, but so be it. Signed-off-by: Darren Hart <[email protected]> CC: Denys Dmytriyenko <[email protected]> --- Makefile | 25 +++++++++++++------------ 1 files changed, 13 insertions(+), 12 deletions(-) diff --git a/Makefile b/Makefile index 4038dcc..e1edf6c 100644 --- a/Makefile +++ b/Makefile @@ -20,7 +20,8 @@ ifneq ($(filter x86_64 i386 ia64 mips powerpc,$(machinetype)),) NUMA := 1 endif -CFLAGS = -D_GNU_SOURCE -Wall -Wno-nonnull -Isrc/include +CFLAGS ?= -D_GNU_SOURCE -Wall -Wno-nonnull -Isrc/include +LDFLAGS ?= PYLIB := $(shell python -c 'import distutils.sysconfig; print distutils.sysconfig.get_python_lib()') @@ -61,41 +62,41 @@ all: $(TARGETS) hwlatdetect -include $(sources:.c=.d) cyclictest: cyclictest.o rt-utils.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) $(NUMA_LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) $(NUMA_LIBS) signaltest: signaltest.o rt-utils.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) pi_stress: pi_stress.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) hwlatdetect: src/hwlatdetect/hwlatdetect.py chmod +x src/hwlatdetect/hwlatdetect.py ln -s src/hwlatdetect/hwlatdetect.py hwlatdetect rt-migrate-test: rt-migrate-test.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) ptsematest: ptsematest.o rt-utils.o rt-get_cpu.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) sigwaittest: sigwaittest.o rt-utils.o rt-get_cpu.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) svsematest: svsematest.o rt-utils.o rt-get_cpu.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) pmqtest: pmqtest.o rt-utils.o rt-get_cpu.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) sendme: sendme.o rt-utils.o rt-get_cpu.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) $(EXTRA_LIBS) pip_stress: pip_stress.o error.o rt-utils.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) hackbench: hackbench.o - $(CC) $(CFLAGS) -o $@ $^ $(LIBS) + $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS) CLEANUP = $(TARGETS) *.o .depend *.*~ *.orig *.rej rt-tests.spec *.d CLEANUP += $(if $(wildcard .git), ChangeLog) -- 1.7.6.5 -- Darren Hart Intel Open Source Technology Center Yocto Project - Linux Kernel _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
