From ff4e8ab52186b25065109228e795138e04d03225 Mon Sep 17 00:00:00 2001 From: Steven Luo Date: Mon, 22 Feb 2010 02:06:00 -0800 Subject: [PATCH] Enable building with -mthumb for ARM by default The Diablo autobuilder passes an invalid DEB_BUILD_OPTIONS (options should be space-separated, not comma-separated), and the Fremantle autobuilder doesn't pass "thumb" in DEB_BUILD_OPTIONS at all, so the existing code was an effective no-op. We know we want Thumb instructions by default, so build with -mthumb by default and offer a "nothumb" option to turn it off. --- debian/rules | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/debian/rules b/debian/rules index e467af0..e5c9873 100755 --- a/debian/rules +++ b/debian/rules @@ -12,8 +12,8 @@ # Optimize for N8x0 CPUs if building for armel ifeq ($(DEB_HOST_ARCH),armel) EXTRA_CFLAGS += -mcpu=arm1136jf-s - # Use Thumb for smaller binaries if requested - ifneq (,$(filter thumb,$(DEB_BUILD_OPTIONS))) + # Use Thumb for smaller binaries unless otherwise requested + ifeq (,$(filter nothumb,$(DEB_BUILD_OPTIONS))) EXTRA_CFLAGS += -mthumb endif endif -- 1.7.9.5