Issue 4828: patch suggestion for webbrowser (original) (raw)

Issue4828

Created on 2009-01-04 10:09 by Yinon, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
webbrowser.patch Yinon,2009-01-04 10:09 review
Messages (5)
msg79041 - (view) Author: Yinon Ehrlich (Yinon) Date: 2009-01-04 10:13
this will find applications/batch files ending with .com or .cmd too.
msg80014 - (view) Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * (Python committer) Date: 2009-01-17 13:36
I was to suggest to use the PATHEXT variable instead of a hardcoded list of extensions, but does all this work at all? For example, I added a "opera.bat" in my current directory. a browser named 'opera' was registered, but it did not start! >>> webbrowser.get('opera').open('http://www.google.com') False And indeed, the subprocess module does not start .bat files unless "shell=True" is given.
msg85720 - (view) Author: anatoly techtonik (techtonik) Date: 2009-04-07 15:53
Windows has a plenty of executable extensions that can be dynamically added. You may want to look at patch at issue 2200 http://bugs.python.org/file11359/spawn.patch.txt for workaround.
msg119677 - (view) Author: anatoly techtonik (techtonik) Date: 2010-10-27 07:21
Any explanations?
msg119680 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2010-10-27 07:30
Oh, sorry. Shouldn't do too many things at once. Well, I think there aren't many webbrowsers that come as a .com file, for example. And as Amaury shows, the patch can't be applied as is anyway.
History
Date User Action Args
2022-04-11 14:56:43 admin set github: 49078
2010-10-27 07:30:31 georg.brandl set nosy: + georg.brandlmessages: +
2010-10-27 07:21:04 techtonik set messages: +
2010-10-27 07:10:57 georg.brandl set status: open -> closedresolution: wont fix
2009-04-07 15:53:05 techtonik set nosy: + techtonikmessages: +
2009-01-17 13:36:28 amaury.forgeotdarc set nosy: + amaury.forgeotdarcmessages: +
2009-01-04 10:13:17 Yinon set messages: +
2009-01-04 10:09:06 Yinon create