[Patches] [ python-Patches-1166780 ] Fix _tryorder in webbrowser.py

[SourceForge.net]

Patches item #1166780, was opened at 2005-03-20 13:25
Message generated for change (Comment added) made by mdehoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1166780&group_id=5470

Category: Library (Lib)
Group: Python 2.5
Status: Open
Resolution: None
Priority: 5
Submitted By: Rodrigo Dias Arruda Senra (rodsenra)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix _tryorder in webbrowser.py

Initial Comment:
At the present time (Py2.4):
"""
Under Unix, if the environment variable BROWSER exists,
it is 
interpreted to override the platform default list of
browsers,...
"""
(extract from
Python-2.4/Doc/html/lib/module-webbrowser.html)

But, when the environment variable BROWSER is messed up,
there is no reason not to try the other detected
browser alternatives.

In this patch, if the BROWSER variable is Ok, than it
is respected.
Otherwise, the previously detected browsers are tryied
out as if
BROWSER variable never existed.

This does not break backward compatibility and adds
more chance
for webbrowser.open() to succeed.

This patch was made against CVS head 2005-03-20, and
aims to 2.5, but can safely be apllied to any 2.4.x bug
fix release.



----------------------------------------------------------------------

Comment By: Michiel de Hoon (mdehoon)
Date: 2005-03-27 17:03

Message:
Logged In: YES 
user_id=488897

I agree that this patch will improve chances that
webbrowser.open() will succeed. I'm a bit worried though
that if a user sets BROWSER incorrectly, it may cause some
confusion to see another browser opening. So we might some
some bug reports saying "webbrowser.py ignores BROWSER
variable" because a user set BROWSER incorrectly. One
solution might be to print some warning message if the
browser specified in BROWSER cannot be used.
(I'm just a patch reviewer, not an official Python
developer, so you don't need to take my comment to seriously
:-)).

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1166780&group_id=5470