You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Sander Striker <st...@apache.org> on 2002/04/23 09:39:47 UTC

Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

> From: Justin Erenkrantz [mailto:jerenkrantz@apache.org]
> Sent: 23 April 2002 09:14

> On Tue, Apr 23, 2002 at 02:01:36AM -0500, jerenkrantz@tigris.org wrote:
> > Author: jerenkrantz
> > Date: 2002-04-23 07:01 GMT
> > New Revision: 1750
> > 
> > Removed:
> >    trunk/expat-lite/
> > Log:
> > Remove expat-lite from the repository since we now rely upon apr-util to
> > find a suitable expat.
> > 
> > (This is also known as "does 'svn rm' on directories correctly" commit.)
> 
> Okay, I removed expat-lite from the SVN repository.
> 
> Let's see what damage this causes.  =)  

Errm.  neon doesn't compile anymore in-tree if you don't have an
expat/libxml installed(me! ;).  Like we discussed on irc, neon doesn't support
the newer expat that comes with apr-util.

I've done some looking around in neons config files (macros/neon-xml-parser.m4),
but I have the feeling it is quite a change to get to work with both old
and new expat.  Since I don't really want to dive into that I'm going to
leave this for people with more energy and karma in the config dept.

Sander


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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Greg Stein <gs...@lyra.org>.
On Thu, Apr 25, 2002 at 12:12:03AM +0200, Branko Äibej wrote:
> Marcus Comstedt wrote:
> >Branko ÄOibej <br...@xbc.nu> writes:
> >
> >>You need automake installed, aclocal is part of automake.
> >
> >Thanks but no thanks.  automake is 3vil.

You bet. We actually used automake *way* back when. I got pissed off at it
enough, that we removed it in favor of our current system.

> I admit I don't know why neon uses aclocal. But if you want to try my 
> neon patch, that's what you need.

Yah... Joe should just sinclude() the files in the macros/ directory, rather
than using aclocal. I believe the only reason for the aclocal is those
macros.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Marcus Comstedt <ma...@mc.pp.se>.
Branko Čibej <br...@xbc.nu> writes:

> I admit I don't know why neon uses aclocal. But if you want to try my
> neon patch, that's what you need.

Well, Joe was really quick in releasing a neon 0.19.4, so in fact I
have tried your patch now (without defiling my system with automake :)
and it does indeed work.  Thanks.


  // Marcus



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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Branko Čibej <br...@xbc.nu>.
Marcus Comstedt wrote:

>Branko ÄŒibej <br...@xbc.nu> writes:
>
>>You need automake installed, aclocal is part of automake.
>>
>
>
>Thanks but no thanks.  automake is 3vil.
>

I admit I don't know why neon uses aclocal. But if you want to try my 
neon patch, that's what you need.

-- 
Brane Čibej   <br...@xbc.nu>   http://www.xbc.nu/brane/




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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Marcus Comstedt <ma...@mc.pp.se>.
Branko Čibej <br...@xbc.nu> writes:

> You need automake installed, aclocal is part of automake.


Thanks but no thanks.  automake is 3vil.


  // Marcus



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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Branko Čibej <br...@xbc.nu>.
Marcus Comstedt wrote:

>brane@xbc.nu writes:
>
>>Can you try the following patch to neon/macros/neon-xml-parser.m4:
>>
>>[...]
>>
>
>For some reason your patch was rejected by gpatch.  I've applied it by
>hand, but...
>
>
>>You'll have to run neon's autogen.sh script to regenerate configure,
>>and you should update subversion's ac-helpers/neon.m4, too.
>>
>
>Neon's autogen.sh script doesn't work.
>
>pelix:~/subversion/neon% ./autogen.sh
>-n aclocal...
>./autogen.sh: aclocal: not found
>pelix:~/subversion/neon% 
>

You need automake installed, aclocal is part of automake.

-- 
Brane Čibej   <br...@xbc.nu>   http://www.xbc.nu/brane/




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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Marcus Comstedt <ma...@mc.pp.se>.
brane@xbc.nu writes:

> Can you try the following patch to neon/macros/neon-xml-parser.m4:
> 
> [...]

For some reason your patch was rejected by gpatch.  I've applied it by
hand, but...


> You'll have to run neon's autogen.sh script to regenerate configure,
> and you should update subversion's ac-helpers/neon.m4, too.

Neon's autogen.sh script doesn't work.

pelix:~/subversion/neon% ./autogen.sh
-n aclocal...
./autogen.sh: aclocal: not found
pelix:~/subversion/neon% 


  // Marcus



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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Branko Čibej <br...@xbc.nu>.
Justin Erenkrantz wrote:

>On Wed, Apr 24, 2002 at 05:59:59PM +0200, brane@xbc.nu wrote:
>
>>Can you try the following patch to neon/macros/neon-xml-parser.m4:
>>
>
>Indeed, I have a similar patch in my tree, but my svn is completely
>corrupt.  I can't get it to checkout or update anything.  I will
>try to build from a fresh tree later tonight.
>
>I would also suggest changing the ac-helpers/neon.m4 check to
>look for expat.h instead of libexpat.la.  That should make the
>bundled expat check a bit better.  -- justin
>
I commited such a change, just before sending in my patch. :-)

-- 
Brane Čibej   <br...@xbc.nu>   http://www.xbc.nu/brane/




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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Justin Erenkrantz <je...@apache.org>.
On Wed, Apr 24, 2002 at 05:59:59PM +0200, brane@xbc.nu wrote:
> Can you try the following patch to neon/macros/neon-xml-parser.m4:

Indeed, I have a similar patch in my tree, but my svn is completely
corrupt.  I can't get it to checkout or update anything.  I will
try to build from a fresh tree later tonight.

I would also suggest changing the ac-helpers/neon.m4 check to
look for expat.h instead of libexpat.la.  That should make the
bundled expat check a bit better.  -- justin

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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by br...@xbc.nu.
Quoting Marcus Comstedt <ma...@mc.pp.se>:

> 
> "Sander Striker" <st...@apache.org> writes:
> 
> > Errm.  neon doesn't compile anymore in-tree if you don't have an
> > expat/libxml installed(me! ;).  Like we discussed on irc, neon doesn't
> support
> > the newer expat that comes with apr-util.
> > 
> > I've done some looking around in neons config files
> (macros/neon-xml-parser.m4),
> > but I have the feeling it is quite a change to get to work with both
> old
> > and new expat.  Since I don't really want to dive into that I'm going
> to
> > leave this for people with more energy and karma in the config dept.
> 
> I don't suppose there is a newer neon that supports the newer expat?
> I can't build subversion any longer either.  :-6
> 
> configure: error: no XML parser was found: expat or libxml>=1.8.3
> required
> configure failed for neon
> 
> 
>   // Marcus

Can you try the following patch to neon/macros/neon-xml-parser.m4:

--- neon-0.19.3-old/macros/neon-xml-parser.m4   Sun Jan  6 22:35:35 2002
+++ neon-0.19.3/macros/neon-xml-parser.m4       Wed Apr 24 17:33:40 2002
@@ -35,13 +35,21 @@
 # FIXME: needs to take second arg for VPATH builds

 AC_DEFUN([NEON_XML_EXTERNAL_EXPAT], [
+ne_found_hdr=no
 case $neon_expat_dir in
 */libexpat.la)
        neon_using_libtool_expat=yes
        neon_expat_dir=`echo $neon_expat_dir | sed 's:/libexpat.la$::'`
 esac
 if test -r "$neon_expat_dir/xmlparse.h"; then
+       ne_found_hdr=yes
        AC_DEFINE(HAVE_XMLPARSE_H, 1, [Define if you have xmlparse.h])
+fi
+if test -r "$neon_expat_dir/expat.h"; then
+       ne_found_hdr=yes
+       AC_DEFINE(HAVE_EXPAT_H, 1, [Define if you have expat.h])
+fi
+if test "$ne_found_hdr" = "yes"; then
        AC_DEFINE(HAVE_EXPAT, 1, [Define if you have expat])
        CFLAGS="$CFLAGS -I$neon_expat_dir"
        dnl add the library (if it isn't a libtool library)



You'll have to run neon's autogen.sh script to regenerate configure,
and you should update subversion's ac-helpers/neon.m4, too.

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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Marcus Comstedt <ma...@mc.pp.se>.
"Sander Striker" <st...@apache.org> writes:

> Errm.  neon doesn't compile anymore in-tree if you don't have an
> expat/libxml installed(me! ;).  Like we discussed on irc, neon doesn't support
> the newer expat that comes with apr-util.
> 
> I've done some looking around in neons config files (macros/neon-xml-parser.m4),
> but I have the feeling it is quite a change to get to work with both old
> and new expat.  Since I don't really want to dive into that I'm going to
> leave this for people with more energy and karma in the config dept.

I don't suppose there is a newer neon that supports the newer expat?
I can't build subversion any longer either.  :-6

configure: error: no XML parser was found: expat or libxml>=1.8.3 required
configure failed for neon


  // Marcus



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

Re: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Branko Čibej <br...@xbc.nu>.
Sander Striker wrote:

>>From: Sander Striker [mailto:striker@apache.org]
>>Sent: 23 April 2002 11:40
>>
>
>>>From: Justin Erenkrantz [mailto:jerenkrantz@apache.org]
>>>Sent: 23 April 2002 09:14
>>>
>>>On Tue, Apr 23, 2002 at 02:01:36AM -0500, jerenkrantz@tigris.org wrote:
>>>
>>>>Author: jerenkrantz
>>>>Date: 2002-04-23 07:01 GMT
>>>>New Revision: 1750
>>>>
>>>>Removed:
>>>>   trunk/expat-lite/
>>>>Log:
>>>>Remove expat-lite from the repository since we now rely upon apr-util to
>>>>find a suitable expat.
>>>>
>>>>(This is also known as "does 'svn rm' on directories correctly" commit.)
>>>>
>>>Okay, I removed expat-lite from the SVN repository.
>>>
>>>Let's see what damage this causes.  =)  
>>>
>>Errm.  neon doesn't compile anymore in-tree if you don't have an
>>expat/libxml installed(me! ;).  Like we discussed on irc, neon doesn't support
>>the newer expat that comes with apr-util.
>>
>>I've done some looking around in neons config files (macros/neon-xml-parser.m4),
>>but I have the feeling it is quite a change to get to work with both old
>>and new expat.  Since I don't really want to dive into that I'm going to
>>leave this for people with more energy and karma in the config dept.
>>
>
>I should mention that the windows build works just fine... ;)
>

/me pats self on back


-- 
Brane Čibej   <br...@xbc.nu>   http://www.xbc.nu/brane/




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

RE: Removed expat-lite, WAS: RE: svn commit: rev 1750 - trunk

Posted by Sander Striker <st...@apache.org>.
> From: Sander Striker [mailto:striker@apache.org]
> Sent: 23 April 2002 11:40

> > From: Justin Erenkrantz [mailto:jerenkrantz@apache.org]
> > Sent: 23 April 2002 09:14
> 
> > On Tue, Apr 23, 2002 at 02:01:36AM -0500, jerenkrantz@tigris.org wrote:
> > > Author: jerenkrantz
> > > Date: 2002-04-23 07:01 GMT
> > > New Revision: 1750
> > > 
> > > Removed:
> > >    trunk/expat-lite/
> > > Log:
> > > Remove expat-lite from the repository since we now rely upon apr-util to
> > > find a suitable expat.
> > > 
> > > (This is also known as "does 'svn rm' on directories correctly" commit.)
> > 
> > Okay, I removed expat-lite from the SVN repository.
> > 
> > Let's see what damage this causes.  =)  
> 
> Errm.  neon doesn't compile anymore in-tree if you don't have an
> expat/libxml installed(me! ;).  Like we discussed on irc, neon doesn't support
> the newer expat that comes with apr-util.
> 
> I've done some looking around in neons config files (macros/neon-xml-parser.m4),
> but I have the feeling it is quite a change to get to work with both old
> and new expat.  Since I don't really want to dive into that I'm going to
> leave this for people with more energy and karma in the config dept.

I should mention that the windows build works just fine... ;)

Sander


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