Hi Richard and tools-discuss team,

I've downloaded a fresh copy of gpyfm earlier today, and noticed there's
a minor typo in the source, which causes a Python traceback:

Traceback (most recent call last):
  File "/home/keramida/tmp/gpyfm/fm/control.py", line 276, in _txt_delete
    sel.view.mbv.set_action_sensitive('undo', len(self.diff.undo) != 0)
NameError: global name 'sel' is not defined

This is what a copy of gpyfm pulled from:
http://hg.intevation.org/mirrors/opensolaris.org/scm-migration/onnv-scm/file/35c2c7eb22c4/usr/src/tools

The attached patch fixes this:

%%%
--- control.py.orig     2007-07-10 17:02:21.000000000 +0300
+++ control.py  2007-07-10 18:18:09.000000000 +0300
@@ -273,7 +273,7 @@
         txt = ntxt.splitlines(True)
         #print txt
         self.diff.merge_change(txt)
-        sel.view.mbv.set_action_sensitive('undo', len(self.diff.undo) != 0)
+        self.view.mbv.set_action_sensitive('undo', len(self.diff.undo) != 0)
 
     def _txt_insert(self, tb, iter, text, length):
         # When a text change happens, see if it is within the same line
%%%

Unfortunately, I've lost my password for bugs.grommit.com so until I
recover it, please feel free to use this diff if the latest gpyfm causes
you grief.

- Giorgos

_______________________________________________
tools-discuss mailing list
[email protected]

Reply via email to