Re: phase 2 of wxWidgets 2.9 build

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Peter Geoghegan <peter(dot)geoghegan86(at)gmail(dot)com>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: phase 2 of wxWidgets 2.9 build
Date: 2011-02-03 10:15:31
Message-ID: AANLkTimNfovhA1m+TEi61xxzcTFQ4fBGfYX-AAwt2Yb3@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Thu, Feb 3, 2011 at 3:00 AM, Peter Geoghegan
<peter(dot)geoghegan86(at)gmail(dot)com> wrote:
>
> I'm going to clean up your commit from the other day per your
> suggestion. I'm also going to get started on moving OGL into the tree,
> and building it ourselves.

OK.

> I've noticed that the sample OGL applications have obvious bugs in
> them, probably because of my cursory fixes to get them to work against
> 2.9. I wasn't too worried about it at the time. Should I be now? I'll
> certainly want to clean up numerous OGL compiler warnings.

We won't be including the sample apps in our tree. Probably just
headers in pgadmin/include/ogl and required source in pgadmin/ogl

> I'm seeing some errors like this:
>
> An error has occurred:
>
> XRC error: file:/home/peter/pgadmin3/pgadmin/ui/dlgTable.xrc:6:
> unknown style flag "wxRESIZE_BOX"
>
> ...and...
>
> An error has occurred:
>
> XRC error: file:/home/peter/pgadmin3/pgadmin/ui/frmHint.xrc:6: unknown
> style flag "wxTHICK_FRAME"

Hmm, interesting. Is that coming from wxrc, or when building xrcDialogs.cpp?

> Any suggestions on a good place to attack to remove the probable AUI
> related weirdness? Sometimes the "Object browser" header (i.e. where
> it says object browser) isn't visible. the other panes seem unaffected
> though. Object browser seems well behaved when docked anywhere other
> than its default position.

My first suggestion would be to select the "Default View" option, and
see if that helps. Failing that, comment out the code that sets either
the default or the stored perspective, and see what happens if you let
the controls figure out their own default layout.

> There's this too, which I see when I start the SQL editor:
>
> Program received signal SIGSEGV, Segmentation fault.
> 0x0000000000460063 in wxWindowBase::GetSize (this=0xd9d9abab8686)
>    at /usr/local/include/wx-2.9/wx/window.h:321
> 321             DoGetSize(& w, & h);
> (gdb) bt
> #0  0x0000000000460063 in wxWindowBase::GetSize (this=0xd9d9abab8686)
>    at /usr/local/include/wx-2.9/wx/window.h:321
> #1  0x00000000008024b1 in gqbSplitter::onVerticalSashResize (this=0x18bbb40,
>    event=...) at ./gqb/gqbController.cpp:596
> #2  0x00007ffff577349d in
> wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&,
> wxEvent&) const () from /usr/local/lib/libwx_baseu-2.9.so.1
> #3  0x00007ffff598dcf1 in
> wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&,
> wxEvtHandler*, wxEvent&) ()
>   from /usr/local/lib/libwx_baseu-2.9.so.1
> #4  0x00007ffff598de53 in wxEventHashTable::HandleEvent(wxEvent&,
> wxEvtHandler*) () from /usr/local/lib/libwx_baseu-2.9.so.1
> #5  0x00007ffff598e1e3 in wxEvtHandler::ProcessEventLocally(wxEvent&) ()
>   from /usr/local/lib/libwx_baseu-2.9.so.1
> #6  0x00007ffff598e25d in wxEvtHandler::ProcessEvent(wxEvent&) ()
>   from /usr/local/lib/libwx_baseu-2.9.so.1
> #7  0x00007ffff65fb611 in wxSplitterWindow::DoSendEvent(wxSplitterEvent&) ()
>   from /usr/local/lib/libwx_gtk2u_core-2.9.so.1
> #8  0x00007ffff65fbce9 in wxSplitterWindow::SetSashPositionAndNotify(int) ()
>   from /usr/local/lib/libwx_gtk2u_core-2.9.so.1
> #9  0x00007ffff65fbeb9 in wxSplitterWindow::OnSize(wxSizeEvent&) ()
>   from /usr/local/lib/libwx_gtk2u_core-2.9.so.1

Trying to get the size of something before it's drawn perhaps?

> I see this, which prevents me from calling the edit table dialog:
>
> An error has occurred:
>
> XRC error: file:/home/peter/pgadmin3/pgadmin/ui/dlgTable.xrc:330: too
> many children in grid sizer: 36 > 3 x 11 (consider omitting the number
> of rows or columns)

Probably a simple counting error. I suspect theres just one row too many.

> I expect to fix quite a few bugs tomorrow. My feeling is that we're
> past the worst of it though. I'd like to close phase 2 by having a
> pgAdmin III that builds against 2.9 with no obvious defects, but
> doesn't yet have OGL in the tree. We'll then have something to commit
> before moving on to phase 3, bringing OGL into the pgAdmin tree and
> building it ourselves. Thoughts?

Sounds like a plan!

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

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2011-02-03 10:22:02 pgAdmin III commit: Fix a regression in my previous patch, which fixed
Previous Message Peter Geoghegan 2011-02-03 03:00:25 Re: phase 2 of wxWidgets 2.9 build