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 _______________________________________________ Patches mailing list [email protected] http://mail.python.org/mailman/listinfo/patches
