[issue20185] Derby #17: Convert 49 sites to Argument Clinic across 13 files

2014-01-23 Thread Vajrasky Kok
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 __

[issue20341] Argument Clinic: add "nullable ints"

2014-01-23 Thread Larry Hastings
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, beca

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
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. -- keyw

[issue20358] test_curses is failing

2014-01-23 Thread Larry Hastings
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

[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files

2014-01-23 Thread Vajrasky Kok
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 __

[issue20355] -W command line options should have higher priority than PYTHONWARNINGS environmental variable

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mai

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Ram Rachum
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`. -- A

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Steve
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 pro

[issue20358] test_curses is failing

2014-01-23 Thread Nadeem Vawda
Changes by Nadeem Vawda : -- nosy: +nadeem.vawda ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Georg Brandl
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 ___

[issue20358] test_curses is failing

2014-01-23 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:

[issue20360] inspect.Signature could provide readable expressions for default values for builtins

2014-01-23 Thread Larry Hastings
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

[issue20338] Idle: increase max calltip width

2014-01-23 Thread Terry J. Reedy
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

[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis : -- stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
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: B

[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files

2014-01-23 Thread Vajrasky Kok
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 __

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Steve
Steve added the comment: wow, that was super quick ! Yes, indeed. Surrounding the escape sequences with '\0x1' & '\0x2' fixes this issue. Thanks ! -- ___ Python tracker ___

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Steve
Steve added the comment: ...of course I meant \x01 and \x02, like you suggested -- ___ Python tracker ___ ___ Python-bugs-list mailing

[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
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

[issue20362] longMessage attribute is ignored in unittest.TestCase.assertRegexpMatches etc

2014-01-23 Thread Dhara
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

[issue20338] Idle: increase max calltip width

2014-01-23 Thread Serhiy Storchaka
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 ___ __

[issue20338] Idle: increase max calltip width

2014-01-23 Thread Serhiy Storchaka
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 ___

[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files

2014-01-23 Thread Vajrasky Kok
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 __

[issue20361] -W command line options and PYTHONWARNINGS environmental variable should not override -b / -bb command line options

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis : -- dependencies: +BytesWarnings triggerred by test suite ___ Python tracker ___ ___ P

[issue20363] BytesWarnings triggerred by test suite

2014-01-23 Thread Arfrever Frehtes Taifersar Arahesis
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 instan

[issue20352] Add support for AUTH command to poplib

2014-01-23 Thread Daniël van Eeden
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

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Steve
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@lo

[issue20336] test_asyncio: relax timings even more

2014-01-23 Thread Stefan Krah
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

[issue20173] Derby #4: Convert 53 sites to Argument Clinic across 5 files

2014-01-23 Thread Vajrasky Kok
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

[issue20336] test_asyncio: relax timings even more

2014-01-23 Thread STINNER Victor
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

[issue20364] Rename & explain sqlite3.Cursor.execute 'parameters' param

2014-01-23 Thread Terry J. Reedy
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 s

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Georg Brandl
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

[issue20336] test_asyncio: relax timings even more

2014-01-23 Thread Stefan Krah
Stefan Krah added the comment: I see. That's astonishing, because usually the builds are ultra-fast on those machines. -- nosy: -gvanrossum ___ Python tracker ___ __

[issue20336] test_asyncio: relax timings even more

2014-01-23 Thread STINNER Victor
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 betwee

[issue20336] test_asyncio: relax timings even more

2014-01-23 Thread Stefan Krah
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

[issue20345] Better logging defaults

2014-01-23 Thread Vinay Sajip
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'

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread Mark Dickinson
Changes by Mark Dickinson : -- nosy: +mark.dickinson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread Mark Dickinson
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). -- ___

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread Mark Dickinson
Changes by Mark Dickinson : -- nosy: +tim.peters ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue19863] Missing function attributes in 2.7 docs.

2014-01-23 Thread Mark Dickinson
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 ___ __

[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures

2014-01-23 Thread Glenn Langford
Changes by Glenn Langford : -- nosy: +mark.dickinson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread STINNER Victor
Changes by STINNER Victor : -- nosy: +sbt ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.o

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread STINNER Victor
STINNER Victor added the comment: @Glenn: Can you maybe suggest a patch fixing the issue? -- nosy: +haypo ___ Python tracker ___ ___ P

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread Glenn Langford
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

[issue20362] longMessage attribute is ignored in unittest.TestCase.assertRegexpMatches etc

2014-01-23 Thread Berker Peksag
Changes by Berker Peksag : -- nosy: +ezio.melotti, michael.foord ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue20341] Argument Clinic: add "nullable ints"

2014-01-23 Thread Antoine Pitrou
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

[issue20319] concurrent.futures.wait() can block forever even if Futures have completed

2014-01-23 Thread STINNER Victor
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

[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures

2014-01-23 Thread STINNER Victor
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

[issue19291] Add docs for asyncio package (Tulip, PEP 3156)

2014-01-23 Thread STINNER Victor
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. --

[issue20352] Add support for AUTH command to poplib

2014-01-23 Thread R. David Murray
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 trac

[issue20313] inspect.signature should raise ValueError for builtins with no signature info

2014-01-23 Thread Larry Hastings
Larry Hastings added the comment: I've worked your changes into the patch for #20189. -- ___ Python tracker ___ ___ Python-bugs-list m

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Christopher Welborn
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

[issue20311] epoll.poll(timeout) and PollSelector.select(timeout) must round the timeout to the upper bound

2014-01-23 Thread Roundup Robot
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

[issue20364] Rename & explain sqlite3.Cursor.execute 'parameters' param

2014-01-23 Thread R. David Murray
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

[issue20311] epoll.poll(timeout) and PollSelector.select(timeout) must round the timeout to the upper bound

2014-01-23 Thread STINNER Victor
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) ---

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Ram Rachum
Changes by Ram Rachum : Removed file: http://bugs.python.org/file33647/pathlib.readwrite2.patch ___ Python tracker ___ ___ Python-bugs-list ma

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Ram Rachum
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 ___ __

[issue19334] test_asyncio hanging for 1 hour (non-AIX version)

2014-01-23 Thread STINNER Victor
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

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Ram Rachum
Changes by Ram Rachum : Removed file: http://bugs.python.org/file33616/patch.patch ___ Python tracker ___ ___ Python-bugs-list mailing list Un

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Antoine Pitrou
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

[issue19936] Executable permissions of Python source files

2014-01-23 Thread Roundup Robot
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: Restor

[issue20360] inspect.Signature could provide readable expressions for default values for builtins

2014-01-23 Thread Nick Coghlan
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

[issue19334] test_asyncio hanging for 1 hour (non-AIX version)

2014-01-23 Thread Stefan Krah
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 tim

[issue20189] inspect.Signature doesn't recognize all builtin types

2014-01-23 Thread Larry Hastings
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 na

[issue20345] Better logging defaults

2014-01-23 Thread Arne Babenhauserheide
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

[issue20345] Better logging defaults

2014-01-23 Thread Arne Babenhauserheide
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 __

[issue19334] test_asyncio hanging for 1 hour (non-AIX version)

2014-01-23 Thread STINNER Victor
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 tes

[issue20365] test_asyncio.test_read_pty_output() fails on "AMD64 Snow Leop 3.x" buildbot

2014-01-23 Thread STINNER Victor
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/lo

[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures

2014-01-23 Thread Glenn Langford
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

[issue20336] test_asyncio: relax timings even more

2014-01-23 Thread Nick Coghlan
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 --

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Ram Rachum
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

[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures

2014-01-23 Thread STINNER Victor
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 ___ __

[issue20341] Argument Clinic: add "nullable ints"

2014-01-23 Thread Larry Hastings
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. -- ___ Pyt

[issue20366] SQLite FTS (full text search)

2014-01-23 Thread Mark Summerfield
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: 2089

[issue19936] Executable permissions of Python source files

2014-01-23 Thread Serhiy Storchaka
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

[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures

2014-01-23 Thread Glenn Langford
Changes by Glenn Langford : Removed file: http://bugs.python.org/file33656/test_dupfuture.py ___ Python tracker ___ ___ Python-bugs-list maili

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Antoine Pitrou
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/bina

[issue20359] Having escape sequences (like color codes) in the sys.ps1 messes up readline's input line calculations

2014-01-23 Thread Steve
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 j

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Ram Rachum
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? -- ___

[issue20341] Argument Clinic: add "nullable ints"

2014-01-23 Thread Antoine Pitrou
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_ssi

[issue20297] concurrent.futures.as_completed() installs waiters for already completed Futures

2014-01-23 Thread Glenn Langford
Changes by Glenn Langford : Added file: http://bugs.python.org/file33657/test_dupfuture.py ___ Python tracker ___ ___ Python-bugs-list mailing

[issue20218] Add `pathlib.Path.write` and `pathlib.Path.read`

2014-01-23 Thread Antoine Pitrou
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

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

2014-01-23 Thread Glenn Langford
Changes by Glenn Langford : -- nosy: +haypo, mark.dickinson, tim.peters ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

2014-01-23 Thread Glenn Langford
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 beh

[issue20368] Tkinter: handle the null character

2014-01-23 Thread Serhiy Storchaka
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 handli

[issue20358] test_curses is failing

2014-01-23 Thread Zachary Ware
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 h

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

2014-01-23 Thread Glenn Langford
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

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

2014-01-23 Thread STINNER Victor
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 ___ Pytho

[issue20364] Rename & explain sqlite3.Cursor.execute 'parameters' param

2014-01-23 Thread Chris Angelico
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 _

[issue19334] test_asyncio hanging for 1 hour (non-AIX version)

2014-01-23 Thread Stefan Krah
Stefan Krah added the comment: OK, I have not seen the original issue in any recent builds either. -- ___ Python tracker ___ ___ Pytho

[issue18695] os.statvfs() not working well with unicode paths

2014-01-23 Thread Giampaolo Rodola'
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. -- ___

[issue20369] concurrent.futures.wait() blocks forever when given duplicate Futures

2014-01-23 Thread Glenn Langford
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( [

[issue20367] concurrent.futures.as_completed() fails when given duplicate Futures

2014-01-23 Thread Guido van Rossum
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 s

[issue20345] Better logging defaults

2014-01-23 Thread Vinay Sajip
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

[issue20370] Docs error in Library reference-17.1.3 threading.Lock.acquire()

2014-01-23 Thread July
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

[issue20370] Docs error in Library reference-17.1.3 threading.Lock.acquire()

2014-01-23 Thread July
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

[issue20172] Derby #3: Convert 67 sites to Argument Clinic across 4 files (Windows)

2014-01-23 Thread Zachary Ware
Changes by Zachary Ware : Added file: http://bugs.python.org/file33661/8eb334d4f4a2.diff ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue20365] test_asyncio.test_read_pty_output() fails on "AMD64 Snow Leop 3.x" buildbot

2014-01-23 Thread STINNER Victor
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

[issue20365] test_asyncio.test_read_pty_output() fails on "AMD64 Snow Leop 3.x" buildbot

2014-01-23 Thread Roundup Robot
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/

[issue19920] TarFile.list() fails on some files

2014-01-23 Thread Serhiy Storchaka
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). -- ___

  1   2   3   >