On Tue, 16 Mar 2010 12:51:35 +0000, Nick wrote:
On Mon, 15 Mar 2010 14:06:29 +0000 eht16@users.sourceforge.net wrote:
Revision: 4763 http://geany.svn.sourceforge.net/geany/?rev=4763&view=rev Author: eht16 Date: 2010-03-15 14:06:29 +0000 (Mon, 15 Mar 2010)
Log Message:
Fix/Improve GIT repository detection (patch by Thomas Martitz, thanks).
...
if svn_rev != '-1': print_message(conf, 'Compiling Subversion revision', svn_rev)
conf.env.append_value('CCFLAGS', '-g
-DGEANY_DEBUG'.split())
conf.env.append_value('CCFLAGS', '-g -O0
-DGEANY_DEBUG'.split())
Do we really want to force -O0 for wscript SVN builds?
Nah, that was a funny mistake :). I committed the other changes of the revision 4763 and while waiting for the commit to finish, I added -O0 to test something and then accidentally hit Ctrl-S to save too early and so this change got committed as well by Subversion.
I'll revert it, also in the geany-plugins' wscript where it was since a long time though I'm not sure why I added it, it definitely makes no sense to add compiler-specific compile flags to the build system. Sorry.
Also, as I don't know Waf well, does it only support building with gcc?
No, it also supports icc and mscc, though I never tested those.
Regards, Enrico