Branko Čibej wrote on Fri, Dec 21, 2012 at 15:03:15 +0100: > You probably noticed that I made a bunch of build system modifications > on a branch. The individual changes are described in the BRANCH-README > file (see below), with "svn diff" invocations that make it easier to > review each change. > > Please take time to review these changes and test them. I did my best to > test on Mac OS and Linux, but as we saw in the previous iteration,
Are you able to test FreeBSD too? on e.g. people.apache.org or subversion.zones.apache.org. Daniel > autoconf isn't exactly obvious. As usual, I can't build swig-db and > swig-py, but did touch their configury. > > I'd eventually like to come to the point where we can, e.g., build the > sqlite3wrapper.c part of libsvn_subr with warnings disabled, as we > disable them for Swig-generated files. > > -- Brane > > > On 21.12.2012 14:56, br...@apache.org wrote: > > Author: brane > > Date: Fri Dec 21 13:56:53 2012 > > New Revision: 1424922 > > > > URL: http://svn.apache.org/viewvc?rev=1424922&view=rev > > Log: > > [On the tweak-build-take-two branch] > > > > * BRANCH-README: New file, describe all changes with diffs. > > > > Added: > > subversion/branches/tweak-build-take-two/BRANCH-README > > > > Added: subversion/branches/tweak-build-take-two/BRANCH-README > > URL: > > http://svn.apache.org/viewvc/subversion/branches/tweak-build-take-two/BRANCH-README?rev=1424922&view=auto > > ============================================================================== > > --- subversion/branches/tweak-build-take-two/BRANCH-README (added) > > +++ subversion/branches/tweak-build-take-two/BRANCH-README Fri Dec 21 > > 13:56:53 2012 > > @@ -0,0 +1,39 @@ > > +This branch is for minor build system improvements. > > + > > +Changes on this branch: > > + > > + ** Split standards-compliance mode and maintainer mode compiler flags > > + out of CFLAGS, so that compilation command lines that do not > > + generate (too many) warnings or are not forced to comply with ISO > > + C '90 can be constructed without having to resort to stripping > > + individual flags out of CFLAGS. > > + > > + $ svn diff -r1424288:1424822 \ > > + ^/subversion/branches/tweak-build-take-two/configure.ac \ > > + ^/subversion/branches/tweak-build-take-two/aclocal.m4 \ > > + > > ^/subversion/branches/tweak-build-take-two/build/ac-macros/compiler.m4 \ > > + ^/subversion/branches/tweak-build-take-two/Makefile.in > > + > > + ** Now that warning and standards-compliance mode macros are no > > + longer part of CFLAGS, stop stripping them in the Swig wrapper > > + configury. > > + > > + $ svn diff -c1424330 \ > > + ^/subversion/branches/tweak-build-take-two/build/ac-macros/swig.m4 > > + > > + ** Allow optimization and debugging to coexist, including in > > + maintainer mode, adding a new configure option > > + --enable-optimize. Neither --enable-optimize nor --enable-debug > > + will override any optimization or debugging flags set by the user > > + in C(XX)FLAGS at configure time. If debugging and optimization are > > + enabled at the same time, we will try to use -O1, then -O; if > > + debuggin is not enabled, we will try -O2 first. > > + > > + $ svn diff -c1424860 \ > > + ^/subversion/branches/tweak-build-take-two/configure.ac > > + > > + ** Remove an obsolete autoconf macro that was not used anywhere and > > + is superceded with SVN_CFLAGS_ADD_IFELSE. > > + > > + $ svn diff -c1424297 \ > > + > > ^/subversion/branches/tweak-build-take-two/build/ac-macros/svn-macros.m4 > > > -- > Branko Čibej > Director of Subversion | WANdisco | www.wandisco.com >
