Re: pgsql: Blind attempt at fixing the non-MSVC Windows builds

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Andres Freund <andres(at)2ndquadrant(dot)com>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Blind attempt at fixing the non-MSVC Windows builds
Date: 2013-02-22 15:34:19
Message-ID: 20130222153419.GH9507@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Andres Freund wrote:
> On 2013-02-22 14:55:06 +0000, Alvaro Herrera wrote:
> > Blind attempt at fixing the non-MSVC Windows builds
> >
> > Apparently, they need -DBUILDING_DLL for the Assert() declarations to
> > work correctly.
>
> More specifically, they need that do access any PGDLLIMPORT symbols. Of
> which
> extern PGDLLIMPORT bool assert_enabled;
> is one. So that seems like the appropriate fix.

Yeah, I'm not really sure if this is okay for both build environments or
is only needed in the backend. Right now there should be no difference
because we're not using relpath.c in frontend programs; but if we add
pg_xlogdump there might be further problems.

--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2013-02-22 15:38:28 Re: pgsql: Blind attempt at fixing the non-MSVC Windows builds
Previous Message Andres Freund 2013-02-22 14:59:37 Re: pgsql: Blind attempt at fixing the non-MSVC Windows builds