Re: [PATCH] Building apr-util with recent libexpat2 using CMake

2017-10-05 Thread William A Rowe Jr
On Thu, Oct 5, 2017 at 5:09 AM, Bert Huijben wrote: > Both Serf and Subversion default to using the shared library version of > apr, but have a flag to switch to using the static library version. > > > > Subversion uses the expat apis directly, and both serf and subversion >

RE: [PATCH] Building apr-util with recent libexpat2 using CMake

2017-10-05 Thread Bert Huijben
<b...@qqmail.nl> Cc: APR Developer List <dev@apr.apache.org> Subject: RE: [PATCH] Building apr-util with recent libexpat2 using CMake Question, Is their default build using libapr-1.lib or apr-1.lib for svn and for serf? I'm trying to wire in a choice for expat.lib (a

RE: [PATCH] Building apr-util with recent libexpat2 using CMake

2017-10-04 Thread William A Rowe Jr
lliam A Rowe Jr [mailto:wr...@rowe-clan.net] > > Sent: woensdag 4 oktober 2017 18:14 > > To: Bert Huijben <b...@qqmail.nl> > > Cc: APR Developer List <dev@apr.apache.org> > > Subject: Re: [PATCH] Building apr-util with recent libexpat2 using CMake > &

RE: [PATCH] Building apr-util with recent libexpat2 using CMake

2017-10-04 Thread Bert Huijben
> -Original Message- > From: William A Rowe Jr [mailto:wr...@rowe-clan.net] > Sent: woensdag 4 oktober 2017 18:14 > To: Bert Huijben <b...@qqmail.nl> > Cc: APR Developer List <dev@apr.apache.org> > Subject: Re: [PATCH] Building apr-util with recent libexpat

Re: [PATCH] Building apr-util with recent libexpat2 using CMake

2017-10-04 Thread William A Rowe Jr
On Wed, Oct 4, 2017 at 10:23 AM, Bert Huijben wrote: > Hi, > > Building apr-util with a recent libexpat2 (2.2.4) requires referencing an > additional .c file on Windows. The attached patch fixes the CMake build. > Perhaps somebody more experienced with CMake can create a

[PATCH] Building apr-util with recent libexpat2 using CMake

2017-10-04 Thread Bert Huijben
Hi, Building apr-util with a recent libexpat2 (2.2.4) requires referencing an additional .c file on Windows. The attached patch fixes the CMake build. Perhaps somebody more experienced with CMake can create a patch that would be compatible with older and newer versions. Note that