[Python-Dev] Re: [Python-checkins] CVS: python/dist/src/Lib webbrowser.py,1.28,1.29

Michael Hudson mwh@python.net
Sun, 17 Mar 2002 15:41:43 +0000 (GMT)


On Sat, 16 Mar 2002, Jack Jansen wrote:

> 
> On zaterdag, maart 16, 2002, at 07:24 , Michael Hudson wrote:
> 
> > Jack Jansen <jackjansen@users.sourceforge.net> writes:
> >
> >> Update of /cvsroot/python/python/dist/src/Lib
> >> In directory usw-pr-cvs1:/tmp/cvs-serv26704
> >>
> >> Modified Files:
> >> 	webbrowser.py
> >> Log Message:
> >> If no webbrowsers were found _tryorder would be empty, cmd would
> >> never be set and the "del cmd" would fail. Fixed.
> >>
> >> 2.2.1 candidate.
> >
> > No it's not :)
> 
> Well, you're the boss, of course, but could you explain, please? 
> This fix addresses a problem where the regression tests fails in 
> MacPython on Mac OS X: if there are no webbrowsers found then 
> "import webbrowser" will crash (because of the unset cmd which 
> is then delled), and test___all__ does an import of webbrowser.

Well, the "del cmd" doesn't exist on the release22-maint branch.

Unless I'm missing something.

Cheers,
M.