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.co <http://www.enterprisedb.com/>m<http://www.enterprisedb.com/> * Follow us on Twitter* @EnterpriseDB Visit EnterpriseDB for tutorials, webinars, whitepapers<http://www.enterprisedb.com/resources-community> and more <http://www.enterprisedb.com/resources-community> On Tue, Jul 2, 2013 at 9:34 PM, Dave Page <[email protected]> wrote: > Hi > > > On Fri, Jun 28, 2013 at 4:55 PM, Dinesh Kumar < > [email protected]> 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 >
