[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files
Vajrasky Kok added the comment: This is updated patch for marshal incorporating the fixes from clinic. Now, I can use 'bytes' named argument and default value of marshal.version instead of hardcoded number. -- Added file: http://bugs.python.org/file33644/clinic_marshal_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20185 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20341] Argument Clinic: add nullable ints
Larry Hastings added the comment: Yes, that is a different name that seems to mean much the same thing. and which is much more understandable by a Python developer. Thanks for your opinion. Changing error to an char and moving it to the end would save exactly zero bytes, because the compiler *will* align stack variables to 4 byte boundaries. Except if other stack variables happen to be shorter than an int, perhaps. Do we use a lot of those? But regardless, it doesn't cost anything to do so, so why not do it? I thought I just got through explaining how it is slightly slower. There is no benefit in changing them to char, so why do it? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20341 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable
Arfrever Frehtes Taifersar Arahesis added the comment: The attached patch moves calls to PySys_AddWarnOption* for -W options to after parsing PYTHONWARNINGS. I think that this bug fix could be applied in Python 3.4. The patch does not introduce any failures in full test suite. -- keywords: +patch nosy: +larry Added file: http://bugs.python.org/file33645/python-warnings.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20355 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20358] test_curses is failing
New submission from Larry Hastings: If I build current trunk on my workstation (Linux 64-bit, Ubuntu 13.10) then run % ./python -m test -u all test_curses I get a traceback. Here's the interesting part: File /home/larry/src/python/buildtrunk/Lib/test/test_curses.py, line 118, in window_funcs win2.overlay(win, 1, 2, 3, 3, 2, 1) _curses.error: copywin() returned ERR Zach: I just saw you had a patch specifically against test_curses.py, can you take a look? -- components: Library (Lib) messages: 208887 nosy: larry, zach.ware priority: high severity: normal stage: needs patch status: open title: test_curses is failing type: behavior versions: Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20358 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files
Vajrasky Kok added the comment: Here is the updated patch of sha1module, based on Larry's review. Thanks! -- Added file: http://bugs.python.org/file33646/clinic_sha1module_v4.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20173 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- stage: needs patch - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20355 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Ram Rachum added the comment: Hi Christopher, I like your patch. One thing I modified is returning to use `file` as the variable instead of `f`, since `file` is no longer a builtin in Python 3, and descriptive variable names are important. Attached as `pathlib.readwrite2.patch`. -- Added file: http://bugs.python.org/file33647/pathlib.readwrite2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
New submission from Steve: If you change the sys.ps1 to have some color, you end up messing up readline's input line calculations and the characters in the line you are typing might not get displayed properly. This behaviour is easier to demonstrate/reporduce than explain ...: On a python promt with readline enabled (for instance, on any linux box): [steve@localhost ~]$ python Python 2.7.5 (default, Nov 12 2013, 16:18:42) [GCC 4.8.2 20131017 (Red Hat 4.8.2-1)] on linux2 Type help, copyright, credits or license for more information. import sys sys.ps1 = '\033[31m \033[0m' # doing a reverse-i-search (Ctrl+r) on the next line will mess up the prompt display ... h)`sys': sys.ps1 = '\033[31m \033[0m' (...press Ctrl-L to clear...) (reverse-i-search)`': (press enter) arch)`': I did some (amateur) investigation and the root cause might be similar to what's reported at: http://trac.haskell.org/haskeline/ticket/78 https://groups.google.com/forum/#!topic/gnu.bash.bug/5P0gWzzLVOU FWIW, I bumped into this issue while customizing my .pythonrc[1] and although it might be a silly small annoyance, it would be really nice if this bug could be fixed. [1] https://gist.github.com/lonetwin/5902720 -- components: Library (Lib) messages: 208890 nosy: lonetwin priority: normal severity: normal status: open title: Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations type: behavior versions: Python 2.7, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20358] test_curses is failing
Changes by Nadeem Vawda nadeem.va...@gmail.com: -- nosy: +nadeem.vawda ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20358 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
Georg Brandl added the comment: Thanks for the report. I think it's your obligation to surround nonprinting escape sequences with \x01 and \x02, with that change it works for me. -- nosy: +georg.brandl resolution: - works for me status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20358] test_curses is failing
Changes by Serhiy Storchaka storch...@gmail.com: -- nosy: +serhiy.storchaka ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20358 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20360] inspect.Signature could provide readable expressions for default values for builtins
New submission from Larry Hastings: The signatures for builtins are stored as text and interpreted at runtime on demand. inspect.Signature gets the default value for a parameter as a text string, which it evaluates (by hand, in a highly constrained way) to produce the parameter's value member. Serhiy cleverly suggested that it'd be nice if Signature objects could also make this string available, as it's often more readable than the value. For example, there's a parameter in zlib with the default MAX_DECOMPRESSION - 1; that turns into 7 I think. Which would you prefer? Right now there's no place to store this in the parameter object, so it can't happen for 3.4. But perhaps in 3.5 we could add it, and figure out how to present it to the user. For example __str__ could use it. Thinking a little laterally: we *could* hide it in a *private* member on the Parameter object, then only use it for calculating __str__. That would look so fancy! Should we consider doing that for 3.4? -- messages: 208892 nosy: brett.cannon, larry, ncoghlan, serhiy.storchaka, yselivanov priority: normal severity: normal stage: needs patch status: open title: inspect.Signature could provide readable expressions for default values for builtins type: enhancement versions: Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20360 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20338] Idle: increase max calltip width
Terry J. Reedy added the comment: 1. For multiple reasons, I want to at least start with only wrapping the signature string. I want to keep the code as simple and fast as possible, and not worry much about rare oddball or PEP8-violating cases. I want to only do what is needed for the purpose of calltips, which is to remind one of the call sequence. Adding more detracts from that purpose. Tips are not intended to replace using the doc or help(f) to learn about f. As it is, I am not happy about possibly including more than 1 or 2 lines of a docstring for Python-coded functions. I intend to revisit that when the dust settles with Clinic and inspect. That fact that another patch *will* be needed, almost certainly, is another reason to not do too much now. Wrapping overly long docstrings should not needed. Even if the initial _MAX_COL slice of the first line of the docstring of a Python-coded function fails to say enough about the return value, there is help(f) to get full details. I reread PEP 8 and, contrary to what I (mis) remembered, it says that docstrings should remain limited to 72 (or 79) chars *even if* a project increases the limit for *code* lines up to say, 100. The reason for this is precisely because other software, like help, Idle, and others, read, process, and display docstrings. If docstrings violate this, I think they should be changed, and not accomadated (except as suggested below). So I want to add, before the docstring fetch, only this: if len(argspec) _MAX_COLS: argspec = textwrap.wrap(argspec, _MAX_COLS, subsequent_indent=' ') dict is definitely an oddball and I do not want to include it in a test at this time. I do not mind suggesting help(dict), though the current docstring is incomplete. Here is the real signature and description, which I verified from the examples below. The multiple example lines should follow a condensed summary. def dict([map_or_it,] **kwds) '''Return new dictionary initialized from pairs of positional-only map_or_it, if present, and updated with kwds, if present. start current docstring with specifics ''' dict({'a':1}, b=2) {'b': 2, 'a': 1} dict({'a':1}, a=2) {'a': 2} 2. For two reasons, I want to increase _MAX_COLS a bit, say to 85. First, when wrapping a signature, the 'words' after the first few typically have the form 'name=value'. They will average much longer than the average English word (4, I think). The average wrapped line length will be _MAX_COLS - half the average 'word' length. For example (one which should become a test): for line in textwrap.wrap(str(signature(textwrap.TextWrapper)), width = 85, subsequent_indent=' '): print(line) (width=70, initial_indent='', subsequent_indent='', expand_tabs=True, replace_whitespace=True, fix_sentence_endings=False, break_long_words=True, drop_whitespace=True, break_on_hyphens=True, tabsize=8, *, max_lines=None, placeholder=' [...]') The lengths, including indents, are 69, 78, 77, and 24, all = 79. Very few signature lines, wrapped or not, will fall in [80-85]. Second, PEP 8 allows initial summary lines to be 79 chars, as they are not flowing long blocks of text. (I verified this with Guido.) Some docstrings in the stdlib 'cheat' a few chars, and 85 will accommodate those without ever being excessively long. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20338 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- stage: - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20361 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options
New submission from Arfrever Frehtes Taifersar Arahesis: Currently -W command line options and PYTHONWARNINGS environmental variable non-intuitively override settings specified by -b / -bb command line options. $ python3.4 -c 'print( == b)' False $ python3.4 -b -c 'print( == b)' -c:1: BytesWarning: Comparison between bytes and string False $ python3.4 -bb -c 'print( == b)' Traceback (most recent call last): File string, line 1, in module BytesWarning: Comparison between bytes and string $ python3.4 -We -b -c 'print( == b)' Traceback (most recent call last): File string, line 1, in module BytesWarning: Comparison between bytes and string $ python3.4 -Wd -bb -c 'print( == b)' -c:1: BytesWarning: Comparison between bytes and string False $ python3.4 -Wi -b -c 'print( == b)' False $ python3.4 -Wi -bb -c 'print( == b)' False $ I suggest to increase priority of -b / -bb command line options. The attached patch should be applied after patch from issue #20355. (Test suite of Python triggers some BytesWarnings, so firstly they will have to be fixed.) -- components: Interpreter Core files: python-bytes_warnings.patch keywords: patch messages: 208894 nosy: Arfrever, barry, brett.cannon, brian.curtin, eric.araujo, larry, pitrou, pjenvey, skrah priority: normal severity: normal status: open title: -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options type: behavior versions: Python 3.4 Added file: http://bugs.python.org/file33648/python-bytes_warnings.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20361 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files
Vajrasky Kok added the comment: Here is the updated patch of sha256module, based on Larry's review. Thanks! -- Added file: http://bugs.python.org/file33649/clinic_sha256module_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20173 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
Steve added the comment: wow, that was super quick ! Yes, indeed. Surrounding the escape sequences with '\0x1' '\0x2' fixes this issue. Thanks ! -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
Steve added the comment: ...of course I meant \x01 and \x02, like you suggested -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options
Arfrever Frehtes Taifersar Arahesis added the comment: By the way, Tools/scripts/run_tests.py (used by 'test' target in Makefile) passes '-W default -bb', which currently fails work as intended (i.e. treating BytesWarnings as errors). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20361 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20362] longMessage attribute is ignored in unittest.TestCase.assertRegexpMatches etc
New submission from Dhara: Various assert methods added in version 2.7 ignore the longMessage attribute: http://docs.python.org/2.7/library/unittest.html#unittest.TestCase.longMessage For example when matching regular expressions using unittest.TestCase.assertRegexpMatches, the following error is generated: AssertionError: custom short message: 'regexp' not found in 'text' Since longMessage is ignored, and the text to be matched can be pages long, the error messages are impossibly long. [See also: http://stackoverflow.com/questions/21157905/suppress-supplementary-error-message-from-unit-test] -- components: Library (Lib) messages: 208899 nosy: Dhara priority: normal severity: normal status: open title: longMessage attribute is ignored in unittest.TestCase.assertRegexpMatches etc type: behavior versions: Python 2.7 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20362 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20338] Idle: increase max calltip width
Serhiy Storchaka added the comment: Well, do this. I think we should add several test functions with too long signatures and/or docstrings. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20338 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20338] Idle: increase max calltip width
Serhiy Storchaka added the comment: And note that currently there is a bug in doc.split('\n', 5)[:_MAX_LINES]. An argument of split() should be _MAX_LINES. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20338 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files
Vajrasky Kok added the comment: Here is the updated patch of sha512module, based on Larry's review. Thanks! -- Added file: http://bugs.python.org/file33650/clinic_sha512module_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20173 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- dependencies: +BytesWarnings triggerred by test suite ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20361 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20363] BytesWarnings triggerred by test suite
New submission from Arfrever Frehtes Taifersar Arahesis: [194/388] test_base64 /tmp/cpython/Lib/base64.py:365: BytesWarning: str() on a bytes instance by {} and {}.format(_A85START, _A85END)) ... [204/388] test_hash /tmp/cpython/Lib/test/test_hash.py:175: BytesWarning: str() on a bytes instance return 'print(hash(eval(%s.decode(utf-8' % repr_.encode(utf-8) ... [234/388] test_configparser /tmp/cpython/Lib/configparser.py:289: BytesWarning: str() on a bytes instance Error.__init__(self, 'Source contains parsing errors: %s' % source) /tmp/cpython/Lib/configparser.py:326: BytesWarning: str() on a bytes instance (filename, lineno, line)) ... [332/388/1] test_distutils /tmp/cpython/Lib/distutils/command/register.py:303: BytesWarning: str() on a bytes instance self.announce('%s%s%s' % (dashes, data, dashes)) -- components: Library (Lib) messages: 208903 nosy: Arfrever, eric.araujo, lukasz.langa, tarek priority: normal severity: normal status: open title: BytesWarnings triggerred by test suite versions: Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20363 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20352] Add support for AUTH command to poplib
Daniël van Eeden added the comment: As far as I understood the RFC: A client should send CAPA and check if there is a SASL tag in the response (e.g. SASL PLAIN). === +OK Dovecot ready. AUTH PLAIN base64_encoded_info +OK Logged in. LIST === I've replace the base64 encoded authentication info with 'base64_encoded_info' For other authentication mechanisms the response can be longer (it may contain a challenge) and the request may only contain the mechanim. I don't have a server which supports anything else than AUTH PLAIN, so I could verify/test this. -- keywords: +patch Added file: http://bugs.python.org/file33651/poplib_auth.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20352 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
Steve added the comment: Although surrounding the escapes with \x01 and \x02 worked to correctly remove the message while doing a reverse-i-search (thus preserving the original line), the input line measurement still appears to be incorrect as demonstated with the following actions: [steve@localhost ~]$ python Python 2.7.5 (default, Nov 12 2013, 16:18:42) [GCC 4.8.2 20131017 (Red Hat 4.8.2-1)] on linux2 Type help, copyright, credits or license for more information. import sys sys.ps1 = '\x01\033[32m \033[0m\x02' # on the next line, press the up arrow to get this line and then backspace, over the entire prompt ... #-- backspaced up until here ... ...again, not a big deal, just annoying when deleting word with a repeated 'alt+backspace' or ctrl+w -- status: closed - open ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20336] test_asyncio: relax timings even more
Stefan Krah added the comment: I'm seeing other unrelated timeouts on the OpenIndiana machines. Jesús, are the machines perhaps operating under an unusually high load? -- nosy: +jcea ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20336 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files
Vajrasky Kok added the comment: Here is the updated patch of md5module, based on Larry's review. Thanks! -- Added file: http://bugs.python.org/file33652/clinic_md5module_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20173 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20336] test_asyncio: relax timings even more
STINNER Victor added the comment: I'm seeing other unrelated timeouts on the OpenIndiana machines. I read somewhere that the host of these virtual machines is *highly* loaded (load 6). It's more a buildbot configuration issue than a test_asyncio issue. -- nosy: +haypo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20336 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20364] Rename explain sqlite3.Cursor.execute 'parameters' param
New submission from Terry J. Reedy: execute(sql[, parameters]) Executes an SQL statement. The SQL statement may be parametrized (i. e. placeholders instead of SQL literals). The sqlite3 module supports two kinds of placeholders: question marks (qmark style) and named placeholders (named style). Experimental facts based on experiments with the code example in the doc, using 3.4.b2: 'parameters' is a single subscriptable collection parameter, sequence or dict, that might be called seq_dict. It is positional only, so whatever name is used is a dummy. Only one placeholder style can be used in a given SQL statement string. If question marks are used, seq_dict must be a sequence. If names are used, seq_dict can be either a sequence or dict or subclass thereof. A UserDict is treated as a sequence and raises KeyError(0). Possible text that encompasses the above, replacing the last sentence: A statement may use one of two kinds of placeholders: question marks (qmark style) or named placeholders (named style). For qmark style, seq_dict must be a sequence. For named style, it can be either a sequence or dict instance. Len(seq_dict) must match the number of placeholders. After cleaning up the test file, I will verify on 2.7 and upload. -- assignee: docs@python components: Documentation, Library (Lib) messages: 208908 nosy: docs@python, terry.reedy priority: normal severity: normal stage: patch review status: open title: Rename explain sqlite3.Cursor.execute 'parameters' param type: behavior versions: Python 2.7, Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20364 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
Georg Brandl added the comment: I think you're using the escapes correctly: you must only escape nonprinting characters. Try sys.ps1 = '\x01\033[32m\x02 \x01\033[0m\x02' -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20336] test_asyncio: relax timings even more
Stefan Krah added the comment: I see. That's astonishing, because usually the builds are ultra-fast on those machines. -- nosy: -gvanrossum ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20336 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20336] test_asyncio: relax timings even more
STINNER Victor added the comment: Tests failing because of timings is really annoying. It would be nice to add something in test.support to configure timings. A function can maybe wait until the system load is lower than a threshold, or we should add a parameter to configure a tolerance between the perfect timing and the expected timing. The tolerance can be a simple factor, a floating point number = 1.0. For example, replace: 0.09 = t1-t0 = 0.9, t1-t0 with 0.1 - 0.01 * factor = t1-t0 = 0.1 + 0.8 * factor or a new test function: check_timing(t1-t0, 0.1, 0.01, 0.8) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20336 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20336] test_asyncio: relax timings even more
Stefan Krah added the comment: +1 for doing something about timing failures. Perhaps also a decorator @skip_if_load_higher_than(x). [Nick, I'm adding you because our current test suite could lead to commit impossible situations with Zuul.] -- nosy: +ncoghlan ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20336 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20345] Better logging defaults
Vinay Sajip added the comment: Can you give me a hint, how I could investigate that? There's no easy way - a lot of Python source is not available for public view, so there's no telling what the impact would be. While the use of the module-level functions is provided as a convenience, I don't feel that the need to call basicConfig () is problematic, even for short scripts. -- status: open - pending ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20345 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
Changes by Mark Dickinson dicki...@gmail.com: -- nosy: +mark.dickinson ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
Mark Dickinson added the comment: Adding Tim Peters to the nosy, since I suspect he has a general interest in this kind of issue. As far as I know Brian Quinlan isn't actively maintaining concurrent.futures at the moment (Brian: please correct me if I'm wrong). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
Changes by Mark Dickinson dicki...@gmail.com: -- nosy: +tim.peters ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19863] Missing function attributes in 2.7 docs.
Mark Dickinson added the comment: LGTM, but may be write Py3k compatible names at first place? Sure; I don't have strong opinions either way. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19863 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures
Changes by Glenn Langford glenn.langf...@gmail.com: -- nosy: +mark.dickinson ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20297 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
Changes by STINNER Victor victor.stin...@gmail.com: -- nosy: +sbt ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
STINNER Victor added the comment: @Glenn: Can you maybe suggest a patch fixing the issue? -- nosy: +haypo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
Glenn Langford added the comment: @Victor: Would like to give a patch but I am not a core developer, and I don't version control set up yet. The proposed fix is based on reading the distribution source code. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20362] longMessage attribute is ignored in unittest.TestCase.assertRegexpMatches etc
Changes by Berker Peksag berker.pek...@gmail.com: -- nosy: +ezio.melotti, michael.foord ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20362 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20341] Argument Clinic: add nullable ints
Antoine Pitrou added the comment: I insist that nullable is a very bad name. It also suggests that somehow we'll get a NULL pointer (in C) when the parameter is absent or None, which is not the case. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20341 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20319] concurrent.futures.wait() can block forever even if Futures have completed
STINNER Victor added the comment: futures.patch: reuse _AcquireFutures context manager to protect the list of futures against concurrent access. Without the patch, stress_wait.py hangs. With the patch, it works correctly. -- keywords: +patch Added file: http://bugs.python.org/file33653/futures.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20319 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures
STINNER Victor added the comment: I attached Glenn's function as a patch. Comments on as_completed_proposed.patch: - _create_and_install_waiters() doesn't remove duplicates using set(), so you change the behaving of as_completed(). I don't think that it is correct. as_completed([f, f]) should yield f twice, but I didn't check the current behaviour. Future._waiters is a list and so accept duplicated waiters. - you replaced the _AcquireFutures context manager on all futures with an individual lock. In my opinion, it should be changed in a different patch. I don't know which option is better, but if it is changed, it should be changed in the whole file. @Glenn: You should take a look at the Python Developer's Guide to learn how to write a patch and how to contribute to Python ;-) http://docs.python.org/devguide/ -- keywords: +patch nosy: +haypo Added file: http://bugs.python.org/file33654/as_completed_proposed.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20297 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19291] Add docs for asyncio package (Tulip, PEP 3156)
STINNER Victor added the comment: I've done a bit of it now, I'll let other people continue. I continued the work on the documentation. Even if it's not complete, I consider that the initial issue Add docs for asyncio package (Tulip, PEP 3156) is done, and so I close the issue. -- resolution: - fixed status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19291 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20352] Add support for AUTH command to poplib
R. David Murray added the comment: imaplib has an API for handling that kind of thing. Maybe we can model the poplib support off of that API. It would be nice to be consistent, assuming it in fact makes sense for poplib as well. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20352 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20313] inspect.signature should raise ValueError for builtins with no signature info
Larry Hastings added the comment: I've worked your changes into the patch for #20189. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20313 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Christopher Welborn added the comment: Ram Rachum: Have you look at the source for pathlib? f and fd are common names for file descriptors, especially when using the open() function. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20311] epoll.poll(timeout) and PollSelector.select(timeout) must round the timeout to the upper bound
Roundup Robot added the comment: New changeset 3637ccf9516a by Victor Stinner in branch 'default': Issue #20311: add debug help in test_selectors http://hg.python.org/cpython/rev/3637ccf9516a -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20311 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20364] Rename explain sqlite3.Cursor.execute 'parameters' param
R. David Murray added the comment: I wonder if the fact that a sequence is accepted in named mode should actually be considered a bug and not documented. Also, is it really true that the number of items must match even in named mode? I think I remember passing a dict with extra elements, but I don't have that code handy to check. -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20364 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20311] epoll.poll(timeout) and PollSelector.select(timeout) must round the timeout to the upper bound
STINNER Victor added the comment: http://buildbot.python.org/all/builders/x86%20Ubuntu%20Shared%203.x/builds/9552 == FAIL: test_timeout_rounding (test.test_epoll.TestEPoll) -- Traceback (most recent call last): File /srv/buildbot/buildarea/3.x.bolen-ubuntu/build/Lib/test/test_epoll.py, line 59, in test_timeout_rounding self.assertGreaterEqual(dt, timeout) AssertionError: 0.009316698648035526 not greater than or equal to 0.01 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20311 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Changes by Ram Rachum r...@rachum.com: Removed file: http://bugs.python.org/file33647/pathlib.readwrite2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Ram Rachum added the comment: You're right. I deleted my 2 patches, so `pathlib.readwrite.patch` is now the best patch for this. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19334] test_asyncio hanging for 1 hour (non-AIX version)
STINNER Victor added the comment: I looked at last tests of FreeBSD 6.4 3.x between builds 4400 and 4432. The builds 4401, 4408, 4416 and 4421 hanged but because of check_interrupted_write_retry() of test_io, but none hanged because of test_asyncio. I also checked builds 3620..3643 of x86 RHEL 6 3.x: all succeeded. Same on x86 Ubuntu Shared 3.x: no hang between builds 9560 and 9568. So I consider this issue as fixed. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19334 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Changes by Ram Rachum r...@rachum.com: Removed file: http://bugs.python.org/file33616/patch.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Antoine Pitrou added the comment: Thanks for the patches, but please let's not hurry this up. We first need to decide on an API (which should be minimal). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19936] Executable permissions of Python source files
Roundup Robot added the comment: New changeset 1cfce469642d by Serhiy Storchaka in branch '3.3': Issue #19936: Restored executable bits for several libffi files. http://hg.python.org/cpython/rev/1cfce469642d New changeset 9adce4e25bdb by Serhiy Storchaka in branch 'default': Issue #19936: Restored executable bits for several libffi files. http://hg.python.org/cpython/rev/9adce4e25bdb -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19936 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20360] inspect.Signature could provide readable expressions for default values for builtins
Nick Coghlan added the comment: It wouldn't be the first time I've used a private variable to fix a problem while still abiding by the no-new-features rule, but I think in this particular case we're better off waiting :) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20360 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19334] test_asyncio hanging for 1 hour (non-AIX version)
Stefan Krah added the comment: It was hanging on FreeBSD 7.2 recently: http://buildbot.python.org/all/builders/x86%20FreeBSD%207.2%203.x/builds/4917/steps/test/logs/stdio The older FreeBSD systems have broken threading though, so we could just disable the offending tests. I would not waste time debugging anything threading related before FreeBSD 9.0. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19334 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20189] inspect.Signature doesn't recognize all builtin types
Larry Hastings added the comment: At last, my refreshed patch. Changes from the previous patch: * Had another mildly bright idea. The name PyTypeObject *cls is a holdover from Python 2.2 days, before the merging of classes and types. Now they're both the same thing and the official name in Python is type. So the self_converter uses the word type. * Added __text_signature__ to PyType_Type. * Removed __text_signature__ from PyMemberDescr_Type, as it's not callable. (What was I thinking!) * Hand-coded signatures for default docstrings for slots. * Hand-coded signature for type and object. (The one for type isn't correct, I know, we'll get there.) * clinic.py now generates self/module/type parameters for most text signatures; these are then stripped off if the function is bound. * Ensured that inspect.signature raises ValueError if it can't generate a signature for a callable object. * Added unit tests. * Suppress the null self parameter for METH_STATIC calls in the impl. * If you have an empty docstring for __new__ or __init__, it suppresses the static variables completely. * Functions with a self converter use the type of self for the parsing function. (Except __new__ and __init__, which must conform to using PyObject *.) Boy am I emotionally ready to check this thing in. -- Added file: http://bugs.python.org/file33655/larry.support.text_signature.on.more.types.6.txt ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20189 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20345] Better logging defaults
Arne Babenhauserheide added the comment: Just calling basicConfig() is not problematic. Specifying the format is what requires mental investment (because for that I need to read up on the config options). Would it be possible to add a simple way to select a certain logging style? In Mercurial I can for example call `hg log --style changelog` to get GNU-style changelog output. This is easy to remember. On the other hand, getting --template string right for somewhat complex log styles is a small task in itself. -- status: pending - open ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20345 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20345] Better logging defaults
Arne Babenhauserheide added the comment: (though calling logging.basicConfig() still produces some overhead, it does not pull me away from the task I’m currently doing) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20345 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19334] test_asyncio hanging for 1 hour (non-AIX version)
STINNER Victor added the comment: It was hanging on FreeBSD 7.2 recently: http://buildbot.python.org/all/builders/x86%20FreeBSD%207.2%203.x/builds/4917/steps/test/logs/stdio This is a new issue, test_read_pty_output() is a recent feature which came with a new test. The previous blocking test was test_signal_handling_while_selecting(). Please open a new issue. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19334 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20365] test_asyncio.test_read_pty_output() fails on AMD64 Snow Leop 3.x buildbot
New submission from STINNER Victor: kqueue doesn't support character devices like PTY before Mac OS 10.9. This issue is discussion on Tulip mailing list, no decision was taken yet how to fix this issue. http://buildbot.python.org/all/builders/AMD64%20Snow%20Leop%203.x/builds/1071/steps/test/logs/stdio == ERROR: test_read_pty_output (test.test_asyncio.test_events.KqueueEventLoopTests) -- Traceback (most recent call last): File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/selector_events.py, line 135, in add_reader key = self._selector.get_key(fd) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/selectors.py, line 181, in get_key raise KeyError({!r} is not registered.format(fileobj)) from None KeyError: '7 is not registered' During handling of the above exception, another exception occurred: Traceback (most recent call last): File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/test/test_asyncio/test_events.py, line 980, in test_read_pty_output self.loop.run_until_complete(connect()) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/base_events.py, line 177, in run_until_complete return future.result() File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/futures.py, line 236, in result raise self._exception File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/tasks.py, line 281, in _step result = next(coro) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/test/test_asyncio/test_events.py, line 974, in connect master_read_obj) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/base_events.py, line 537, in connect_read_pipe transport = self._make_read_pipe_transport(pipe, protocol, waiter) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/unix_events.py, line 149, in _make_read_pipe_transport return _UnixReadPipeTransport(self, pipe, protocol, waiter, extra) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/unix_events.py, line 200, in __init__ self._loop.add_reader(self._fileno, self._read_ready) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/asyncio/selector_events.py, line 138, in add_reader (handle, None)) File /Users/buildbot/buildarea/3.x.murray-snowleopard/build/Lib/selectors.py, line 462, in register self._kqueue.control([kev], 0, 0) OSError: [Errno 22] Invalid argument -- components: Tests keywords: buildbot messages: 208941 nosy: gvanrossum, haypo priority: normal severity: normal status: open title: test_asyncio.test_read_pty_output() fails on AMD64 Snow Leop 3.x buildbot versions: Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20365 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures
Glenn Langford added the comment: Comments on @Victor's comments - I will have a look at the dev guide. :-) I think you have identified another bug in the current code. Future._waiters is a list and so accept duplicated waiters. What this means is that the following code is broken, because it attempts to remove the Future twice from the pending set. for future in finished: yield future pending.remove(future) ## KeyError triggered on as_completed( [f,f] ) See attached test which demonstrates the breakage. Also, the behaviour of as_completed([f,f]) and wait([f,f], return_when=ALL_COMPLETED) is currently different. wait will only yield f once. -- Added file: http://bugs.python.org/file33656/test_dupfuture.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20297 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20336] test_asyncio: relax timings even more
Nick Coghlan added the comment: I have a draft PEP for the Zuul idea, and yes, it notes the increased consequences of declaring a buildbot stable when it sometimes isn't. There's a reason OpenStack treats third party test results as advisory only rather than as gating criteria :P -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20336 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Ram Rachum added the comment: Antoine: Which parts of the API merit discussion? The method names? Whether to include readlines/writelines? The arguments? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures
STINNER Victor added the comment: I think you have identified another bug in the current code. Please open a separated issue in this case. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20297 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20341] Argument Clinic: add nullable ints
Larry Hastings added the comment: I think it's a fine name, otherwise I would not have chosen it in the first place. Your high-spirited bikeshedding has been noted and not acted upon. I have no plans to change the name. Please drop it. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20341 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20366] SQLite FTS (full text search)
New submission from Mark Summerfield: This is a feature request that future versions of SQLite 3 that are bundled with Python 3 in the Windows binary packages (.msi files) has been build with the FTS4 (full text search version 4) enabled. -- components: Extension Modules messages: 208946 nosy: mark priority: normal severity: normal status: open title: SQLite FTS (full text search) type: enhancement versions: Python 3.4, Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20366 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19936] Executable permissions of Python source files
Serhiy Storchaka added the comment: Now findnoshebang.py shows only example files in the documentation, Windows-specific bat and pyw files, generated files, and the Modules/_ctypes/libffi/ltmain.sh file. I have not touched ltmain.sh file which have no shebang nor executable bits in the libffi distribution, because I'm not sure this is safe. Thanks all for reviews. -- resolution: - fixed stage: commit review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19936 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures
Changes by Glenn Langford glenn.langf...@gmail.com: Removed file: http://bugs.python.org/file33656/test_dupfuture.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20297 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Antoine Pitrou added the comment: Which parts of the API merit discussion? The method names? Whether to include readlines/writelines? The arguments? I think the readlines/writelines methods, as well as the size argument, should be dropped. Furthermore, there also should be distinct text/binary variants. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations
Steve added the comment: Hi Georg, Thanks again for the responses and your help. After a bit of research, I discovered the /reasons/ behind needing the \001 and \002 escapes. Thought I'd log the links here for posterity sake: - To color something in on a color capable terminal console you just need to use the \033[color codem escape sequence. This would be sufficient[1] - However readline messes up the line width calculation because it measures the escape sequences as a characters too. To avoid this you have to wrap the escape sequences within \001 and \002.[2] - On some terminal applications (like the one I am using - terminator[3]), if you add the \001 and \002 escapes to color text which is *not* interpreted by readline, (for instance if you have a single function to color text and you want to use it to color both your sys.ps1 and output text), the \001 and \002 codes will get printed out using a representation (like a unicode 'box'[4]). So, one would have to workaround that in the text coloring function. [1] http://en.wikipedia.org/wiki/ANSI_escape_code#Colors [2] bugs.python.org/issue17337/ and http://stackoverflow.com/questions/9468435/look-how-to-fix-column-calculation-in-python-readline-if-use-color-prompt [3] http://gnometerminator.blogspot.sg/p/introduction.html [4] http://en.wikipedia.org/wiki/Control_character#Display -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20359 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Ram Rachum added the comment: I see. I don't have an opinion about these 3 issues (readlines/writelines, size and binary separation.) So I'm cool with making these changes. If we do separate out the binary versions, what do you have in mind for the method names and signatures? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20341] Argument Clinic: add nullable ints
Antoine Pitrou added the comment: The only place where nullable appears in the doc tree is... clinic.rst (in reference to places where a C pointer can be NULL, which *is* a reasonable use of the term). There's another problem in the patch: if you define nullable_int_t and nullable_Py_ssize_t in a public header (longobject.h), then you probably need to prefix them with _Py (so: _Py_nullable_int_t and _Py_nullable_ssize_t, for example). Same for the #define's. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20341 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures
Changes by Glenn Langford glenn.langf...@gmail.com: Added file: http://bugs.python.org/file33657/test_dupfuture.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20297 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`
Antoine Pitrou added the comment: If we do separate out the binary versions, what do you have in mind for the method names and signatures? Perhaps we should look at what other Path APIs do here, and how they name it. Realistically, we need at least read_bytes() and read_text() methods. The write() method may stay unique if it's polymorphic. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20218 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures
Changes by Glenn Langford glenn.langf...@gmail.com: -- nosy: +haypo, mark.dickinson, tim.peters ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20367 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures
New submission from Glenn Langford: concurrent.futures.as_completed([f,f]) will yield f twice, then fail with a KeyError for a Future f which is not completed. If the Future has already completed, as_completed([f,f]) will yield f once and does not trigger an exception. What is the correct behaviour? as_completed( [f,f] ) - yield f twice ? wait( [f,f], return_when=ALL_COMPLETED ) - yield f twice ? -- components: Library (Lib) files: test_dupfuture.py messages: 208952 nosy: glangford priority: normal severity: normal status: open title: concurrent.futures.as_completed() fails when given duplicate Futures type: behavior versions: Python 3.3, Python 3.4 Added file: http://bugs.python.org/file33658/test_dupfuture.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20367 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20368] Tkinter: handle the null character
New submission from Serhiy Storchaka: Tcl/Tk uses modified UTF-8 encoding to represent strings as C strings (char*). Because C strings are NUL-terminated, the null character represented as illegal UTF-8 sequence \xc0\x80. Current Tkinter code is not very aware about this. It has special handling the \xc0\x80 string (i.e. encoded single null character) in one place, but doesn't handle encoded null character contained in larger string. As result Tkinter may truncate strings contained the null character, or return wrong result. The proposed patch fixes many issues with the null character (converting from Tcl to Python strings). NUL is still forbidden in string arguments of many methods. Also the patch enhances error handling for variable-related commands. -- components: Extension Modules, Tkinter files: tkinter_null_character.patch keywords: patch messages: 208954 nosy: gpolo, kbk, loewis, roger.serwy, serhiy.storchaka, terry.reedy priority: normal severity: normal stage: patch review status: open title: Tkinter: handle the null character type: behavior versions: Python 2.7, Python 3.3, Python 3.4 Added file: http://bugs.python.org/file33659/tkinter_null_character.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20368 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20358] test_curses is failing
Zachary Ware added the comment: I'm on Windows until this evening; I'll see what I can see when I'm on a machine that can actually build _curses. I don't see any failures on any of the buildbots, so I'm not sure how likely it is I'll be able to reproduce your failure. (And btw, that patch I have against test_curses is just a mechanical conversion to unittest, the only curses I'm an expert at are ones directed at problems...) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20358 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures
Glenn Langford added the comment: There is a subtlety in the as_completed() code which explains a lot - note that finished starts off as a set in the _AcquireFutures block. So if a Future f has already completed, as_completed( [f,f] ) will only yield f once, because f appears once in the finished set. Later on when waiter events are processed, finished turns into a list because of the line: finished = waiter.finished_futures So any duplicates in that list will cause problems in pending.remove(Future). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20367 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures
STINNER Victor added the comment: Since the new asyncio module has also a Future class and functions like as_completed(), this issue concerns also asyncio. concurrent.futures and asyncio should have the same behaviour. -- nosy: +gvanrossum ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20367 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20364] Rename explain sqlite3.Cursor.execute 'parameters' param
Chris Angelico added the comment: Small quibble: The last sentence capitalizes a Python built-in, which is confusing (Len(seq_dict) must match...). Tweak of grammar to have it not at the beginning of the sentence: Either way, len(seq_dict) must match -- nosy: +Rosuav ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20364 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19334] test_asyncio hanging for 1 hour (non-AIX version)
Stefan Krah added the comment: OK, I have not seen the original issue in any recent builds either. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19334 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue18695] os.statvfs() not working well with unicode paths
Giampaolo Rodola' added the comment: Either way it's fine with me. Regardless I think these tests have some value because all those os functions are not currently tested so it might makes sense to apply Serhiy's suggestions and port them to Python 3.4. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue18695 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20369] concurrent.futures.wait() blocks forever when given duplicate Futures
New submission from Glenn Langford: For a Future f which has already completed, wait( [f,f], return_when=ALL_COMPLETED ) blocks forever. This is because the test in wait(): if len(done) == len(fs) is comparing the length of a set to the length of a list. If f has not completed, wait( [f,f] ) will yield f once. The behaviour should be consistent with as_completed() - see issue #20367. -- files: test_dupfuture_wait.py messages: 208960 nosy: glangford, haypo, mark.dickinson, tim.peters priority: normal severity: normal status: open title: concurrent.futures.wait() blocks forever when given duplicate Futures versions: Python 3.3, Python 3.4 Added file: http://bugs.python.org/file33660/test_dupfuture_wait.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20369 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures
Guido van Rossum added the comment: I think you the caller was wrong to pass in [f, f] in the first place. In asyncio, the argument is converted into a set before using it, but there's still a bug if you pass it a list containing two references to the same coroutine -- it gets wrapped in two separate Futures. I think the better behavior is to convert to a set first and then map coroutines to Futures. So concurrent.futures should also convert to a set first. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20367 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20345] Better logging defaults
Vinay Sajip added the comment: Specifying the format is what requires mental investment (because for that I need to read up on the config options). Not sure what you mean by config options. Is it too much to remember the format= parameter in your first post? I'm not planning to change the default because of already stated backward compatibility concerns, as it's easy enough to specify what you want, and it's not as if this type of request has come from lots of people over the years that logging has been in use (basicConfig has been there in its present form since Python 2.4). -- status: open - pending ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20345 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20370] Docs error in Library reference-17.1.3 threading.Lock.acquire()
July added the comment: a sad story of a non-native English speaker,I just realized that it's me misunderstanding the sentence. well, close it. -- resolution: - invalid status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20370 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20370] Docs error in Library reference-17.1.3 threading.Lock.acquire()
New submission from July: In docs of Library reference-17.1.3 : threading.Lock.acquire(blocking=True,timeout=-1): line3 said: If a call with blocking set to True would block, return False immediately Here I suspect the blocking argument ought to be False as described subsequently. If it's set to True,it would block but return False immediately. P.S. first try of a non-native English speaker,do not care if any syntax error was made up there,please -- assignee: docs@python components: Documentation messages: 208962 nosy: docs@python, yonoho priority: normal severity: normal status: open title: Docs error in Library reference-17.1.3 threading.Lock.acquire() type: enhancement versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20370 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20172] Derby #3: Convert 67 sites to Argument Clinic across 4 files (Windows)
Changes by Zachary Ware zachary.w...@gmail.com: Added file: http://bugs.python.org/file33661/8eb334d4f4a2.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20172 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20365] test_asyncio.test_read_pty_output() fails on AMD64 Snow Leop 3.x buildbot
STINNER Victor added the comment: The issue was discussed here: https://groups.google.com/d/msg/python-tulip/I8oJ65Ww2Zw/tVL2ppXiU54J The decision is to skip the test on Mac OS X 10.9. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20365 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20365] test_asyncio.test_read_pty_output() fails on AMD64 Snow Leop 3.x buildbot
Roundup Robot added the comment: New changeset 424564cffad0 by Victor Stinner in branch 'default': Close #20365: Skip test_asyncio.test_events.test_read_pty_output() on Mac OS X http://hg.python.org/cpython/rev/424564cffad0 -- nosy: +python-dev resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20365 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19920] TarFile.list() fails on some files
Serhiy Storchaka added the comment: I have added new portion of nitpicks on Rietveld. For you questions: 1. Yes, we can add unencodable tarinfo.linkname later. Just add tests for external tar files. 2. Use support.captured_stdout() (see test_list_command_verbose). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19920 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com