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

RE: svn commit: rev 3075 - trunk trunk/build

From: McCarty, Andy <William.McCarty_at_med.va.gov>
Date: 2002-08-28 01:56:36 CEST

Works for me. Thanks. I have been waiting for this all day.

> -----Original Message-----
> From: Blair Zajac [mailto:blair@orcaware.com]
> Sent: Tuesday, August 27, 2002 4:38 PM
> To: Branko ?ibej
> Cc: dev@subversion.tigris.org
> Subject: Re: svn commit: rev 3075 - trunk trunk/build
>
>
> Branko ?ibej wrote:
> >
> > >
> > >
> > >+Version 0.14.3 [Alpha Interim 3] (released ?? ??? ????,
> revision ????)
> > >+
> > >+ Developer-visible changes:
> > >+ * upgrade to neon 0.23.2 to fix Windows build issues
> > >+
> > >
> >
> > I don't know about you, but this commit does _not_ fix the
> Windows build
> > issues. For one thing, build_neon.bat has to be upgraded to set
> > EXPAT_FLAGS correcty. But that's trivial -- what's worse is that
> > neon-0.23.2 doesn't even compile on Windows when DAV is enabled:
> >
> > src\ne_socket.c(363) : error C2065: 'EINTR' : undeclared identifier
> > src\ne_socket.c(410) : error C2065: 'EPIPE' : undeclared identifier
> >
>
> I've built and compiled svn with Neon just fine, but then I always
> build with zlib and openssl support. I just tried to build Neon
> without openssl and it failed with the same errors you had.
>
> Try this patch and let me know how it works.
>
> Index: build/win32/build_neon.bat
> ===================================================================
> --- build/win32/build_neon.bat
> +++ build/win32/build_neon.bat 2002-08-27
> 16:36:51.000000000 -0700
> @@ -11,8 +11,14 @@
> @rem * NOTE: The paths should be relative to the Neon
> directory, ..\..\neon
> @rem
> **************************************************************
> ************
>
> -@rem Must set EXPAT_FLAGS, not EXPAT_SRC, to define HAVE_EXPAT_H
> -set EXPAT_FLAGS="/I ../apr-util/xml/expat/lib /D HAVE_EXPAT_H"
> +@rem Must use EXPAT_FLAGS and not EXPAT_SRC to set the
> include location
> +@rem correctly for APR's expat and define HAVE_EXPAT and
> HAVE_EXPAT_H,
> +@rem since Neon's neon.mak is set up to use the pre-compiled Windows
> +@rem binary Expat installation.
> +@rem Also define HAVE_ERRNO_H to work around a bug in 0.23.2 when
> +@rem compiling without OpenSSL, where EINTR and EPIPE are not defined
> +@rem in src/ne_socket.c unless errno.h is included.
> +set EXPAT_FLAGS="/I ../apr-util/xml/expat/lib /D HAVE_EXPAT
> /D HAVE_EXPAT_H /D HAVE_ERRNO_H"
>
> set exitcode=0
>
>
> > Please revert this change. We're not ready to upgrade Neon yet.
>
> The solution to everything is not to revert (this is the second revert
> you've suggested recently)!
>
> Best,
> Blair
>
> --
> Blair Zajac <blair@orcaware.com>
> Web and OS performance plots - http://www.orcaware.com/orca/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Aug 28 13:49:37 2002

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.