Fixed bug
[#495695] webbrowser.py: selection of browser
* Lib/webbrowser.py
Only include graphic browsers in _tryorder if DISPLAY is set. Also,
included skipstone support, as suggested by Fred in the mentioned bug.
* Misc/NEWS
Mention fix and skipstone inclusion.
diff --git a/Lib/webbrowser.py b/Lib/webbrowser.py
index efdebdc..cb9dbb3 100644
--- a/Lib/webbrowser.py
+++ b/Lib/webbrowser.py
@@ -261,8 +261,7 @@
# 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 = ["galeon", "mozilla", "netscape", "kfm",
- "grail", "links", "lynx", "w3m",]
+ _tryorder = ["links", "lynx", "w3m"]
# Easy cases first -- register console browsers if we have them.
if os.environ.get("TERM"):
@@ -278,6 +277,9 @@
# X browsers have more in the way of options
if os.environ.get("DISPLAY"):
+ _tryorder = ["galeon", "skipstone", "mozilla", "netscape",
+ "kfm", "grail"] + _tryorder
+
# First, the Netscape series
if _iscommand("mozilla"):
register("mozilla", None, Netscape("mozilla"))
@@ -293,6 +295,11 @@
if _iscommand("galeon"):
register("galeon", None, Galeon("galeon"))
+ # Skipstone, another Gtk/Mozilla based browser
+ if _iscommand("skipstone"):
+ register("skipstone", None, GenericBrowser(
+ "skipstone '%s' >/dev/null &"))
+
# Konqueror/kfm, the KDE browser.
if _iscommand("kfm") or _iscommand("konqueror"):
register("kfm", Konqueror, Konqueror())
diff --git a/Misc/NEWS b/Misc/NEWS
index d1fa3f9..2f78694 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -628,6 +628,10 @@
the config file, including the contents of the referred filename in
the "%verifyscript" section of the rpm spec file.
+- Fixed bug #495695: webbrowser module would run graphic browsers in a
+ unix environment even if DISPLAY was not set. Also, support for
+ skipstone browser was included.
+
Tools/Demos
-----------