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

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

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Fri, 21 Dec 2012 17:03:55 +0200

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, 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
>
Received on 2012-12-21 16:04:34 CET

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