Re: [PATCH] fix for a missing include

Momchil Velikov <velco@fadata.bg> writes:

> Hi there,
>
> I have a system with a libxmltok installed (Debian package
> libxmlrok1-dev), so configure detects it ans set SVN_HAVE_OLD_EXPAT.
> Unfortunately, this macro is not picked up by svn_xml.h, which tries
> to include the nonexistent expat.h.
>
> ~velco
> Index: svn_xml.h
> ===================================================================
> --- svn_xml.h (revision 3672)
> +++ svn_xml.h (working copy)
> @@ -24,6 +24,8 @@
> #include <apr_pools.h>
> #include <apr_hash.h>
>
> +#include "svn_private_config.h"
> +
> #ifdef SVN_HAVE_OLD_EXPAT
> #include "xmlparse.h"
> #else

I think we need a patch that adds SVN_HAVE_OLD_EXPAT to the CPPFLAGS
used to build Subversion. The patch should also modify the svn-config
script so that svn-config --cppflags includes SVN_HAVE_OLD_EXPAT to
support compiling from the installed Subversion with the correct
flags.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org