Merge lp:~smartgpx/bzr-gtk/workaround_for_lp494140 into lp:bzr-gtk/gtk2

[David Roberts]

[David Roberts (smartgpx)]

This demonstrates a workaround (not truely a 'solution') to lp:494140. With this patch, all gtk tools can be run under xandros, a debian-etch based OS supplied by asus.

It is not clear whether any vital functionality is lost by this patch: the author does not have access to a recent debian install for comparison purposes.

Rejection of this merge proposal in favour of a solution understood and supported by the developers will be a satisfactory outcome.

[Vincent Ladeuil (vila)]

With your patch, brz viz issues a warning:

Traceback (most recent call last):
  File "/home/vila/.bazaar/plugins/gtk/viz/branchwin.py", line 416, in _treeselection_changed_cb
    self.prev_button.set_menu(prev_menu)
AttributeError: 'gtk.ToolButton' object has no attribute 'set_menu'

I added some getattr() calls so that the code is still active for PyGtk-2.10, can you merge it and check it still works for you ?

The branch is: lp:~vila/bzr-gtk/494140-xandros-compat

You can push your branch again when you've got a working version.

review: Needs Fixing

[lp:~smartgpx/bzr-gtk/workaround_for_lp494140]

673. By David Roberts on 2009-12-09

Further consequential changes for PyGtk-2.8

[David Roberts (smartgpx)]

Thank you for your attempt to 'mentor' by giving me the opportunity to
explore this.

I thought it was beyond my gtk/python skills at present. But I've got
a proposal.

My thinking is as follows. 'self.prev_button.set_menu' expects self to
be a MenuToolButton, which we should have established by calling
set_tool_item_type.
But for PyGtk < 2.10 we haven't done that.

But to avoid the runtime error we can at least inspect whether we can
call set_menu
on the button we actually have...

And we need to do this twice, for prev_button and next_button.

Seems to run Visualize cleanly now.

Did I merge your changes correctly and following correct etiquette/attribution?

2009/12/9 Vincent Ladeuil <email address hidden>:
> Review: Needs Fixing
> With your patch, brz viz issues a warning:
>
> Traceback (most recent call last):
>  File "/home/vila/.bazaar/plugins/gtk/viz/branchwin.py", line 416, in _treeselection_changed_cb
>    self.prev_button.set_menu(prev_menu)
> AttributeError: 'gtk.ToolButton' object has no attribute 'set_menu'
>
> I added some getattr() calls so that the code is still active for PyGtk-2.10, can you merge it and check it still works for you ?
>
> The branch is: lp:~vila/bzr-gtk/494140-xandros-compat
>
> You can push your branch again when you've got a working version.
> --
> https://code.edge.launchpad.net/~smartgpx/bzr-gtk/workaround_for_lp494140/+merge/15855
> You are the owner of lp:~smartgpx/bzr-gtk/workaround_for_lp494140.
>

[Vincent Ladeuil (vila)]

> Thank you for your attempt to 'mentor' by giving me the opportunity to
> explore this.
>
> I thought it was beyond my gtk/python skills at present. But I've got
> a proposal.

Hehe, obviously you're too humble here since you demonstrate your understanding of the issue below :)

>
> My thinking is as follows. 'self.prev_button.set_menu' expects self to
> be a MenuToolButton, which we should have established by calling
> set_tool_item_type.
> But for PyGtk < 2.10 we haven't done that.

But if you don't encounter the issue, does it matter ?
I mentioned the warning only to give you feedback about why my changes were needed.
If the behavior is fine on your side with them, no need to do *more* changes :)

>
> But to avoid the runtime error we can at least inspect whether we can
> call set_menu
> on the button we actually have...

That's sound, but not needed because we did that earlier.

>
> And we need to do this twice, for prev_button and next_button.
>
> Seems to run Visualize cleanly now.
>

Good. AIUI you may not have the history associated with the 'next' and 'previous' buttons, but that's the price to pay for running an older PyGtk version I'm afraid.

> Did I merge your changes correctly and following correct
> etiquette/attribution?

I have no idea since you didn't push :)

The diff below is still your original one and the associated branch is still at revno 672.

[Vincent Ladeuil (vila)]

@David: Any progress to report ?
Can you at least push your current branch here so we continue the discussion ?

review: Needs Information

=== modified file 'revisionview.py'
--- revisionview.py	2009-06-10 18:07:35 +0000
+++ revisionview.py	2009-12-11 09:04:12 +0000
@@ -51,13 +51,15 @@
             webbrowser._tryorder.insert(0, '%s "%%s"' % cmd)
     webbrowser.open(uri)
 
-gtk.link_button_set_uri_hook(_open_link)
+if getattr(gtk, 'link_button_set_uri_hook', None) is not None:
+    # Not available before PyGtk-2.10
+    gtk.link_button_set_uri_hook(_open_link)
 
 class BugsTab(gtk.VBox):
 
     def __init__(self):
         super(BugsTab, self).__init__(False, 6)
-    
+
         table = gtk.Table(rows=2, columns=2)
 
         table.set_row_spacings(6)

=== modified file 'viz/branchwin.py'
--- viz/branchwin.py	2009-05-25 15:57:23 +0000
+++ viz/branchwin.py	2009-12-11 09:04:12 +0000
@@ -1,11 +1,10 @@
-# -*- coding: UTF-8 -*-
 """Branch window.
 
 This module contains the code to manage the branch information window,
 which contains both the revision graph and details panes.
 """
 
-__copyright__ = "Copyright © 2005 Canonical Ltd."
+__copyright__ = "Copyright (c) 2005 Canonical Ltd."
 __author__    = "Scott James Remnant <[e-mail]>"
 
 
@@ -80,7 +79,9 @@
         self.accel_group = gtk.AccelGroup()
         self.add_accel_group(self.accel_group)
 
-        gtk.Action.set_tool_item_type(gtk.MenuToolButton)
+        if getattr(gtk.Action, 'set_tool_item_type', None) is not None:
+            # Not available before PyGtk-2.10
+            gtk.Action.set_tool_item_type(gtk.MenuToolButton)
 
         self.prev_rev_action = gtk.Action("prev-rev", "_Previous Revision", "Go to the previous revision", gtk.STOCK_GO_DOWN)
         self.prev_rev_action.set_accel_path("<viz>/Go/Previous Revision")
@@ -413,7 +414,8 @@
                 self.prev_rev_action.set_sensitive(False)
                 prev_menu.hide()
 
-            self.prev_button.set_menu(prev_menu)
+            if getattr(self.prev_button, 'set_menu', None) is not None:
+		self.prev_button.set_menu(prev_menu)
 
             next_menu = gtk.Menu()
             if len(children) > 0:
@@ -433,7 +435,8 @@
                 self.next_rev_action.set_sensitive(False)
                 next_menu.hide()
 
-            self.next_button.set_menu(next_menu)
+            if getattr(self.next_button, 'set_menu', None) is not None:
+		self.next_button.set_menu(next_menu)
 
             self.revisionview.set_revision(revision)
             self.revisionview.set_children(children)