[Python-checkins] r74643 - in python/trunk: Lib/webbrowser.py Misc/NEWS

[georg.brandl]

Author: georg.brandl
Date: Fri Sep  4 08:59:20 2009
New Revision: 74643

Log:
Issue #2666: Handle BROWSER environment variable properly for unknown browser names in the webbrowser module.


Modified:
   python/trunk/Lib/webbrowser.py
   python/trunk/Misc/NEWS

Modified: python/trunk/Lib/webbrowser.py
==============================================================================
--- python/trunk/Lib/webbrowser.py	(original)
+++ python/trunk/Lib/webbrowser.py	Fri Sep  4 08:59:20 2009
@@ -625,7 +625,9 @@
     # and prepend to _tryorder
     for cmdline in _userchoices:
         if cmdline != '':
-            _synthesize(cmdline, -1)
+            cmd = _synthesize(cmdline, -1)
+            if cmd[1] is None:
+                register(cmdline, None, GenericBrowser(cmdline), -1)
     cmdline = None # to make del work if _userchoices was empty
     del cmdline
     del _userchoices

Modified: python/trunk/Misc/NEWS
==============================================================================
--- python/trunk/Misc/NEWS	(original)
+++ python/trunk/Misc/NEWS	Fri Sep  4 08:59:20 2009
@@ -364,6 +364,9 @@
 Library
 -------
 
+- Issue #2666: Handle BROWSER environment variable properly for unknown browser
+  names in the webbrowser module.
+
 - Issue #6054: Do not normalize stored pathnames in tarfile.
 
 - Issue #6794: Fix Decimal.compare_total and Decimal.compare_total_mag: NaN