Re: PATCH: Ticket#99: Support for Default Privileges

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: Dave Page <dave(dot)page(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Ticket#99: Support for Default Privileges
Date: 2010-04-05 11:52:26
Message-ID: n2u8cf965d31004050452qa1af3a9dvc8ca65ac592126ed@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

hmm.. That's strange...
It should.. :-(

The log, sent by you, suggests that, they were created on your system..
i.e.
*/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXDEBUG__
-D__WXGTK__ -g -O0 -I/usr/include/libxml2 -I/usr/include/libxml2
-DDATA_DIR=\"/opt/pgadmin-trunk/share/pgadmin3/\" -Wall
-Wno-non-virtual-dtor -fno-strict-aliasing -I../pgadmin/include -MT
ctlDefaultSecurityPanel.o -MD -MP -MF .deps/ctlDefaultSecurityPanel.Tpo -c
-o ctlDefaultSecurityPanel.o `test -f './ctl/ctlDefaultSecurityPanel.cpp' ||
echo './'`./ctl/ctlDefaultSecurityPanel.cpp*
*./ctl/ctlDefaultSecurityPanel.cpp:812: error: ISO C++ does not permit
‘wxPanel::sm_eventTable’ to be defined as
‘ctlDefaultSecurityPanel::sm_eventTable’*
*/opt/wxgtk-2.8/include/wx-2.8/wx/generic/panelg.h:91: error: ‘const
wxEventTableEntry wxPanel::sm_eventTableEntries []’ is private*
*./ctl/ctlDefaultSecurityPanel.cpp:812: error: within this context*

--
Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise Postgres Company<http://www.enterprisedb.com>

On Mon, Apr 5, 2010 at 5:17 PM, Guillaume Lelarge <guillaume(at)lelarge(dot)info>wrote:

> Le 05/04/2010 13:22, Ashesh Vashi a écrit :
> > Hi Guillaume,
> >
> > I am able to compile the patch well on my side.
> > I faced this issue, when I tried to apply the patch, when
> > include/ctl/ctlDefaultPrivileges.h and ctl/ctlDefaultPrivileges.cpp. I
> had
> > to remove these two files before applying the patch.
> >
> > Would you please verify on your side?
> >
>
> I don't have these two files, so I didn't need to remove them. Should
> the patch create them? because they aren't part of it.
>
>
> --
> Guillaume.
> http://www.postgresqlfr.org
> http://dalibo.com
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2010-04-05 11:54:31 Re: PATCH: Ticket#99: Support for Default Privileges
Previous Message Guillaume Lelarge 2010-04-05 11:48:29 Re: Missing non-login roles in properties dialog