[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Maintainer mode and debug flags

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: 2007-04-19 22:55:34 CEST

Hyrum K. Wright wrote:
> C. Michael Pilato wrote:
>> Hyrum K. Wright wrote:
>>> While attempting a build without debugs flags earlier today, I noticed
>>> that the vanilla configure includes '-g -O2' in CFLAGS. This is kind of
>>> pointless, because of the fact that debugging with optimizations is
>>> painful at best, futile at worst. r24655 was a somewhat misguided
>>> effort to fix that problem, and quickly managed to break the
>>> buildslaves. It was reverted in r24666.
>>> So, my proposal is thus:
>>> 1) Currently configure complains if '--disable-debug' is given with
>>> '--enable-maintainer-mode' because the second implies
>>> '--enable-debug'. After some discussion on IRC, dlr proposed the
>>> following:
>>> I think '--enable-maintainer-mode' '--disable-debug' should define
>>> SVN_DEBUG, but turn off -g.
>>> I agree, and propose it be implemented as such.
>>> 2) Debug symbols, that is including '-g' in CFLAGS, should be off by
>>> default, and only enabled when requested by '--enable-debug' or
>>> '--enable-maintainer-mode'. We may have had this debate before, but
>>> what is the reason for a vanilla './configure' including '-g'?
>> You don't explicitly address the -O2 complaint. What's your plan?
>> (I think --enable-maintainer-mode should turn off optimizations.)
> Agreed. Especially since it implies --enable-debug, and hence -g.

I believe the attached patch implements these changes. If someone could
double-check my autoconf, I'll commit this.


* configure.in
  Disable debug symbols by default, unless requested by '--enable-debug'
  or '--enable-maintainer-mode'. Also, disable optimizations when using
  maintainer mode.

Received on Thu Apr 19 22:55:46 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.