Martin Tomes <lists@tomes.org> writes:
> The new instructions are split into sections which describe what is
> needed, how to prepare the files (unpacking, environment etc.) and how
> to actually build the binaries with all available options turned on.
>
> * INSTALL
> Update Win32 build instructions.
Thanks! Some comments:
> Index: INSTALL
> ===================================================================
> --- INSTALL (revision 9468)
> +++ INSTALL (working copy)
> @@ -161,9 +161,9 @@
> 4. Neon library 0.24.5 (http://www.webdav.org/neon/)
>
> The Neon library allows a Subversion client to interact with remote
> - repositories over the Internet via a WebDAV based protocol. If you
> - want to use Subversion to connect to a server over ra_dav (via a
> - http:// or https:// url), you will require Neon.
> + repositories over the Internet via a WebDAV based protocol. If you
> + want to use Subversion to connect to a server over ra_dav (via a
> + http:// or https:// url), you will require Neon.
How did you generate this patch? The hunk above seems to make no
changes -- it replaces text with completely identical text. Strange.
> The Neon library source code can be installed in "./neon" if you
> want Subversion to build it. The source code is included with the
> @@ -578,134 +578,276 @@
> E. Building the Latest Source under Windows
> ----------------------------------------
>
> - (These instructions assume you have already installed Subversion
> - and checked out a Subversion working copy. See the section
> - "Bootstrapping from a Zip or Installer File under Windows" above
> - for more.)
> + E.1 Prerequisites
>
> - There is some support for building Subversion on Win32
> - platforms. The project files generated by gen-make.py are for
> - Microsoft Visual C++ 6.x; earlier versions of the compiler are
> - not supported. You'll also need a fairly recent version of the
> - Microsoft Platform SDK; the header files that come with MSVC 6.x
> - are too old.
> + * Visual Studio 6 with the latest service pack. It can be built
> + with Visual Studio 7 but these instructions assumes VS6.
> + * A recent Windows SDK, the one provided with Visual Studio 6 is
> + too old. You only need the 'Core SDK'. You can get it from MSDN
> + if you have it or from
> + http://www.microsoft.com/msdownload/platformsdk/sdkupdate/ if you
> + don't.
> + * Python 2.2, downloaded from http://www.python.org/ which is
> + used to generate the project files.
> + * Perl 5.8 from http://www.activestate.com/
> + * Neon 0.24.5, downloaded from
> + http://www.webdav.org/neon/neon-0.24.5.tar.gz which is required
> + for building the client components.
> + * Berkeley DB is required for support for local server components,
> + version 4.2.52 is available from
> + http://subversion.tigris.org/servlets/ProjectDocumentList as
> + db-4.2.52-win32.zip. For more information see Section I.5.
> + * Apache 2 source, downloaded from
> + http://httpd.apache.org/download.cgi, these instructions assume
> + version 2.0.49.
> + * Apache 2 msi install file, also from
> + http://httpd.apache.org/download.cgi (required for running the
> + tests)
> + * ZLib 1.1.4 binaries http://www.winimage.com/zLibDll/zlib114dll.zip and
> + sources http://www.gzip.org/zlib/zlib114.zip
> + * openssl-0.9.7d.tar.gz obtained from http://www.openssl.org/source/
> + * An assembler, e.g., nasm which is avaliable from
> + http://www.kernel.org/pub/software/devel/nasm/binaries/win32/
> + * The latest version of apr-util (see Section I.1) must be told
> + where to find the Berkeley DB include files and libraries. You
> + can do this with the w32locatedb.pl script (located in the
> + "build" directory in the apr-util sources), which modifies
> + apr-util's header and DSP files. The instructions at the top of
> + the script file explain how to use the script.
Having a separate Prerequisites section seems like a very good idea.
Most of the version requirements should say " or higher", I think.
For example:
* Neon 0.24.5 or higher. Download from
http://www.webdav.org/neon/neon-0.24.5.tar.gz.
Neon is required for building the client components.
(The original document has this problem, independent of your patch, of
course.)
I can't really evaluate the rest of the patch, as I don't build on
Windows myself, but it looks much more complete and detailed. Any
Windows users have any comments?
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Apr 26 18:28:31 2004