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

Re: svn commit: r10037 - in trunk: build/generator build/win32 subversion/bindings/swig

From: Branko Čibej <brane_at_xbc.nu>
Date: 2004-06-21 00:46:59 CEST

rey4@tigris.org wrote:

>Author: rey4
>Date: Sun Jun 20 14:24:10 2004
>New Revision: 10037
>
>Removed:
> trunk/build/win32/gen_swig_runtime.py
>Modified:
> trunk/build/generator/gen_msvc_dsp.py
> trunk/build/generator/gen_vcnet_vcproj.py
> trunk/build/generator/gen_win.py
> trunk/build/generator/msvc_dsp.ezt
> trunk/subversion/bindings/swig/INSTALL
>Log:
>On windows, generate swig runtime libraries by invoking swig instead
>of cobbling together internal swig source files. This change simplifies
>the build process but breaks compatibility with SWIG 1.3.19 and earlier
>on windows.
>
>
So now you took all the work that was done in order to support both
versions of SWIG on Windows (like we do on Unix) out of the window.
Also, whoever wants to build both 1.0.x and trunk must now have _two_
versions of SWIG installed. All that without discussion on the dev@ list.

-1

There's no reason to drop this compatibility, and I strongly object to
the way it was done (i.e., throwing away all parametrisation of SWIG
options). If generating the SWIG runtime is a problem (which I don't see
why it would be), then let's instead tell people to build the SWIG
runtime libraries themselves -- swigwin provides the necessary .dsp files

Please revert this change until we've discussed it and you convince me
it's necessary at this point.

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 21 00:47:32 2004

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.