[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: Blair Zajac <blair_at_orcaware.com>
Date: 2002-08-28 01:37:57 CEST

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
Received on Wed Aug 28 01:38:32 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.