All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.