Re: BUG #7485: 9.2 beta3 libxml2 can't be loaded on Windows

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Rikard Pavelic <rikard(dot)pavelic(at)zg(dot)htnet(dot)hr>, Craig Ringer <ringerc(at)ringerc(dot)id(dot)au>, pgsql-bugs <pgsql-bugs(at)postgresql(dot)org>, Sachin Srivastava <sachin(dot)srivastava(at)enterprisedb(dot)com>, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
Subject: Re: BUG #7485: 9.2 beta3 libxml2 can't be loaded on Windows
Date: 2012-08-16 17:46:33
Message-ID: 4121.1345139193@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Dave Page <dpage(at)pgadmin(dot)org> writes:
> On Thu, Aug 16, 2012 at 3:28 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> On Thu, Aug 16, 2012 at 3:26 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>> Seems pretty brute-force. Why not just
>>>
>>> #if LIBXML_VERSION > 20703
>>> #define HAVE_XMLSTRUCTUREDERRORCONTEXT
>>> #endif
>>>
>>> in some suitable place (probably xml.c itself, since I don't think we
>>> want to #include xmlversion.h in global headers)?
>>>
>>> Or actually, maybe we could just unconditionally define
>>> HAVE_XMLSTRUCTUREDERRORCONTEXT in pg_config.h.win32. Is anybody
>>> likely to still be building PG with ancient libxml on Windows?

>> That works for me. Using older libxml's would be a bad idea anyway -
>> and there are precompiled binaries available, so it's not hard to get.

> Works as in, I'm happy with it. I haven't tested. Though, it'll likely
> break some of the old BF animals (temporarily, I hope).

On the whole plan A sounds safer (ie look at LIBXML_VERSION in place of
using a configure test). I'll work on that tonight or tomorrow, if
nobody beats me to it.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Bruce Momjian 2012-08-16 22:16:46 Re: 'pg_ctl restart' confused about pathname to postgresql.conf
Previous Message Kevin Grittner 2012-08-16 16:46:30 Re: Re-2: BUG #7495: chosen wrong index