[issue28483] python --version prints output to stderr

2016-10-20 Thread Thomas Becker
New submission from Thomas Becker: I am using Python 2.7.10 on Windows 8. While writing some code in NodeJS I noticed the following behaviour: `python --version` prints its output to stderr instead of stdout. In most cases this won't make any difference because both streams print to the scre

[issue28483] python --version prints output to stderr

2016-10-20 Thread SilentGhost
Changes by SilentGhost : -- resolution: -> duplicate stage: -> resolved status: open -> closed superseder: -> Python -V and --version output to stderr instead of stdout ___ Python tracker ___

[issue28478] Built-in module 'Time' does not enable functions if -Wno-error specified in the build environment

2016-10-20 Thread SilentGhost
Changes by SilentGhost : -- components: +Build type: -> behavior versions: +Python 3.6 ___ Python tracker ___ ___ Python-bugs-list ma

[issue28484] test_capi, test_regrtest fail when multithreading disabled

2016-10-20 Thread Martin Panter
New submission from Martin Panter: Two tests in test_capi assume that the GIL is always used, but I don’t think it is available when Python is built with “configure --without-threads”. So I propose to skip these tests if the “threading” module is unavailable. In test_regrtest, a test is rerun

[issue28485] compileall.compile_dir(workers=) does not raise ValueError if multithreading disabled

2016-10-20 Thread Martin Panter
New submission from Martin Panter: According to the documentation and tests, a negative value for the “workers” parameter to compileall.compile_dir() should trigger ValueError. But if Python is built with “configure --without-threads”, the parameter is not checked. So I propose a patch to fix

[issue28480] Compile error on Modules/socketmodule.c

2016-10-20 Thread Martin Panter
Martin Panter added the comment: I also committed a similar but independent fix in Python 2.7 building Modules/_sqlite/connection.c, caused by revision 649937bb8f1c, and adjusted some tests to work when multithreading is disabled. For the record, I also opened Issue 28482, Issue 28484 and Issu

[issue28478] Built-in module 'time' does not enable functions if -Werror specified in the build environment

2016-10-20 Thread Martin Panter
Changes by Martin Panter : -- title: Built-in module 'Time' does not enable functions if -Wno-error specified in the build environment -> Built-in module 'time' does not enable functions if -Werror specified in the build environment ___ Python track

[issue28484] test_capi, test_regrtest fail when multithreading disabled

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: Each time I see a bug error on --without-treads, I want to remove the option. We had a buildbot in the past, but I don't see it anymore. mt-victor.patch LGTM except of a minor comment, see my review. -- ___ Python t

[issue21955] ceval.c: implement fast path for integers with a single digit

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: Between inline2.patch and fastint6.patch, it seems like inline2.patch is faster (between 9% and 12% faster than fastint6.patch). Microbenchmark on Python default (rev 554fb699af8c), compilation using LTO (./configure --with-lto), GCC 6.2.1 on Fedora 24, Intel(

[issue21955] ceval.c: implement fast path for integers with a single digit

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: Result of performance 0.3.3 (and perf 0.8.3). No major benchmark is faster. A few benchmarks seem to be event slower using fastint6.patch (but I don't really trust pybench). == fastint6.patch == $ python3 -m perf compare_to master.json fastint6.json --group-

[issue21955] ceval.c: implement fast path for integers with a single digit

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: fastint6_inline2_json.tar.gz: archive of JSON files - fastint6.json - inline2.json - master.json - timeit-fastint6.json - timeit-inline2.json - timeit-master.json -- Added file: http://bugs.python.org/file45150/fastint6_inline2_json.tar.gz

[issue28486] Wrong end index and subgroup for group match

2016-10-20 Thread Pavel Cisar
Changes by Pavel Cisar : -- versions: +Python 3.5 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.

[issue28486] Wrong end index and subgroup for group match

2016-10-20 Thread Pavel Cisar
New submission from Pavel Cisar: Hi, python re returns wrong end index of searched group and also subgroup itself. Example: In: price_string = u"1 307 000,00 Kč" In: match = re.search(r"([,\.]00)\s?.*$", price_string) In: print price_string, "|", match.groups(), "|", match.group(0), "|", match

[issue28486] Wrong end index and subgroup for group match

2016-10-20 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: group(0) returns the whole match. group(1) returns the first captured subgroup. -- nosy: +serhiy.storchaka resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker

[issue21955] ceval.c: implement fast path for integers with a single digit

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: The fatest patch (inline2.patch) has a negligible impact on benchmarks. The purpose of an optimization is to make Python faster, it's not the case here, so I close the issue. Using timeit, the largest speedup is 1.29x faster. Using performance, spectral_norm

[issue21955] ceval.c: implement fast path for integers with a single digit

2016-10-20 Thread STINNER Victor
Changes by STINNER Victor : -- resolution: fixed -> rejected ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue28444] Missing extensions modules when cross compiling python 3.5.2 for arm on Linux

2016-10-20 Thread Xavier de Gaye
Changes by Xavier de Gaye : -- nosy: +akuchling ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.py

[issue21955] ceval.c: implement fast path for integers with a single digit

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset 61fcb12a9873 by Victor Stinner in branch 'default': Issue #21955: Please don't try to optimize int+int https://hg.python.org/cpython/rev/61fcb12a9873 -- nosy: +python-dev ___ Python tracker

[issue27659] Check for the existence of crypt()

2016-10-20 Thread Xavier de Gaye
Changes by Xavier de Gaye : -- versions: +Python 3.7 -Python 3.6 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue28487] missing _math.o target in 2.7 Makefile

2016-10-20 Thread Skip Montanaro
New submission from Skip Montanaro: Unless I have my hg repo dependencies incorrect, this target seems to be missing from the 2.7 Makefile.pre.in: # This is shared by the math and cmath modules Modules/_math.o: Modules/_math.c Modules/_math.h $(CC) -c $(CCSHARED) $(PY_CORE_CFLAGS) -o $@

[issue28487] missing _math.o target in 2.7 Makefile

2016-10-20 Thread Skip Montanaro
Skip Montanaro added the comment: Is this perhaps related to the (closed) issue 24421? -- ___ Python tracker ___ ___ Python-bugs-list

[issue28487] missing _math.o target in 2.7 Makefile

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: > Unless I have my hg repo dependencies incorrect, this target seems to be > missing from the 2.7 Makefile.pre.in: The math module is compiled by setup.py: # math library functions, e.g. sin() exts.append( Extension('math', ['mathmodule.c'],

[issue28488] shutil.make_archive (xxx, zip, root_dir) is adding './' entry to archive which is wrong

2016-10-20 Thread Alexander Belchenko
New submission from Alexander Belchenko: Running shutil.make_archive('a', 'zip', 'subdir') is created wrong and not really needed entry "./" which is visible in zipfile.ZipFile.namelist(): ['./', 'foo/', 'hello.txt', 'foo/bar.txt'] This "./" affects some (windows) unzip tools which produces wa

[issue28488] shutil.make_archive (xxx, zip, root_dir) is adding './' entry to archive which is wrong

2016-10-20 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- nosy: +alanmcintyre, serhiy.storchaka, tarek, twouters stage: -> needs patch type: -> behavior versions: +Python 3.5, Python 3.6, Python 3.7 -Python 3.4 ___ Python tracker ___

[issue27910] Doc/library/traceback.rst — references to tuples should be replaced with new FrameSummary object

2016-10-20 Thread Petr Viktorin
Petr Viktorin added the comment: ping Anything I can do to move this forward? -- ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue28474] WinError(): Python int too large to convert to C long

2016-10-20 Thread Steve Dower
Steve Dower added the comment: All HRESULT values are, since the topmost bit indicates that it's an error, but all the others should be 16-bit positive integers IIRC. I don't think this function is meant to work with HRESULTs, but I could be wrong - fairly sure it's meant for Win32 error codes

[issue27910] Doc/library/traceback.rst — references to tuples should be replaced with new FrameSummary object

2016-10-20 Thread Emanuel Barry
Emanuel Barry added the comment: This is not a regression, the documentation was just not fully updated when the new feature was added. Patch looks good. This should probably be applied to the 3.5 branch as well. -- nosy: +ebarry stage: patch review -> commit review versions: +Python 3

[issue28482] test_typing fails if asyncio unavailable

2016-10-20 Thread Guido van Rossum
Guido van Rossum added the comment: Can you submit this as a PR upstream, to github.com/python/typing? -- ___ Python tracker ___ ___ P

[issue28447] socket.getpeername() failure on broken TCP/IP connection

2016-10-20 Thread R. David Murray
R. David Murray added the comment: Unless I'm missing something, this indicates that the problem is that once the far end closes, Windows will no longer return the peer name. And, unless I'm misreading, the behavior will be the same on Unix. The man page for getpeername says that ENOTCONN i

[issue27593] Deprecate sys._mercurial and create sys._git

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Hello Brett, after my review, I propose this patch where I have added the missing indents and remove the ./configure file (because this one can be generated by autoreconf) -- nosy: +matrixise Added file: http://bugs.python.org/file45152/issue27593-wi

[issue28487] missing _math.o target in 2.7 Makefile

2016-10-20 Thread Skip Montanaro
Skip Montanaro added the comment: I'm not sure you understood my problem. Starting with make distclean ./configure make the math and cmath modules fail to build because Modules/_math.o is missing. It's not a make thing. I did notice the cpython Makefile has a target for Modules/_math.o and ass

[issue28487] missing _math.o target in 2.7 Makefile

2016-10-20 Thread Zachary Ware
Zachary Ware added the comment: Your HG setup sounds fine, though you may be interested in the 'share' extension. It allows you to share the data store between several checkouts such that doing a 'pull' in one checkout makes any new changesets available in all checkouts without having to do a

[issue28092] Build failure for 3.6 on Centos 5.11

2016-10-20 Thread Robin Becker
Robin Becker added the comment: for what it's worth I tried reverse patching https://hg.python.org/cpython/rev/63ae310b60ff/ and https://hg.python.org/cpython/rev/2f77a9f0b9d6/ in the manylinux docker and the make then proceeds fine with one warning at the end *** WARNING: renaming "_sqlite

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-10-20 Thread Justin Foo
Justin Foo added the comment: Is #24781 likely to make it into Python 3.6? Otherwise, would this patch be of any benefit in the meantime? -- ___ Python tracker ___ _

[issue28488] shutil.make_archive (xxx, zip, root_dir) is adding './' entry to archive which is wrong

2016-10-20 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Thank you for your report Alexander. Yes, this regression was introduced in issue24982. Proposed patch fixes it. -- assignee: -> serhiy.storchaka keywords: +patch stage: needs patch -> patch review Added file: http://bugs.python.org/file45153/shutil

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-10-20 Thread Terry J. Reedy
Terry J. Reedy added the comment: I have not been working on IDLE for the last month, but may get back to it. Patched like this can go in any release, not just .0 version releases. (See PEP 434 for why.) -- ___ Python tracker

[issue27593] Deprecate sys._mercurial and create sys._git

2016-10-20 Thread Brett Cannon
Brett Cannon added the comment: The inclusion of changes to ./configure is on purpose since the file is in the repository and not everyone has autoconf installed to necessarily re-generate the file (and we have had issues with people using really old versions of autoconf in the past). ---

[issue27593] Deprecate sys._mercurial and create sys._git

2016-10-20 Thread Ned Deily
Ned Deily added the comment: I'll get to this one shortly. -- assignee: brett.cannon -> ned.deily ___ Python tracker ___ ___ Python-bu

[issue27593] Deprecate sys._mercurial and create sys._git

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Hi Brett, With your comment, I have added the modified ./configure. Here is the last patch including the indentation. -- Added file: http://bugs.python.org/file45154/issue27593-with-indent-3.7-2.diff ___ Python tra

[issue28474] WinError(): Python int too large to convert to C long

2016-10-20 Thread Eryk Sun
Eryk Sun added the comment: Kelvin is calling WlanScan [1], which returns an error code that apparently can include HRESULT (signed) values cast as DWORD (unsigned) values, including 0x80342002 (ERROR_NDIS_DOT11_POWER_STATE_INVALID). ctypes.FormatError calls FormatMessage [2] with the flag F

[issue24381] Got warning when compiling ffi.c on Mac

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: @victor There are two issues. 1. The bundled version of CFFI is just outdated 2. There is an issue for the removal of libffi_osx (http://bugs.python.org/issue27979) so, we can fix the bundled libcffi or close this issue. --

[issue26010] document CO_* constants

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Hello Yury, I have updated your patch, with a small description for CO_ASYNC_GENERATOR. I need a review for the description. Stephane -- Added file: http://bugs.python.org/file45155/issue26010-2.diff ___ Python tr

[issue24381] Got warning when compiling ffi.c on Mac

2016-10-20 Thread Zachary Ware
Zachary Ware added the comment: Correction: #27979 is unrelated to Modules/_ctypes/libffi_osx. A hypothetical (at this point) removal of libffi_osx will be a separate issue. I think it's probably worthwhile to fix this issue. -- nosy: +zach.ware _

[issue26010] document CO_* constants

2016-10-20 Thread Stéphane Wirtel
Changes by Stéphane Wirtel : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscrib

[issue28214] Improve exception reporting for problematic __set_name__ attributes

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Serhiy, your patch works fine. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue26010] document CO_* constants

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset 681924a9eefd by Yury Selivanov in branch '3.5': Issue #26010: Document CO_* constants https://hg.python.org/cpython/rev/681924a9eefd New changeset 5023c182a8a4 by Yury Selivanov in branch '3.6': Merge 3.5 + document CO_ASYNC_GENERATOR; issue #26010

[issue26010] document CO_* constants

2016-10-20 Thread Yury Selivanov
Yury Selivanov added the comment: Thank you Stephane for bumping this issue up. I've committed the patch. I've documented CO_ASYNC_GENERATOR in more detail, and I also added "versionadded" tags to CO_COROUTINE and CO_ITERABLE_COROUTINE. -- resolution: -> fixed __

[issue26010] document CO_* constants

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Nice Yury, sorry for the missing versionadded directive. And now, I have a description for CO_ASYNC_GENERATOR ;-) -- ___ Python tracker ___ __

[issue26010] document CO_* constants

2016-10-20 Thread Stéphane Wirtel
Changes by Stéphane Wirtel : -- status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue24381] Got warning when compiling ffi.c on Mac

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: @zach.ware so in this case, we could merge the patch in 3.6 & 3.7. -- ___ Python tracker ___ ___ Py

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Ryan Gonzalez
New submission from Ryan Gonzalez: Attached is a little fix for a comment in tokenizer.c. I noticed that it was never updated for the inclusion of f-strings. -- assignee: docs@python components: Documentation files: 0001-Fix-comment-in-tokenizer.c.patch keywords: patch messages: 279056

[issue18219] csv.DictWriter is slow when writing files with large number of columns

2016-10-20 Thread Hugh Brown
Hugh Brown added the comment: I came across this problem today when I was using a 1000+ column CSV from a client. It was taking about 15 minutes to process each file. I found the problem and made this change: # wrong_fields = [k for k in rowdict if k not in self.fieldnames]

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Thank you for this patch. -- nosy: +matrixise versions: +Python 3.7 ___ Python tracker ___ ___ Pyth

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Eric V. Smith
Eric V. Smith added the comment: I'd actually change this to be something like: Process b"", r"", u"", and the various legal combinations. There are 24 total combinations when you add upper case. I actually wrote a script in Lib/tokenize.py to generate them all. And when I add binary f-string

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Eric V. Smith
Eric V. Smith added the comment: FWIW, in Lib/tokenize.py, it's _all_string_prefixes(): >>> _all_string_prefixes() set(['', 'FR', 'rB', 'rF', 'BR', 'Fr', 'RF', 'rf', 'RB', 'fr', 'B', 'rb', 'F', 'Br', 'R', 'U', 'br', 'fR', 'b', 'f', 'Rb', 'Rf', 'r', 'u', 'bR']) My basic point is that trying to

[issue28490] inappropriate OS.Error "Invalid cross-device link"

2016-10-20 Thread Steve Newcomb
tly due to the fact that a file already exists at the target path: (Pdb) os.path.isfile( '/persist/nobackup/backupDisks/d38BasLijPupBak/d38-backup.20161020/d38-_,.,_home2_,.,_rack/.Xauthority') True (Pdb) os.path.isfile( '/persist/nobackup/backupDisks/d38BasLijPupBak/d38-201610

[issue18219] csv.DictWriter is slow when writing files with large number of columns

2016-10-20 Thread SilentGhost
Changes by SilentGhost : -- stage: -> commit review versions: +Python 3.5, Python 3.6, Python 3.7 -Python 3.4 ___ Python tracker ___

[issue28490] inappropriate OS.Error "Invalid cross-device link"

2016-10-20 Thread Steve Newcomb
Steve Newcomb added the comment: Oops. My bad. There was a symlink in one of those paths. The message is valid. -- resolution: -> not a bug status: open -> closed ___ Python tracker ___

[issue28490] inappropriate OS.Error "Invalid cross-device link"

2016-10-20 Thread Eryk Sun
Changes by Eryk Sun : -- stage: -> resolved ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue28491] Remove bundled libffi for OSX

2016-10-20 Thread Zachary Ware
New submission from Zachary Ware: Here's a patch that at least allows _ctypes to be built against a Homebrew libffi and pass the test_ctypes. It is almost certainly not a complete patch, but may serve as a basis for something that will actually work. -- components: Build, ctypes, macO

[issue28491] Remove bundled libffi for OSX

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: I will test on my OSX (El Capitan) -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue28491] Remove bundled libffi for OSX

2016-10-20 Thread Chi Hsuan Yen
Chi Hsuan Yen added the comment: With this change pkg-config becomes mandatory on macOS, too. As Ned has mentioned in issue28207, pkg-config is not pre-installed with macOS. -- ___ Python tracker _

[issue20825] containment test for "ip_network in ip_network"

2016-10-20 Thread James Schneider
James Schneider added the comment: Please consider for implementation in 3.6. I'd love it even more for 3.5 but I don't think that will happen. With the latest patch, I don't believe there are any backwards-incompatible changes, though. -- ___ Pytho

[issue28491] Remove bundled libffi for OSX

2016-10-20 Thread Ned Deily
Ned Deily added the comment: Yes, we shouldn't be depending on pkg-config being available. I am not at all keen on adding a dependency on a third-party library supplied by another distributor. What I would like to see is: (1) add libffi to the third-party libs built and used for the macOS ins

[issue25953] re fails to identify invalid numeric group references in replacement strings

2016-10-20 Thread SilentGhost
SilentGhost added the comment: Updated patch taking Serhiy's comments into account. There was another case on line 725 for when zero is used as a group number, I'm not sure though if it falls within the scope of this issue, so it's not included in the current patch. -- versions: -Pyt

[issue28491] Remove bundled libffi for OSX

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: On my OSX with El Capitan and HomeBrew Configuration: env PKG_CONFIG_PATH=/usr/local/opt/libffi/lib/pkgconfig ./configure --prefix=$PWD-build Compilation: make Tests: ./python.exe -c 'import ctypes' Results: * No SIGSEGV * Add a new dependency to pkg-con

[issue28492] C implementation of asyncio.Future doesn't set value of StopIteration correctly

2016-10-20 Thread Yury Selivanov
New submission from Yury Selivanov: Specifically when the result of the Future is tuple. -- components: asyncio messages: 279070 nosy: gvanrossum, inada.naoki, yselivanov priority: normal severity: normal status: open title: C implementation of asyncio.Future doesn't set value of StopIte

[issue24381] Got warning when compiling ffi.c on Mac

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset 2b089035a453 by Ned Deily in branch '3.5': Issue #24381: Avoid unused function warning when building bundled macOS libffi. https://hg.python.org/cpython/rev/2b089035a453 New changeset c60d41590054 by Ned Deily in branch '3.6': Issue #24381: merge fr

[issue24381] Got warning when compiling ffi.c on Mac

2016-10-20 Thread Ned Deily
Ned Deily added the comment: Thank you for the patch! Now fixed in the bundled versions of libffi for OS X. -- resolution: -> fixed stage: commit review -> resolved status: open -> closed versions: +Python 2.7, Python 3.7 ___ Python tracker

[issue28492] C implementation of asyncio.Future doesn't set value of StopIteration correctly

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset cfe2109ce2c0 by Yury Selivanov in branch '3.6': Issue #28492: Fix how StopIteration is raised in _asyncio.Future https://hg.python.org/cpython/rev/cfe2109ce2c0 New changeset 79b9257f3386 by Yury Selivanov in branch 'default': Merge 3.6 (issue #28492

[issue28492] C implementation of asyncio.Future doesn't set value of StopIteration correctly

2016-10-20 Thread Yury Selivanov
Yury Selivanov added the comment: Inada-san, please take a look at my commit if you have time. -- resolution: -> fixed stage: -> commit review status: open -> closed type: -> behavior ___ Python tracker

[issue26240] Docstring of the subprocess module should be cleaned up

2016-10-20 Thread Tim Mitchell
Tim Mitchell added the comment: Changes as per Martins review. -- Added file: http://bugs.python.org/file45159/subprocess3.patch ___ Python tracker ___ __

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Ryan Gonzalez
Ryan Gonzalez added the comment: @eric.smith How's this instead? -- Added file: http://bugs.python.org/file45160/0001-Fix-comment-in-tokenizer.c.patch ___ Python tracker ___ ___

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Ryan Gonzalez
Ryan Gonzalez added the comment: Ugh, hit Submit too soon. I meant to say the patch that has 20:13 as the date (I should've probably given them different names...). -- ___ Python tracker __

[issue28448] C implemented Future doesn't work on Windows

2016-10-20 Thread Yury Selivanov
Yury Selivanov added the comment: Latest patch looks good. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue28493] Typo in _asynciomodule.c

2016-10-20 Thread Stéphane Wirtel
New submission from Stéphane Wirtel: a small fix for a typo. -- assignee: yselivanov components: Interpreter Core files: _asynciomodule.diff keywords: patch messages: 279079 nosy: matrixise, yselivanov priority: normal severity: normal status: open title: Typo in _asynciomodule.c version

[issue28493] Typo in _asynciomodule.c

2016-10-20 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: sorry, I can't commit it in the default branch, so I pass via an issue for this kind of issue. -- ___ Python tracker ___ _

[issue26010] document CO_* constants

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset 761414979de5 by Yury Selivanov in branch '3.6': Issue #26010: fix typos; rewording https://hg.python.org/cpython/rev/761414979de5 New changeset 3821599fc74d by Yury Selivanov in branch 'default': Merge 3.6 (issue #26010) https://hg.python.org/cpytho

[issue28493] Typo in _asynciomodule.c

2016-10-20 Thread Yury Selivanov
Yury Selivanov added the comment: Thanks, Stéphane! Fixed. -- resolution: -> fixed stage: -> resolved status: open -> closed type: -> enhancement ___ Python tracker ___ __

[issue28493] Typo in _asynciomodule.c

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset 03528baa8c2c by Yury Selivanov in branch '3.6': Issue #28493: Fix typos in _asynciomodule.c https://hg.python.org/cpython/rev/03528baa8c2c New changeset 74eb9c51d64e by Yury Selivanov in branch 'default': Merge 3.6 (issue #28493) https://hg.python.o

[issue28494] is_zipfile false positives

2016-10-20 Thread Thomas Waldmann
New submission from Thomas Waldmann: zipfile.is_zipfile has false positives way too easily. I just have seen it in practive when a MoinMoin wiki site with a lot of pdf attachments crashed with 500. This was caused by a valid PDF that just happened to contain PK\005\006 somewhere in the middle

[issue28494] is_zipfile false positives

2016-10-20 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- nosy: +alanmcintyre, serhiy.storchaka, twouters versions: +Python 3.6, Python 3.7 ___ Python tracker ___

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Eric V. Smith
Eric V. Smith added the comment: That's great. Thanks! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https

[issue28459] _pyio module broken on Cygwin / setmode not usable

2016-10-20 Thread Masayuki Yamamoto
Masayuki Yamamoto added the comment: I agree to use setmode() at _pyio module for Cygwin. However, I have two reasons that I disagree to the implementation of os.setmode(). First, FreeBSD has another setmode() that operate file permission bits [1]. Therefore the implementation of os.setmode() w

[issue28092] Build failure for 3.6 on Centos 5.11

2016-10-20 Thread Ned Deily
Ned Deily added the comment: "*** WARNING: renaming "_sqlite3" since importing it failed: build/lib.linux-x86_64-3.6/_sqlite3.cpython-36m-x86_64-linux-gnu.so: undefined symbol: sqlite3_stmt_readonly" That's a different issue: most likely you are building with an old version of libsqlite3. Se

[issue28494] is_zipfile false positives

2016-10-20 Thread Thomas Waldmann
Thomas Waldmann added the comment: patch for py2.7 The EOCD structure is at EOF. It either does not contain a comment (this is what the existing code checks first) or it contains a comment of the length that is specified in the structure. The patch checks consistency specified length vs. rea

[issue28494] is_zipfile false positives

2016-10-20 Thread Thomas Waldmann
Thomas Waldmann added the comment: Note: checking the first bytes of the file (PK..) might be another option. But this has the "problem" that a self-extracting zip starts with an executable that has different first bytes. So whether this is an option or not depends on whether is_zipfile() shou

[issue28484] test_capi, test_regrtest fail when multithreading disabled

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset b811ec148337 by Martin Panter in branch '3.6': Issue #28484: Skip tests if GIL is not used or multithreading is disabled https://hg.python.org/cpython/rev/b811ec148337 -- nosy: +python-dev ___ Python trac

[issue28471] Python 3.6b2 crashes with "Python memory allocator called without holding the GIL"

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset 2fd92794f775 by Martin Panter in branch '3.6': Issue #28471: Avoid ResourceWarning by detaching test socket https://hg.python.org/cpython/rev/2fd92794f775 -- ___ Python tracker

[issue26240] Docstring of the subprocess module should be cleaned up

2016-10-20 Thread Martin Panter
Martin Panter added the comment: V3 looks good to me -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:/

[issue28489] Fix comment in tokenizer.c

2016-10-20 Thread Eric V. Smith
Changes by Eric V. Smith : -- stage: -> commit review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue28484] test_capi, test_regrtest fail when multithreading disabled

2016-10-20 Thread Martin Panter
Changes by Martin Panter : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue23214] BufferedReader.read1(size) signature incompatible with BufferedIOBase.read1(size=-1)

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset b6886ac88e28 by Martin Panter in branch 'default': Issue #23214: Implement optional BufferedReader, BytesIO read1() argument https://hg.python.org/cpython/rev/b6886ac88e28 -- nosy: +python-dev ___ Python

[issue26163] FAIL: test_hash_effectiveness (test.test_set.TestFrozenSet)

2016-10-20 Thread Martin Panter
Martin Panter added the comment: I just got this failure again today. I think I have seen it once or twice before. Is the failure actually indicating a problem with Python, or is the test just too strict? It seems that it may be like a test ensuring that a random.randint(1, 100) is never equa

[issue28430] asyncio: C implemeted Future cause Tornado test fail

2016-10-20 Thread INADA Naoki
INADA Naoki added the comment: pure Python Future.__iter__ don't use what yield-ed. def __iter__(self): if not self.done(): self._asyncio_future_blocking = True yield self # This tells Task to wait for completion. assert self.done(), "yield from wasn'

[issue26163] FAIL: test_hash_effectiveness (test.test_set.TestFrozenSet)

2016-10-20 Thread Tim Peters
Tim Peters added the comment: I think Raymond will have to chime in. I assume this is due to the `letter_range` portion of the test suffering hash randomization dealing it a bad hand - but the underlying string hash is "supposed to be" strong regardless of seed. The self.assertGreater(4

[issue23214] BufferedReader.read1(size) signature incompatible with BufferedIOBase.read1(size=-1)

2016-10-20 Thread Roundup Robot
Roundup Robot added the comment: New changeset d4fce64b1c65 by Martin Panter in branch 'default': Issue #23214: Remove BufferedReader.read1(-1) workaround https://hg.python.org/cpython/rev/d4fce64b1c65 -- ___ Python tracker

[issue28495] MagickMock created by patсh annotation refuses to apply function side_effect at second time

2016-10-20 Thread Сергей Варюхин
New submission from Сергей Варюхин: You can see example in test.py. It should fails with AssertionErorr if mock works as expected. But it not fails. -- components: Library (Lib) files: test.py messages: 279098 nosy: Сергей Варюхин priority: normal severity: normal status: open title: Ma

[issue28447] socket.getpeername() failure on broken TCP/IP connection

2016-10-20 Thread Georgey
Georgey added the comment: Hello David, Yes I had the same thought with you that the information of socket is lost at operating syetem level. However, I hope at Python level this kind of information will not be lost. Once the socket has been created by incoming connection, the addres

[issue23214] BufferedReader.read1(size) signature incompatible with BufferedIOBase.read1(size=-1)

2016-10-20 Thread STINNER Victor
STINNER Victor added the comment: - .. method:: read1(size=-1) + .. method:: read1([size]) I don't understand this change: the default size is documented as -1. Did I miss something? + If *size* is −1 (the default) Is the "−" character deliberate in "−1"? I would expected ``-1``. --

  1   2   >