Re: Patch for supporting CFLAGS & Co env vars at configure time

Duncan Grisby <[email protected]> Mon, 29 Dec 2008 14:45:21 +0000
Newsgroups gmane.comp.corba.omniorb.devel
Message-ID <[email protected]>
On Tuesday 2 December, "Floris Bruynooghe" wrote:

> However recently I discovered that making the default environment
> variables work is actually really trivial.  This patch does so, it
> only misses CXXCPP since the build system doesn't seem to have that
> separate currently.  I've tried this patch with gcc builds on a
> variety of Solaris, Debian and RHEL machines and so far haven't seen
> any problems with it.
> 
> The reason the patch clears out the CDEBUGFLAGS and CXXDEBUGFLAGS is
> because @CFLAGS@ and @CXXFLAGS@ seem to contain "-g -O2" by default
> (but as soon as you supply a CFLAGS yourself that one overwrites the
> default as you would expect).  If you think this will be a problem I'm
> happy to look if this can be changed.

Thanks for the patch. I've tweaked it a bit to restore the original
behaviour of using -O2 by default rather than "-g -O2". I've checked the
change in to CVS.

Cheers,

Duncan.

-- 
 -- Duncan Grisby         --
  -- [email protected]     --
   -- http://www.grisby.org --