From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Olly Betts <olly(at)survex(dot)com>, pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Compilation on Debian SID |
Date: | 2011-12-04 09:37:06 |
Message-ID: | 1322991426.2311.14.camel@localhost.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Mon, 2011-11-28 at 17:40 +0000, Dave Page wrote:
> On Sat, Nov 26, 2011 at 11:05 PM, Olly Betts <olly(at)survex(dot)com> wrote:
> > On Sat, Nov 26, 2011 at 08:18:31PM +0000, Dave Page wrote:
> >>
> > This isn't original wx code - a number of projects define equivalent
> > macros (and often with the same names, which is where the problems lie
> > when they do it in API headers and you want to use two or more such
> > projects at once).
> >
> > So just find one with a licence that's compatible:
>
> The irony is that on closer examination we don't actually need the
> macro at all, as in a Unicode build of wxWidgets (pgAdmin requires
> Unicode support), it gets defined away to nothing. So, the patch ends
> up looking like the attached.
>
> Guillaume, please give it a quick check in case I missed anything.
>
Trying it now...
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2011-12-04 12:44:10 | pgAdmin III commit: Fix compilation on Debian |
Previous Message | Ashesh Vashi | 2011-12-02 18:30:59 | Re: PATCH: EXTENSION keyword not supported by pgAdmin3 |