* [Buildroot] [PATCH] Fix xenomai build with Linaro toolchain
@ 2012-12-13 22:06 Thierry Bultel
2012-12-13 22:54 ` Arnout Vandecappelle
0 siblings, 1 reply; 2+ messages in thread
From: Thierry Bultel @ 2012-12-13 22:06 UTC (permalink / raw)
To: buildroot
This fixes the build with toolchains that have -mthumb by default
(Not supported by xenomai yet)
Signed-off-by: thierry.bultel at wanadoo.fr
--- xenomai.mk 2012-12-13 22:48:05.841982812 +0100
+++ xenomai.mk.orig 2012-12-13 23:01:13.473982051 +0100
@@ -24,6 +23,12 @@
XENOMAI_CONF_OPT += --includedir=/usr/include/xenomai/
+XENOMAI_CONF_ENV = \
+ CFLAGS="$(TARGET_CFLAGS) -marm"
+
+XENOMAI_MAKE_ENV = \
+ CFLAGS="$(TARGET_CFLAGS) -marm"
+
ifeq ($(BR2_HAVE_DEVFILES),)
define XENOMAI_REMOVE_DEVFILES
for i in xeno-config xeno-info wrap-link.sh ; do \
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Buildroot] [PATCH] Fix xenomai build with Linaro toolchain
2012-12-13 22:06 [Buildroot] [PATCH] Fix xenomai build with Linaro toolchain Thierry Bultel
@ 2012-12-13 22:54 ` Arnout Vandecappelle
0 siblings, 0 replies; 2+ messages in thread
From: Arnout Vandecappelle @ 2012-12-13 22:54 UTC (permalink / raw)
To: buildroot
On 13/12/12 23:06, Thierry Bultel wrote:
> This fixes the build with toolchains that have -mthumb by default
> (Not supported by xenomai yet)
>
>
> Signed-off-by: thierry.bultel at wanadoo.fr
> --- xenomai.mk 2012-12-13 22:48:05.841982812 +0100
> +++ xenomai.mk.orig 2012-12-13 23:01:13.473982051 +0100
This won't apply with git am. Could you create the patch with
git send-email?
> @@ -24,6 +23,12 @@
>
> XENOMAI_CONF_OPT += --includedir=/usr/include/xenomai/
>
> +XENOMAI_CONF_ENV = \
> + CFLAGS="$(TARGET_CFLAGS) -marm"
Wouldn't it be a good idea to make this conditional on BR2_arm? :-)
I would also use XENOMAI_CONF_ENV +=, easier for the future.
> +
> +XENOMAI_MAKE_ENV = \
> + CFLAGS="$(TARGET_CFLAGS) -marm"
Is this needed? Should be enough to do it in the CONF_ENV because
the CFLAGS are directly replaced in the Makefiles...
Regards,
Arnout
> +
> ifeq ($(BR2_HAVE_DEVFILES),)
> define XENOMAI_REMOVE_DEVFILES
> for i in xeno-config xeno-info wrap-link.sh ; do \
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-12-13 22:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-13 22:06 [Buildroot] [PATCH] Fix xenomai build with Linaro toolchain Thierry Bultel
2012-12-13 22:54 ` Arnout Vandecappelle
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.