Re: PATCH: Ticket#99: Support for Default Privileges - Mailing list pgadmin-hackers

From Ashesh Vashi
Subject Re: PATCH: Ticket#99: Support for Default Privileges
Date
Msg-id n2u8cf965d31004050452qa1af3a9dvc8ca65ac592126ed@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: Ticket#99: Support for Default Privileges  (Guillaume Lelarge <guillaume@lelarge.info>)
Responses Re: PATCH: Ticket#99: Support for Default Privileges  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Re: PATCH: Ticket#99: Support for Default Privileges  (Guillaume Lelarge <guillaume@lelarge.info>)
List 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



On Mon, Apr 5, 2010 at 5:17 PM, Guillaume Lelarge <guillaume@lelarge.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.


--

pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: Missing non-login roles in properties dialog
Next
From: Ashesh Vashi
Date:
Subject: Re: PATCH: Ticket#99: Support for Default Privileges