Marvellous, thanks for the blazingly fast fix :)
On 2007-07-10 08:38, Stephen Lau <[EMAIL PROTECTED]> wrote:
> Thanks Giorgos,
> I've gone ahead and patched and pushed it. Many thanks for the fix
>
> cheers,
> steve
>
> Giorgos Keramidas wrote:
> > 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]
>
>
> --
> stephen lau // [EMAIL PROTECTED] | http://whacked.net
> opensolaris // solaris kernel development
_______________________________________________
tools-discuss mailing list
[email protected]
