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
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 |