_tryorder should always be a list, then the problem Jack had to fix in

gvanrossum · gvanrossum · commit cb33165ca2f8 · 2001-12-03T15:51:31.000Z
1.24 wouldn't have occurred in the first place.

Remove a debug print command accidentally inserted by Martin in 1.23.
diff --git a/Lib/webbrowser.py b/Lib/webbrowser.py
@@ -116,7 +116,6 @@ def _remote(self, action, autoraise):
         cmd = "%s %s -remote '%s' >/dev/null 2>&1" % (self.name,
                                                       raise_opt,
                                                       action)
-        print cmd
         rc = os.system(cmd)
         if rc:
             import time
@@ -233,7 +232,7 @@ def open_new(self, url):
 # the TERM and DISPLAY cases, because we might be running Python from inside
 # an xterm.
 if os.environ.get("TERM") or os.environ.get("DISPLAY"):
-    _tryorder = ("mozilla","netscape","kfm","grail","links","lynx","w3m")
+    _tryorder = ["mozilla","netscape","kfm","grail","links","lynx","w3m"]
 
     # Easy cases first -- register console browsers if we have them.
     if os.environ.get("TERM"):
@@ -282,7 +281,7 @@ def open_new(self, url):
 #
 
 if sys.platform[:3] == "win":
-    _tryorder = ("netscape", "windows-default")
+    _tryorder = ["netscape", "windows-default"]
     register("windows-default", WindowsDefault)
 
 #
@@ -296,15 +295,15 @@ def open_new(self, url):
 else:
     # internet-config is the only supported controller on MacOS,
     # so don't mess with the default!
-    _tryorder = ("internet-config", )
+    _tryorder = ["internet-config"]
     register("internet-config", InternetConfig)
 
 #
 # Platform support for OS/2
 #
 
 if sys.platform[:3] == "os2" and _iscommand("netscape.exe"):
-    _tryorder = ("os2netscape",)
+    _tryorder = ["os2netscape"]
     register("os2netscape", None,
              GenericBrowser("start netscape.exe %s"))
 
