[issue14026] test_cmd_line_script should include more sys.argv checks

2012-02-24 Thread Nick Coghlan
Changes by Nick Coghlan : -- assignee: -> ncoghlan ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: > - test_glob_to_re() was doing two levels of escaping (r'\' -> r'') > for its expected output when it should only do one (r'\' -> r'\\'). Fix merged. I don’t fully understand why one place needs two escapes and the others just one. > - test_process_templa

[issue1112955] move_file()'s return value when dry_run=1 unclear

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: My opinion about dry-run modes is that the goal is to be as close as possible to the real operation, but leave the world in a clean state. Therefore, testing for file existence is okay (nevermind about modifying the last accessed time, it isn’t important IMO),

[issue14097] Improve the "introduction" page of the tutorial

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: Be careful with whitespace changes. Sometimes not putting whitespace around an operator helps to see the grouping of operations, for example. The important thing is “Readability counts”, not “Always put whitespace around operators”. A huge part of understandi

[issue13491] Fixes for sqlite3 doc

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: Here’s a patch for 2.7. I haven’t changed the text_factory example. The second patch is for 3.2 and contains a few additional changes that I did in my first patch. -- Added file: http://bugs.python.org/file24634/sqlite-doc-tweaks-2.7.diff _

[issue13491] Fixes for sqlite3 doc

2012-02-24 Thread Éric Araujo
Changes by Éric Araujo : Added file: http://bugs.python.org/file24635/sqlite-doc-tweaks-3.2.diff ___ Python tracker ___ ___ Python-bugs-list m

[issue13973] urllib.parse is imported twice in xmlrpc.client

2012-02-24 Thread Éric Araujo
Changes by Éric Araujo : -- status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue14081] Allow "maxsplit" argument to str.split() to be passed as a keyword argument

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: LGTM. (Nick: I think the author directly translated the regex function calls from other languages to using re in Python.) -- nosy: +eric.araujo ___ Python tracker _

[issue6085] Logging in BaseHTTPServer.BaseHTTPRequestHandler causes lag

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: +1 to Antoine’s proposal of removal. -- nosy: +eric.araujo versions: +Python 3.3 -Python 3.2 ___ Python tracker ___ ___

[issue1116520] Prefix search is filesystem-centric

2012-02-24 Thread Éric Araujo
Changes by Éric Araujo : -- nosy: +carljm, eric.araujo versions: -Python 3.2 ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue14116] Lock.__enter__() method returns True instead of self

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: IIUC returning True is not incorrect, only useless. In the stdlib I usually see “with lock:”. Can you tell what is the use case for accessing the condition object inside the context block? Does it apply only to Condition or also to *Lock and Semaphore? -

[issue13447] Add tests for some scripts in Tools/scripts

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: Good points. New docstring: Tests for scripts in the Tools directory. This file contains regression tests for some of the scripts found in the Tools directory of a Python checkout or tarball, such as reindent.py. When I commit I’ll also send to python-dev

[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: I think it’s a common English term (i.e. “shallow water” to describe a small lake-like thing where you can go without swimming), but non-native speakers may not know it (I don’t remember if I knew it before learning Python, for example). What about this: All s

[issue14114] 2.7.3rc1 chm gives JS error

2012-02-24 Thread Ezio Melotti
Ezio Melotti added the comment: "$" refers to JQuery, and should be defined in http://docs.python.org/_static/jquery.js. Maybe you have to import/include that file before http://docs.python.org/_static/copybutton.js? There are also other js scripts there that use JQuery so I'm not sure why it'

[issue14074] argparse allows nargs>1 for positional arguments but doesn't allow metavar to be a tuple

2012-02-24 Thread Tshepang Lekhonkhobe
Changes by Tshepang Lekhonkhobe : -- title: argparse does not allow nargs>1 for positional arguments but doesn't allow metavar to be a tuple -> argparse allows nargs>1 for positional arguments but doesn't allow metavar to be a tuple ___ Python track

[issue14089] Patch to increase fractions lib test coverage

2012-02-24 Thread Éric Araujo
Éric Araujo added the comment: LGTM -- nosy: +eric.araujo versions: +Python 2.7, Python 3.2 ___ Python tracker ___ ___ Python-bugs-li

<    1   2