hi miller.

On 2010-07-12 18:22, Miller Puckette wrote:

i guess you'll need to add all the Makefile.am's right now (they are scattered over various places in ./src). i'll probably will have a look at that and send you a changeset

did you receive my "[PATCH] only add window-bindings after it has been fully initialized to pdwindow"? there's a fix included in that.

it turned out to be a race-condition, between the initialization of Pd and Pd-gui.

another (simpler, better) fix can be found in the attachment: it's really only checking whether the menu is there before trying to disable entries within the menubar)

anyhow, here is the backtrace: <snip> invalid command name ".menubar.file" invalid command name ".menubar.file" while executing "$menubar.file entryconfigure [_ "Save"] -state disabled" (procedure "::pd_menus::configure_for_pdwindow" line 6) invoked from within "::pd_menus::configure_for_pdwindow " (procedure "::pd_bindings::window_focusin" line 9) invoked from within "::pd_bindings::window_focusin .pdwindow" (command bound to event) </snip>

fgmasdr IOhannes

diff --git a/tcl/pd_menus.tcl b/tcl/pd_menus.tcl index 7157c5e..65f3174 100644 --- a/tcl/pd_menus.tcl +++ b/tcl/pd_menus.tcl @@ -50,6 +50,7 @@ proc ::pd_menus::create_menubar {} {

proc ::pd_menus::configure_for_pdwindow {} { variable menubar + if { ![winfo exists $menubar] } { return } # these are meaningless for the Pd window, so disable them # File menu $menubar.file entryconfigure [_ "Save"] -state disabled