Second try at build system changes [was: svn commit: r1424922]

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,
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, brane_at_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