From: Lan Yixun (dlan) <dennis.yxun@xxxxxxxxx> according to autoconf manual[1], build script should allow user override the CFLAGS, LDFLAGS varible (passed via ./configure). And actually, we should not directly use CFLAGS, LDFLAGS, but have to create another one (eg. AUTOFS_CFLAGS). FYI, see section "Variable: CFLAGS", the last paragraph. Here, I slightly modify the code so that we can still fallback to the default one. [1] http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Preset-Output-Variables.html#Preset-Output-Variables Edited by: Ian Kent <raven@xxxxxxxxxx> - allow for previous LDFLAGS patch. - rename patch to reflect what's actually done but leave the original description in place as a reminder. Signed-off-by: Lan Yixun (dlan) <dennis.yxun@xxxxxxxxx> --- CHANGELOG | 1 + Makefile.rules | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CHANGELOG b/CHANGELOG index 8f349a2..ec9d7f0 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -64,6 +64,7 @@ - fix dead LDAP symbolic link when LDAP support is disabled. - add missing libtirpc lib to mount_nfs.so when TIRPC enabled. - use compiler determined by configure instead of hard-coded ones. +- remove hard-coded STRIP variable. 25/07/2012 autofs-5.0.7 ======================= diff --git a/Makefile.rules b/Makefile.rules index 9aebd19..d00ba3c 100644 --- a/Makefile.rules +++ b/Makefile.rules @@ -25,16 +25,16 @@ endif ifdef DEBUG CFLAGS ?= -g -Wall -DDEBUG LDFLAGS ?= -g -STRIP = : +STRIP ?= : else ifdef DONTSTRIP CFLAGS ?= -O2 -g LDFLAGS ?= -g -STRIP = : +STRIP ?= : else CFLAGS ?= -O2 -Wall LDFLAGS ?= -s -STRIP = strip --strip-debug +STRIP ?= strip --strip-debug endif endif -- To unsubscribe from this list: send the line "unsubscribe autofs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
