[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: > ./Include/pyatomic.h:37:5: error: 'atomic_int' does not name a type I don't understand how you got this error. The line 37 of pyatomic.h is only compiled when you build Python itself. At least, since Python 3.5.1 and Python 3.6.0. Did you try to build

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: PR to fix type in moduleTearDown. -- ___ Python tracker ___ ___

[issue30862] parent logger should also check the level

2017-07-06 Thread Vinay Sajip
Vinay Sajip added the comment: New changeset 0653fba51c03d20fa4381ba0836acd17fd05b04b by Vinay Sajip in branch 'master': bpo-30862: Updated Logger.setLevel documentation. (GH-2604) https://github.com/python/cpython/commit/0653fba51c03d20fa4381ba0836acd17fd05b04b --

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-07-06 Thread Terry J. Reedy
Changes by Terry J. Reedy : -- pull_requests: +2673 ___ Python tracker ___ ___

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: See the following commit of bpo-25150: commit 6df29ada02d22c43a8d439a70b820cb1ceacca42 Author: Victor Stinner Date: Fri Sep 18 15:06:34 2015 +0200 Issue #25150: Hide the private _Py_atomic_xxx symbols from the public

[issue27584] New addition of vSockets to the python socket module

2017-07-06 Thread Cathy Avery
Cathy Avery added the comment: So I revised my code based on the reviews and I passed all the checks ... now what? Thanks, Cathy -- ___ Python tracker

[issue26506] [EASY] hex() documentation: mention "%x" % int

2017-07-06 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: New changeset 67ba4fa467825d6a0c0a21cc54ff1df2ed1b by Mariatta (Manvisha Kodali) in branch 'master': bpo-26506: hex() documentation: mention %x % int (GH-2525) https://github.com/python/cpython/commit/67ba4fa467825d6a0c0a21cc54ff1df2ed1b --

[issue30779] IDLE: configdialog -- factor out Changes class

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: I am working on this now. -- ___ Python tracker ___ ___ Python-bugs-list

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: Oh, wait, I read your message backward. You said that the fix doesn't work? Hum, how can I reproduce the issue? What is your use case? -- resolution: fixed -> status: closed -> open ___ Python tracker

[issue30862] parent logger should also check the level

2017-07-06 Thread Vinay Sajip
Changes by Vinay Sajip : -- pull_requests: +2672 ___ Python tracker ___ ___

[issue30532] email.policy.SMTP.fold() mangles long headers

2017-07-06 Thread R. David Murray
R. David Murray added the comment: Thanks, Joel! -- resolution: -> fixed stage: backport needed -> resolved status: open -> closed type: -> behavior ___ Python tracker

[issue30532] email.policy.SMTP.fold() mangles long headers

2017-07-06 Thread R. David Murray
R. David Murray added the comment: New changeset 3bbdf990a2c1b0b303b950058e3177a1bd5f697a by R. David Murray (Joel Hillacre) in branch '3.5': bpo-30532: Fix whitespace folding in certain cases (#2592) https://github.com/python/cpython/commit/3bbdf990a2c1b0b303b950058e3177a1bd5f697a

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread Melroy van den Berg
Melroy van den Berg added the comment: Yea I'm building Python 3.6.1 from source, using gcc and --with-cxx-main=correct/location/to/g++ flag. So the warning message appears during ./configure about that it can't find g++. I didn't try to patch it using the attachments of this issue. Since I

[issue30819] Linking with 'ld -b' fails with 64-bit using Itanium HP compiler

2017-07-06 Thread Robert Boehne
Changes by Robert Boehne : -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list

[issue30861] StreamReader does not return reamaing and ready data buffer before raise the Exeption

2017-07-06 Thread pfreixes
pfreixes added the comment: As was said, the assumption here is the data that came to the buffer must be available. For example, the next snippet shows a Redis client that expects the data message plus the RST packet, where the redis-server was configured to accept max N connections, the

[issue30532] email.policy.SMTP.fold() mangles long headers

2017-07-06 Thread R. David Murray
R. David Murray added the comment: New changeset c60d2f5e8609b040ab58c498fde23928fe9dbef5 by R. David Murray (Joel Hillacre) in branch '3.6': bpo-30532: Fix whitespace folding in certain cases (#2591) https://github.com/python/cpython/commit/c60d2f5e8609b040ab58c498fde23928fe9dbef5

[issue30860] Consolidate stateful C globals under a single struct.

2017-07-06 Thread Antoine Pitrou
Antoine Pitrou added the comment: After looking at the PR, I'm a bit skeptical about this. Suddenly a lot of things which are implementation details get moved to the public include files, which makes things more confusing from my POV. I also don't know why all globals should be consolidated

[issue30183] [HPUX] compilation error in pytime.c with cc compiler

2017-07-06 Thread Robert Boehne
Changes by Robert Boehne : -- pull_requests: +2674 ___ Python tracker ___ ___

[issue30861] StreamReader does not return reamaing and ready data buffer before raise the Exeption

2017-07-06 Thread Guido van Rossum
Guido van Rossum added the comment: We seem to have a failure to communicate. I'm sure your example code "works", but you're not showing what's in the data it receives that is important for the app to read (your example just prints it). And surely your app should be robust even if the

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: I don't think that CPython can be built by g++. If you consider that it should, please open a new issue since it's a different use case. -- ___ Python tracker

[issue27268] Incorrect error message on float('')

2017-07-06 Thread Wolfgang Maier
Wolfgang Maier added the comment: Could somebody turn this into a PR to move things forward? I guess Nofar mistakenly set resolution to "works for me", but meant "patch works for me"? -- nosy: +wolma ___ Python tracker

[issue30863] Rewrite PyUnicode_AsWideChar() and PyUnicode_AsWideCharString()

2017-07-06 Thread Serhiy Storchaka
New submission from Serhiy Storchaka: Since Python 3.3 PyUnicode_AsUnicodeAndSize() is deprecated in favour of PyUnicode_AsWideChar() and PyUnicode_AsWideCharString(). But the latter two are implemented using PyUnicode_AsUnicodeAndSize(). This prevents adding the deprecation compiler warning

[issue30863] Rewrite PyUnicode_AsWideChar() and PyUnicode_AsWideCharString()

2017-07-06 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- pull_requests: +2667 ___ Python tracker ___ ___

[issue30861] StreamReader does not return reamaing and ready data buffer before raise the Exeption

2017-07-06 Thread Dima Tisnek
Dima Tisnek added the comment: My 2c: Pau's concern seems valid, in a sense that stream should work like TCP. That's what most users would assume -- read out data until the end, only then you can see what the actual error was (socket closed, or timeout or hard error) However, I suspect the

[issue30862] parent logger should also check the level

2017-07-06 Thread Vinay Sajip
Vinay Sajip added the comment: As RDM says, this behaviour is as designed. The logger's setLevel documentation says: "Sets the threshold for this logger to lvl. Logging messages which are less severe than lvl will be ignored. When a logger is created, the level is set to NOTSET (which causes

[issue30855] [3.5] test_tk: test_use() of test_tkinter.test_widgets randomly fails with "integer value too large to represent" on with AMD64 Windows8 3.5

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: New changeset f6d6480b93eca6f353784579108957108750c004 by Victor Stinner in branch '2.7': [2.7] bpo-30855: Trying to fix test_use on Windows. (#2586) https://github.com/python/cpython/commit/f6d6480b93eca6f353784579108957108750c004 --

[issue30847] asyncio: selector_events: add_urgent() for urgent data to read, 3rd argument of select.select()

2017-07-06 Thread Pim Klanke
Pim Klanke added the comment: This is in fact the third catagory of select(), "exceptional conditions", but because some find the term "exceptional" confusing when used in a Python module, we decided to use the term "urgent data", borrowed from the poll(2) man page. (see bpo-30844) An

[issue30799] Improved test coverage Lib/_osx_support.py 99%

2017-07-06 Thread chexex
New submission from chexex: I will appreciate any feedback. -- nosy: +ned.deily, ronaldoussoren ___ Python tracker ___

[issue30862] parent logger should also check the level

2017-07-06 Thread TaoQingyun
TaoQingyun added the comment: yes, I understand the effective level. my question is that before call ancestor's handler, should also check `c.isEnabledFor(record.levelno)` ``` def callHandlers(self, record): """ Pass a record to all relevant handlers. Loop through

[issue30865] python cannot import module located on a "VOLUME" directory

2017-07-06 Thread Adrien Pré
New submission from Adrien Pré: This issue created on python-docker project https://github.com/docker-library/python/issues/210 indicates that python cannot import a module if it is located on a docker-volume. yosifkit's comment suggest there is an issue in the UNC resolution with paths

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
Changes by STINNER Victor : -- pull_requests: +2670 ___ Python tracker ___ ___

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Usually we first merge a patch in master and after that cherry-pick it to other branches. -- ___ Python tracker ___

[issue30867] Add necessary macro `HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid declaration

2017-07-06 Thread signal1587
New submission from signal1587: Compile Python (2.7.13) on platform which has OpenSSL (<0.9.8) will met error: /opt/Python-2.7.13/Modules/_ssl.c:177: error: syntax error before ‘*’ token Since return type `X509_VERIFY_PARAM` got added to OpenSSL in version 0.9.8. Surround the implementation

[issue30319] Change socket.close() to ignore ECONNRESET

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: Thanks Martin for the long explanation. To simplify a lot, there is and was never any warranty that a successful sock.send() call delivered data to the peer. Each layer does its best, but the data can be lost at any layer, and the peer is free to close the

[issue30855] [3.5] test_tk: test_use() of test_tkinter.test_widgets randomly fails with "integer value too large to represent" on with AMD64 Windows8 3.5

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: I backported fixes to 2.7 and 3.5 branches. The test fails randomly, so I close the issue and hope that it doesn't come back. If it comes back on 3.5, we should now get the identifier which caused the failure. I will reopen the issue if the bug occurs again.

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
Changes by STINNER Victor : -- pull_requests: +2669 ___ Python tracker ___ ___

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
New submission from STINNER Victor: In issues bpo-28870, bpo-29227, bpo-29233 and bpo-29234, I worked on reducing the C stack consumption. I now would to add the _testcapi.stack_pointer() that I used to be able to easily measure the stack consumption. Try for example attached

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
Changes by STINNER Victor : -- pull_requests: +2668 ___ Python tracker ___ ___

[issue30865] python cannot import module located on a "VOLUME" directory

2017-07-06 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware ___ Python tracker

[issue30864] Compile failure for linux socket CAN support

2017-07-06 Thread Riccardo Magliocchetti
New submission from Riccardo Magliocchetti: I have an issue related to this while trying to compile statically Python 3.6.1 (but latest master looks the same) against a static musl. The problem is that i have AF_CAN defined because it's defined in linux/socket.h but by not having

[issue30847] asyncio: selector_events: add_urgent() for urgent data to read, 3rd argument of select.select()

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: Ok, it makes sense. Thanks :-) -- ___ Python tracker ___ ___

[issue30847] asyncio: selector_events: add_urgent() for urgent data to read, 3rd argument of select.select()

2017-07-06 Thread Pim Klanke
Pim Klanke added the comment: > "Let's say that we got read event on sockets A and B (in an ordered list from > selectors: A, then B), but B gets urgent data: should we handle B urgent data > before not-urgent A data?" IMO No. The same strategy applies. urgent data events on B have priority

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
Changes by STINNER Victor : -- pull_requests: +2671 ___ Python tracker ___ ___

[issue30339] test_multiprocessing_main_handling: "RuntimeError: Timed out waiting for results" on x86 Windows7 3.x

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: http://buildbot.python.org/all/builders/AMD64%20Windows7%20SP1%203.5/builds/229/steps/test/logs/stdio == FAIL: test_directory (test.test_multiprocessing_main_handling.SpawnCmdLineTest)

[issue30867] Add necessary macro that insure`HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid function declaration

2017-07-06 Thread signal1587
Changes by signal1587 : -- title: Add necessary macro `HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid declaration -> Add necessary macro that insure`HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid function declaration ___ Python

[issue30867] Add necessary macro that insure `HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid function declaration

2017-07-06 Thread signal1587
Changes by signal1587 : -- title: Add necessary macro that insure`HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid function declaration -> Add necessary macro that insure `HAVE_OPENSSL_VERIFY_PARAM` to avoid invalid function declaration

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread Melroy van den Berg
Melroy van den Berg added the comment: I also got this problem with gcc/g++ version 4.9.3. ./Include/pyatomic.h:37:5: error: 'atomic_int' does not name a type I'm using Python 3.6.1. It's not fixed! -- nosy: +Melroy van den Berg versions: +Python 3.6 -Python 3.5

[issue30862] parent logger should also check the level

2017-07-06 Thread R. David Murray
R. David Murray added the comment: @qingyunha: we are telling you that that would *introduce* a bug. This is working the way it is supposed to. Vinay, what if we rewrote the beginning of that paragraph like this: Sets the threshold for this logger to lvl. Logging messages which are less

[issue30865] python cannot import module located on a "VOLUME" directory

2017-07-06 Thread R. David Murray
R. David Murray added the comment: Reading through some of the linked material, it looks like the issue is with how UNC "symlinks" are resolved. -- nosy: +r.david.murray ___ Python tracker

[issue30858] Keyword can't be an expression?

2017-07-06 Thread R. David Murray
R. David Murray added the comment: If I saw your message, I would think "what is a 'simple name'?". There's no glossary entry for that, nor is it a concept used elsewhere in the documentation as far as I remember. One could instead use "single identifier", but the problem with both of

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: > Usually we first merge a patch in master and after that cherry-pick it to > other branches. Right. But in this case, I needed the patch for each branch, to run the test on all branches to write a blog post :-) Here are results (blog post spoiler!): Table

[issue30866] Add _testcapi.stack_pointer() to measure the C stack consumption

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: I just published my blog post: "My contributions to CPython during 2017 Q1" https://haypo.github.io/contrib-cpython-2017q1.html See the "Stack consumption" section. -- ___ Python tracker

[issue30860] Consolidate stateful C globals under a single struct.

2017-07-06 Thread Jeremy Kloth
Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker ___ ___

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread Melroy van den Berg
Melroy van den Berg added the comment: Ow sorry, I though it was already fixed in the latest stable version of 3.x. Since is issue is from 2015. -- ___ Python tracker

[issue23644] g++ module compile fails with ‘_Atomic’ does not name a type

2017-07-06 Thread STINNER Victor
STINNER Victor added the comment: > I'm using Python 3.6.1. It's not fixed! Thanks for the confirmation of the fix ;-) -- ___ Python tracker ___

[issue30779] IDLE: configdialog -- factor out Changes class

2017-07-06 Thread Terry J. Reedy
Changes by Terry J. Reedy : -- pull_requests: +2677 ___ Python tracker ___ ___

[issue30869] regrtest: Add .idlerc to saved_test_environment

2017-07-06 Thread Louie Lu
Changes by Louie Lu : -- pull_requests: +2679 ___ Python tracker ___ ___ Python-bugs-list

[issue30869] regrtest: Add .idlerc to saved_test_environment

2017-07-06 Thread Louie Lu
New submission from Louie Lu: In bpo #30780, there is a mistake of tearDownModule didn't restore the use rCfg. To prevent future mistake, Adding .idlerc to regrtest saved_test_environment, so that `--fail-env-changed` option can detect .idlerc been changed in IDLE test. --

[issue30779] IDLE: configdialog -- factor out Changes class

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: Whoops, we seems to have partly overlapped (and made some of the same changes). I plan to push my PR sometime tomorrow (it also changes configdialog and revised tests) after looking as yours. Then move on to adding config tests, #30780. The tests for

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset 25a4206c243e3b1fa6f5b1c72a11b409b007694d by terryjreedy in branch 'master': bpo-30780: Fix error in idlelib.test_idle.test_configdialog (#2606) https://github.com/python/cpython/commit/25a4206c243e3b1fa6f5b1c72a11b409b007694d --

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: I posted PR 2612 for #30779 and expect to merge it tomorrow after sleep and final review. It includes passing revisions of existing configdialog tests. A PR dependent on 2612 could be posted before I do the merge. Follow the model of using xyzpage names in

[issue30868] IDLE: Improve configuration tests with mock Save.

2017-07-06 Thread Terry J. Reedy
New submission from Terry J. Reedy: When writing tests for config and configdialog, I blocked the user configs from hitting the filesystem by passing '' as the file name. This disables the Save function. To know that Save is called, a mock is needed. To know what would have been written to

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-07-06 Thread Terry J. Reedy
Changes by Terry J. Reedy : -- pull_requests: +2678 ___ Python tracker ___ ___

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset df0f99329843c10701ffaefbd3948ac698c12220 by terryjreedy in branch '3.6': [3.6] bpo-30780: Fix error in idlelib.test_idle.test_configdialog (GH-2606) (#2613) https://github.com/python/cpython/commit/df0f99329843c10701ffaefbd3948ac698c12220

[issue30869] regrtest: Add .idlerc to saved_test_environment

2017-07-06 Thread Terry J. Reedy
Terry J. Reedy added the comment: I would have liked the mistake in #30780 to have been noticed, but I don't think the patch would have caught it. What I failed to restore was an in memory date structure that is part of the config module. The patch seems to be concerned only with the

[issue30779] IDLE: configdialog -- factor out Changes class

2017-07-06 Thread Cheryl Sabella
Changes by Cheryl Sabella : -- pull_requests: +2675 ___ Python tracker ___ ___

[issue30779] IDLE: configdialog -- factor out Changes class

2017-07-06 Thread Cheryl Sabella
Cheryl Sabella added the comment: I've made the first pull request for the changes to config.py. I have to apologize because I really couldn't figure out how to do 'save_all' without including 'set_user_value' since 'set_user_value' updates userCfg with the values from ConfigChanges. Also,

[issue30790] Can't use proxy to connect internet on windows

2017-07-06 Thread Wu Zongyong
Changes by Wu Zongyong : -- pull_requests: +2676 ___ Python tracker ___ ___

[issue30860] Consolidate stateful C globals under a single struct.

2017-07-06 Thread Nick Coghlan
Nick Coghlan added the comment: The core motivation driving the original refactoring was to better understand and consolidate our runtime state in order to clarify what the GIL is actually protecting (aside from the reference counts). That then turned out to have surprising performance

[issue30797] ./pyconfig.h:1438:0: warning: "_GNU_SOURCE" redefined [enabled by default]

2017-07-06 Thread Ned Deily
Ned Deily added the comment: New changeset 05b72ede95521b2d897cb4c7b034139b5437c592 by Ned Deily in branch '3.6': bpo-30797, bpo-30694: Avoid _GNU_SOURCE redefined warning in xmlparse.c (#2615) https://github.com/python/cpython/commit/05b72ede95521b2d897cb4c7b034139b5437c592 -- nosy:

[issue30694] Update embedded copy of expat to 2.2.1

2017-07-06 Thread Ned Deily
Ned Deily added the comment: New changeset 05b72ede95521b2d897cb4c7b034139b5437c592 by Ned Deily in branch '3.6': bpo-30797, bpo-30694: Avoid _GNU_SOURCE redefined warning in xmlparse.c (#2615) https://github.com/python/cpython/commit/05b72ede95521b2d897cb4c7b034139b5437c592 --

[issue30694] Update embedded copy of expat to 2.2.1

2017-07-06 Thread Ned Deily
Ned Deily added the comment: See bpo-30797 for compile warning introduced here. -- assignee: -> haypo ___ Python tracker ___

[issue30797] ./pyconfig.h:1438:0: warning: "_GNU_SOURCE" redefined [enabled by default]

2017-07-06 Thread Ned Deily
Ned Deily added the comment: Thanks for the report. This warning is currently visible on many/all of the buildbots on all branches. The conflict is caused by the upgrade of the bundled to libexpat to version 2.2.1 (bpo-30694). 2.2.1 introduced a define of _GNU_SOURCE into

[issue30797] ./pyconfig.h:1438:0: warning: "_GNU_SOURCE" redefined [enabled by default]

2017-07-06 Thread Ned Deily
Changes by Ned Deily : -- pull_requests: +2680 ___ Python tracker ___ ___ Python-bugs-list

[issue30694] Update embedded copy of expat to 2.2.1

2017-07-06 Thread Ned Deily
Changes by Ned Deily : -- pull_requests: +2681 ___ Python tracker ___ ___ Python-bugs-list