Re: [PATCH] Update build instructions for Win32 platform.

Mark Phippard wrote:

Thanks for the comments, very helpful...

> 1) When Python is installed it has to be added to the PATH.

Added that to instructions.

> 2) I had problems building Apache relating to mod_ssl and mod_deflate.

Error in instructions, I intended to not build them, although my latest INSTALL instructions do add
zlib to apache and it works.

> 3) I tried a lot of different versions of ZLIB, some with source, some
> without, and never could get mod_deflate to build.

It appears to work with zlib 1.1.4.

> 4) For mod_ssl, the make file needed some GNU utilities like bison, I

Now not built, as you say it's not needed anyway.

> 5) There is a newer version of OpenSSL. Instructions should point to it,

I now use 0.9.7d which should be OK.

> 6) There are many newer versions of ZLIB. I couldn't get any of them to
> build with Apache, but worked fine with SVN.

I got both to build with 1.1.4

> 7) If OpenSSL is to be used with mod_ssl it has to be unpacked into an

Fixed the instructions.

> 8) Your "xcopy" instructions for APR folders left off the "/S" switch.

Fixed, thanks.

> 9) Your instructions had the user create an SVN repository for no obvious
> reason why. This became a little confusing later when running the tests.

Mistake, now removed.

> 10) Finally, your instructions for running the tests gives an invalid
> URL. It should just be http://localhost
Fixed.

I am going to submit two patches, one for INSTALL on 1.0.x and another for the trunk as they are so
different.

-- 
Martin Tomes
echo 'Martin x Tomes at controls x eurotherm x co x uk'\
  | sed -e 's/ x /\./g' -e 's/ at /@/'
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org