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

This patch is based on the patch, you sent me to work on..
(Ticket99_Imcomplete_v1.patch)
That's why - I told you, if those files present in your directory
structure.. :-(

--
Thanks & Regards,

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

On Mon, Apr 5, 2010 at 5:22 PM, Ashesh Vashi
<ashesh(dot)vashi(at)enterprisedb(dot)com>wrote:

> 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
>>
>
>