On Fri, 6 Sep 2013 14:51:56 -0600
Leif Hedstrom <zw...@apache.org> wrote:

> On Sep 6, 2013, at 4:06 AM, n...@apache.org wrote:
> 
> > Make libxml2 the configuration default XML parser
> > (so it should get a bit of test-driving).
> > 
> > Separate micro-commit so it's an easy distinction we can confine
> > to trunk even if the new code makes it into a release as an option
> > before it's well-tested.
> 
> 
> 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?).

Heh.  I guess I should've left the expat default, at least
for a while.  If you specify
  --with-xml=expat
it'll call the old expat-detection code, which I haven't changed.

But if you don't specify any --with-xml option, it should try
both before giving that error.  If it doesn't, it's a bug in my M4.

> checking for existence of /usr/lib64/tclConfig.sh... loading
> checking "For XML parser"... configure: error: An XML parser (expat or 
> libxml2) is required.

Damn.  Guess we could use some temporary debug so you get
a trace of what it tried.

I should find time to revisit it over the weekend.

-- 
Nick Kew

Reply via email to