[MERGE] tweaked copy of Michael Ellermans editor selection fix.

Olaf Conradi oohlaf at gmail.com
Fri Apr 28 02:05:33 BST 2006


On 28/04/06, Robert Collins <robertc at robertcollins.net> wrote:
> Desperately seeking +1's for 0.8.
>
> Rob

     if os.name == "nt":

         yield "notepad.exe"

     elif os.name == "posix":

-        yield "/usr/bin/vi"

+        for editor in ['vi', 'pico', 'nano', 'joe']:

+            yield editor

I would prefer the same test as in osutils, i.e. sys.platform == "win32"

We could make some editor preferences for win32 too, like tednpad.exe
or uedit32.exe (maybe not the last one as it's not free).

Cheers
 -Olaf




More information about the bazaar mailing list