Re: pgAdmin Event Trigger Compatibility - Mailing list pgadmin-hackers

H

i Dave.

Thanks for your inputs. I have built this patch on Linux and found one minor issue. I am not sure this is a known behavior or problem in building.

Hence, i would like to discuss this with our team once and will update this thread with the new patch.

Thanks in advance.

Dinesh

-- 
Dinesh Kumar

Software Engineer

Ph: +918087463317

Skype ID: dinesh.kumar432

www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more

On Tue, Jul 2, 2013 at 9:34 PM, Dave Page <dpage@pgadmin.org> wrote:

Hi

On Fri, Jun 28, 2013 at 4:55 PM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:

Hi Dave,

Please find the attached patch file for the pgAdmin's event trigger compatibility. This patch has been build on the pgAdmin's master branch.

I would like to request you to share inputs and suggestions on this patch.

I'm guessing you only tested this on Windows? Please update the GNU build system and check it works properly on Linux or Mac too. At minimum you need to fix:

In file included from dlg/dlgProperty.cpp:63:

../pgadmin/include/schema/pgEventTrigger.h:48: error: extra qualification ‘pgEventTrigger::’ on member ‘IsUpToDate’ 

and 

Undefined symbols for architecture i386:

  "enabledisableEventTriggerFactory::enabledisableEventTriggerFactory(menuFactoryList*, wxMenu*, ctlMenuToolbar*)", referenced from:

      frmMain::CreateMenus()      in frmMain.o

  "_eventTriggerFactory", referenced from:

      pgDatabase::ShowTreeDetail(ctlTree*, frmMain*, ctlListView*, ctlSQLBox*)     in pgDatabase.o

ld: symbol(s) not found for architecture i386

(which almost certainly occurs because the new  files haven't been added to the makefile fragments).

Thanks.

(and as a side-note - please don't CC me on pgAdmin patch submissions; I don't use my EDB address on the mailing lists. Thanks.)

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company