It looks like some of the m4 files incorrectly implement the AC_ARG_WITH when 
there is no argument.  Looking at the autoconf manual the 4th argument should 
be the block to run if there is no action set (e.g. no --with-xml  on the 
configure).  This was included inside of the 3rd argument previously and this 
was the part that was failing on the builds, because it was not being executed.

The second problem was using AC_MSG_ERROR instead of AC_MSG_WARN for the xml2 
check and it would fail/stop if xml2 was not found.

autoconf section 15.2:
http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/External-Software.html#External-Software

-Bryan

On Sep 7, 2013, at 8:25 PM, James Peach <jpe...@apache.org> wrote:

> On Sep 7, 2013, at 5:00 PM, Nick Kew <n...@apache.org> wrote:
> 
>> 
>> On 7 Sep 2013, at 01:35, Nick Kew wrote:
>> 
>>>> Something in these commits breaks the builds completely. I have both expat 
>>>> and libxml2 installed (with dev packages), and it doesn't detect either. 
>>>> Same problem (I think) on all build bots (they all fail as far as I can 
>>>> tell?).
>> 
>> Somewhat to my surprise, I've failed to find any platform where
>> the build doesn't work out-of-the-box[1].
> 
> Bryan made 2 commits to this yesterday. Maybe that's the complete fix?
> 
> At any rate, the build farm is healthy now :)
> 
>> It also correctly
>> finds and uses expat when I manually move the libxml2
>> headers so configure failed to find them.
>> 
>> Are you working from a clean repo, did you run autoreconf,
>> could there be something lurking from outside /trunk/?
>> If not, what platform, and how is your libxml2 laid out?
>> 
>> I'd be reluctant to commit a lot of extra debug to trunk!
>> 
>> [1] Except for needing to feed it a working PCRE path on Mac.
>> 
>> -- 
>> Nick Kew
> 

Reply via email to