[issue8823] urllib2 does not catch httplib.BadStatusLine

2012-07-11 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: By the way, the issue can be recreated by running the following: netcat -l -p -e echo HTTP/1.1 1000 OK python -c import urllib2; urllib2.urlopen('http://localhost:') This happens on 2.6, 2.7 and 3 by the looks

[issue11176] give more meaningful argument names in argparse documentation

2012-07-09 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: Hi David Ok, I like the idea of working on a solution together. I like your idea of the pizza-making more than the current foo bar examples. Should we collaborate outside of the bug tracker? Greg On 9 July 2012 07:11, David Lam rep

[issue11176] give more meaningful argument names in argparse documentation

2012-07-09 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: I've made the minor edits required after the review to my simplification of the first example. David, perhaps we combine our efforts? Use my simple first example to explain the very basics, then continue the explanation with the pizza example

[issue1076515] shutil.move clobbers read-only files.

2012-07-09 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: I can add some more info to the docs if anybody feels they are required, but I think they are sufficient. It already mentions that it copies file contents and that the correct permissions are required. This is mentioned in the docs

[issue11176] give more meaningful argument names in argparse documentation

2012-07-07 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: Is this still an issue? If so, I've created a simpler first example as suggested below. If we decide these docs still need a bit more work, I can also continue to provide better examples than the foo bar ones. -- keywords: +patch nosy

[issue15183] it should be made clear that the statement in the --setup option and the setup kw arg aren't included in the count

2012-07-07 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: Confirmed that the setup statement is correctly excluded from the overall timed run. I've updated the docstring and rst to make this clearer. -- keywords: +patch nosy: +groodt Added file: http://bugs.python.org/file26288

[issue8823] urllib2 does not catch httplib.BadStatusLine

2012-07-07 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: I've made a small change to urllib2 to catch the httplib.BadStatusLine and raise as URLError. This exception should rarely happen as it means the server is returning invalid responses. Nevertheless, I've added a test and hopefully fixed the issue

[issue5815] locale.getdefaultlocale() missing corner case

2012-07-07 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: Bumping this as part of a bug scrub at EuroPython. Is this still an issue? Should we fix in docs or in code? -- nosy: +groodt ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5815

[issue5931] Python runtime name hardcoded in wsgiref.simple_server

2012-07-07 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: Bumping as part of bug scrub at EuroPython. Patch seems sensible to me. Obvious options to me are to either accept patch, close bug or write docs. Bug has not seen activity in years, so can probably be closed. -- nosy: +groodt

[issue1076515] shutil.move clobbers read-only files.

2012-07-07 Thread Greg Roodt
Greg Roodt gro...@gmail.com added the comment: I believe this can be closed. shutil.copyfile attempts to copy file contents from src to dst, so it makes sense for this to fail if dst is read-only. shutil.move replaces the dst, which is permitted based on directory permissions