On Wed, Oct 01, 2008 at 06:28:42PM -0400, John Center wrote:
> I had the same problem.  I fixed one line in configure:
> 
> if test "x$debug" = "xyes" ; then
>         CFLAGS="${CFLAGS} -g"
> else
>         CFLAGS="-O"
> fi
> 
> The second if statement was clearing my CFLAGS setting, replacing it 
> with just "-O".  I changed the line from CFLAGS="-O" to 
> CFLAGS="${CFLAGS} -O".  This kept my settings.  I don't know if this is 
> the correct fix, but it worked for me.

Pushed the attached patch which should also solve it,
configure is made from configure.in.

Volker
From 73f9e17e6457be8dfb6c1132cc6ca43ef65c677c Mon Sep 17 00:00:00 2001
From: Volker Lendecke <[EMAIL PROTECTED]>
Date: Thu, 2 Oct 2008 02:17:04 +0200
Subject: [PATCH] Fix bug 5798: "CFLAGS info lost in configure"

Michael, please check and merge to the other branches if it's right.

Thanks,

Volker
---
 source3/configure.in |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/source3/configure.in b/source3/configure.in
index f04ddbe..545a565 100644
--- a/source3/configure.in
+++ b/source3/configure.in
@@ -128,7 +128,7 @@ fi
 if test "x$debug" = "xyes" ; then
 	CFLAGS="${CFLAGS} -g"
 else
-	CFLAGS="-O"
+	CFLAGS="${CFLAGS} -O"
 fi
 
 m4_include(../lib/socket_wrapper/config.m4)
-- 
1.5.5

Attachment: pgpuCL7vatIeS.pgp
Description: PGP signature

-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

Reply via email to