From: | Jasmin Dizdarevic <jasmin(dot)dizdarevic(at)gmail(dot)com> |
---|---|
To: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: pgAdmin III commit: Pull keywords dynamically from PostgreSQL headers |
Date: | 2011-01-06 16:34:26 |
Message-ID: | AANLkTinmFUjCrArAxMkenkduUD-znn4xh+=sq3Lo3djk@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
since this patch is applied, i can't build
pgAdmin. %pgdir%\include\server\parser doesn't contain a gram.h file.
What I'm doing wrong?
Ty
Jasmin
2011/1/5 Magnus Hagander <magnus(at)hagander(dot)net>
> On Wed, Jan 5, 2011 at 06:05, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>wrote:
>
>>
>>
>> On Wed, Jan 5, 2011 at 12:37 AM, Magnus Hagander <magnus(at)hagander(dot)net>wrote:
>>
>>> On Tue, Jan 4, 2011 at 18:46, Ashesh Vashi <
>>> ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Magnus/Dave,
>>>>
>>>> This check-in has added one compilation error.
>>>> The following line introduce an error in file '*pgadmin/db/keywords.c*'
>>>> *#include <server/parser/gram.h>
>>>> #include <server/parser/kwlist.h>*
>>>>
>>>> Because - these header files does not always find under *`pg_config
>>>> --includedir`* folder, but can be found under *`pg_config
>>>> --pkgincludedir`*.
>>>>
>>>> Please find the patch to resolve the issue. This applies on both master
>>>> and REL-1_12_PATCHES branch.
>>>>
>>>>
>>> Looks good in general, but do we actually need to put it on 1.12? The
>>> keyword build changes were only applied to master, so unless we actually
>>> *need* it on the back branch, I don't think we should put it there. Is it
>>> needed for something else as well?
>>>
>> You're right.
>> This patch should only be applied on master.
>>
>
> Thanks, confirmed and applied.
>
>
> --
> Magnus Hagander
> Me: http://www.hagander.net/
> Work: http://www.redpill-linpro.com/
>
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2011-01-06 16:38:08 | Re: pgAdmin III commit: Pull keywords dynamically from PostgreSQL headers |
Previous Message | Dave Page | 2011-01-06 14:53:40 | Re: Ticket 266: Unable to backup table when quotation marks used in the name |