[issue20104] expose posix_spawn(p)

2018-05-13 Thread STINNER Victor
STINNER Victor added the comment: For Python 3.7, I discussed with Pablo and he seems to be ok to remove the function from Python 3.7, to get more time to polish the API in the master branch (future Python 3.8). Can someone please write a PR to remove the function from

[issue33479] Document tkinter and threads

2018-05-13 Thread Ivan Pozdeev
Ivan Pozdeev added the comment: I was composing a letter to python-dev with all I know of how tkinter works with regard to threads and Tcl, and the fixing plans so that we're all on the same page if you wish to participate. I'm no longer sure if it belongs in the

[issue33490] pthread auto-detection can use AX_PTHREAD

2018-05-13 Thread Eitan Adler
New submission from Eitan Adler : There is a ton of logic in configure.ac to detect pthreads support. There is a standard macro to use instead: https://www.gnu.org/software/autoconf-archive/ax_pthread.html#ax_pthread In order to simplify our pthread logic we should make

[issue33490] pthread auto-detection can use AX_PTHREAD

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- keywords: +patch pull_requests: +6475 stage: -> patch review ___ Python tracker ___

[issue33477] Document that compile(code, 'exec') has different behavior in 3.7+

2018-05-13 Thread INADA Naoki
INADA Naoki added the comment: ref #32911 And it's documented already. https://docs.python.org/3.7/whatsnew/3.7.html#changes-in-the-python-api -- nosy: +inada.naoki ___ Python tracker

[issue33471] string format with 'n' failling with french locales

2018-05-13 Thread David Vasseur
David Vasseur added the comment: >>> locale.localeconv() {'int_curr_symbol': 'EUR ', 'currency_symbol': '€', 'mon_decimal_point': ',', 'mon_thousands_sep': '\u202f', 'mon_grouping': [3, 0], 'positive_sign': '', 'negative_sign': '-', 'int_frac_digits': 2, 'frac_digits': 2,

[issue33481] configparser.write() does not save comments.

2018-05-13 Thread R. David Murray
Change by R. David Murray : -- nosy: +lukasz.langa ___ Python tracker ___ ___

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread miss-islington
miss-islington added the comment: New changeset 0d17e60b33aca1a4d151a8a2bd6eaa331f0ec658 by Miss Islington (bot) in branch '3.6': bpo-28055: Fix unaligned accesses in siphash24(). (GH-6123)

[issue33474] Support immutability per-field in dataclasses

2018-05-13 Thread Eric V. Smith
Eric V. Smith added the comment: Can you explain your use case for this? Also, how do you envision this working with the existing frozen, hashing, and equality behavior? There are a lot of interactions there, and we'd want to do something that makes sense holistically,

[issue33478] PEP 8 CapWords reference wrong?

2018-05-13 Thread Amit Saha
Amit Saha added the comment: Thanks for the reply. I think I was not clear - the behavior of string.capitalize() is correct as per documentation. But the PEP8 referring to CapWords as cap words case is the bit I am not clear about, since `Capwords` back when you call

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread miss-islington
miss-islington added the comment: New changeset 8ed545f6de37efdadbcf71c45bb8136b8cb9619d by Miss Islington (bot) in branch '3.7': bpo-28055: Fix unaligned accesses in siphash24(). (GH-6123)

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread miss-islington
Change by miss-islington : -- pull_requests: +6463 ___ Python tracker ___

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread miss-islington
Change by miss-islington : -- pull_requests: +6464 ___ Python tracker ___

[issue33464] AIX and "specialized downloads" links

2018-05-13 Thread Michael Felt
Michael Felt added the comment: Took me a while to get this 'posted' properly. - please see https://github.com/python/pythondotorg/issues/1156 My apologies for the noise. -- stage: -> resolved status: open -> closed ___

[issue33481] configparser.write() does not save comments.

2018-05-13 Thread David James Peters
New submission from David James Peters : The ConfigParser().write() does not save the comments; this makes using comments harder because it requires a separate demo ini file the user must be able to locate and read from without learning anything from the INI file they are

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: New changeset 1e2ec8a996daec65d8d5a3d43b66a9909c6d0653 by Serhiy Storchaka (Rolf Eike Beer) in branch 'master': bpo-28055: Fix unaligned accesses in siphash24(). (GH-6123)

[issue33478] PEP 8 CapWords reference wrong?

2018-05-13 Thread R. David Murray
R. David Murray added the comment: In PEP 8 'CapWords" does not refer to the capwords function, as is fairly clear from context. A big clues is that it is not spelled 'capwords', and case matters in python function names :) I can understand your getting confused, but

[issue33478] PEP 8 CapWords reference wrong?

2018-05-13 Thread Berker Peksag
Berker Peksag added the comment: 'CapWord' is a single word so string.capwords() works as documented: Split the argument into words using str.split(), capitalize each word using str.capitalize(), and join the capitalized words using str.join().

[issue31500] IDLE: Tiny font on HiDPI display

2018-05-13 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Something was changed on my computer (I upgraded the OS twice since creating this issue), and now I don't see any difference between patched and unpatched 2.7. Terry, could you please test whether there are any visible effects

[issue33480] Improvement suggestions for urllib.parse.urlparser

2018-05-13 Thread brent s.
New submission from brent s. : Currently, a parsed urlparse() object looks (roughly) like this: urlparse('http://example.com/foo;key1=value1?key2=value2#key3=value3#key4=value4') returns: ParseResult(scheme='http', netloc='example.com', path='/foo',

[issue33483] build system requires explicit compiler, but should discover it

2018-05-13 Thread Eitan Adler
New submission from Eitan Adler : configure.ac requires explicit configuration for finding the preferred compiler. This results in a non-native way of configuring the system, and does not actually work on most platforms. Expected behavior: CC is used to discover the C

[issue33483] build system requires explicit compiler, but should discover it

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- keywords: +patch pull_requests: +6466 stage: -> patch review ___ Python tracker ___

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread Stefan Krah
Stefan Krah added the comment: MSVC optimizes memcpy() to an assignment, sometimes too well (pgo): https://bugs.python.org/issue15993 But that is fixed long ago, so I also think that the memcpy() approach is best. -- ___

[issue33484] build system runs when it merely link

2018-05-13 Thread Eitan Adler
New submission from Eitan Adler : The build system attempts to run certain test code when it can actually just link the code. This is a minor performance optimization but is really a semantic correctness issue. -- components: Build messages: 316471 nosy:

[issue33485] autoconf target does not

2018-05-13 Thread Eitan Adler
New submission from Eitan Adler : The current "autoconf" target is suboptimal in a few ways: - it does not use a quoted srcdir - it uses autoheader and autoconf instead of autoreconf - it does act sanely if 'cd' fails. -- components: Build messages: 316472 nosy:

[issue33485] autoconf target does not behave correctly

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- title: autoconf target does not -> autoconf target does not behave correctly ___ Python tracker ___

[issue33484] build system runs when it merely link

2018-05-13 Thread R. David Murray
R. David Murray added the comment: Thanks for wanting to improve Python. Your PR seems to contain quite a few changes that aren't obviously related to your topic. As for the topic, I know there are at least some cases where we really do have to *run* the code to prove

[issue33485] autoconf target does not behave correctly

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- keywords: +patch pull_requests: +6469 stage: -> patch review ___ Python tracker ___

[issue33484] build system runs when it merely link

2018-05-13 Thread R. David Murray
R. David Murray added the comment: Yes, we are one of those projects that wants it to be possible to build python without having the tools that generate build artifacts. You will note that we also check in build artifacts that are produced by our own python scripts, to

[issue33486] regen autotools related files

2018-05-13 Thread Eitan Adler
New submission from Eitan Adler : There are few if any functional changes, but this is helpful when making other changes to the build system. -- components: Build messages: 316477 nosy: eitan.adler priority: normal severity: normal status: open title: regen

[issue33484] build system runs when it merely link

2018-05-13 Thread Eitan Adler
Eitan Adler added the comment: I believe I have removed the regen step from all my patches. I will separately upgrade the generated files (particularly since it is only a minor version bump). -- ___ Python tracker

[issue28055] pyhash's siphash24 assumes alignment of the data pointer

2018-05-13 Thread Antoine Pitrou
Antoine Pitrou added the comment: Indeed the memcpy() approach is the common idiom in such situations, and sounds like the right thing. -- ___ Python tracker

[issue33484] build system runs when it merely link

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- keywords: +patch pull_requests: +6468 stage: -> patch review ___ Python tracker ___

[issue33484] build system runs when it merely link

2018-05-13 Thread Eitan Adler
Eitan Adler added the comment: The unrelated changes are likely due to the "regen" step. I am using newer tools than previously used. It is unclear to me why we include generated files in the repository. Some projects prefer this to avoid the need to have autotools

[issue33486] regen autotools related files

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- keywords: +patch pull_requests: +6470 stage: -> patch review ___ Python tracker ___

[issue33482] codecs.StreamRecoder.writelines is broken

2018-05-13 Thread Jelle Zijlstra
Change by Jelle Zijlstra : -- keywords: +patch pull_requests: +6465 stage: -> patch review ___ Python tracker ___

[issue27632] build on AIX fails when builddir != srcdir, more than bad path to ld_so_aix

2018-05-13 Thread Michael Felt
Michael Felt added the comment: OOT builds are working for Python2.7, Python3-3.5, Python3-3.6 (and later I expect) but not for Python3-3.4(.8) unable to execute '../git/python3-3.4.8/Modules/ld_so_aix': No such file or directory while building OOT

[issue33482] codecs.StreamRecoder.writelines is broken

2018-05-13 Thread Jelle Zijlstra
New submission from Jelle Zijlstra : codecs.StreamRecoder.writelines is implemented as: def writelines(self, list): data = ''.join(list) data, bytesdecoded = self.decode(data, self.errors) return self.writer.write(data) It can't take a

[issue27632] build on AIX fails when builddir != srcdir, more than bad path to ld_so_aix

2018-05-13 Thread Michael Felt
Michael Felt added the comment: Wishing I could edit a post... root@x066:[/data/prj/python]find ./python3-3.4.8 git/python3-3.4.8 -name ld_so_aix ./python3-3.4.8/Modules/ld_so_aix -- ___ Python tracker

[issue22269] Resolve distutils option conflicts with priorities

2018-05-13 Thread Nicholas Chammas
Change by Nicholas Chammas : -- nosy: +nchammas ___ Python tracker ___ ___

[issue33488] github pull request template does not satisfy markdownlint

2018-05-13 Thread Eitan Adler
New submission from Eitan Adler : markdownlint is, as the name implies, a tool for linting markdown files. The current template has the following lint warnings: ``` ./.github/PULL_REQUEST_TEMPLATE.md:8: MD031 Fenced code blocks should be surrounded by blank lines

[issue33488] github pull request template does not satisfy markdownlint

2018-05-13 Thread Eitan Adler
Change by Eitan Adler : -- keywords: +patch pull_requests: +6473 stage: -> patch review ___ Python tracker ___

[issue31500] IDLE: Tiny font on HiDPI display

2018-05-13 Thread Terry J. Reedy
Terry J. Reedy added the comment: Does IDLE now work on your HiDPI screen without the patch? or does it not work even with the patch? I just got a 2.7 worktree working with Python built. 2.7.15+ comes with 8.5.19 instead of 8.5.15. IDLE runs as I expect and windows look

[issue33488] github pull request template does not satisfy markdownlint

2018-05-13 Thread Benjamin Peterson
Benjamin Peterson added the comment: New changeset 5cd22cf2097a6dd20cf97858d54564a3be87 by Benjamin Peterson (Eitan Adler) in branch 'master': bpo-33488: Satisfy markdownlint for the pull request template. (GH-6786)

[issue33480] Improvement suggestions for urllib.parse.urlparser

2018-05-13 Thread Ivan Pozdeev
Ivan Pozdeev added the comment: Such drastic changes of uncertain usefulness are best discussed at python-ideas first. What you're really asking for seems to be to parse all "levels" at the same time. Try to think of a use case that would make that help anything

[issue33453] from __future__ import annotations breaks dataclasses ClassVar and InitVar handling

2018-05-13 Thread Eric V. Smith
Eric V. Smith added the comment: There have been comments on the PR, but I'd like to focus the higher level issue back here. Specifically, see my comment https://github.com/python/cpython/pull/6768#discussion_r187813919 To summarize: I still think string inspections are

[issue33480] Improvement suggestions for urllib.parse.urlparser

2018-05-13 Thread R. David Murray
R. David Murray added the comment: These are actually reasonable requests, and in fact have been brought up before and implemented: >>> x = >>> urlparse('http://me:myp...@example.com:800/foo;key1=value1?key2=value2#key3=value3#key4=value4') >>> x

[issue33461] json.loads(encoding=) does not emit deprecation warning.

2018-05-13 Thread Matthias Bussonnier
Change by Matthias Bussonnier : -- pull_requests: +6471 ___ Python tracker ___

[issue33461] json.loads(encoding=) does not emit deprecation warning.

2018-05-13 Thread Matthias Bussonnier
Matthias Bussonnier added the comment: I've opened https://github.com/python/cpython/pull/6784 as an alternative that removes it. I was worried to remove it so close to 3.7. -- ___ Python tracker

[issue33274] minidom removeAttributeNode returns None

2018-05-13 Thread Fred L. Drake, Jr.
Fred L. Drake, Jr. added the comment: While I've no strong objection to updating to follow the specification, I also don't see any real value here. The current minidom implementation has been considered sufficient for many years now (if you consider the DOM desirable at

[issue33487] BZ2File(buffering=None) does not emit deprecation warning, deprecation version not documented.

2018-05-13 Thread Matthias Bussonnier
New submission from Matthias Bussonnier : BZ2File(buffering=None) should emit a deprecation warning as the argument is deprecated. The deprecation message and documentation should say since when it is deprecated. (AFAICT since 3.3) -- messages: 316482

[issue33487] BZ2File(buffering=None) does not emit deprecation warning, deprecation version not documented.

2018-05-13 Thread Matthias Bussonnier
Change by Matthias Bussonnier : -- keywords: +patch pull_requests: +6472 stage: -> patch review ___ Python tracker ___

[issue33488] github pull request template does not satisfy markdownlint

2018-05-13 Thread Benjamin Peterson
Change by Benjamin Peterson : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue33488] github pull request template does not satisfy markdownlint

2018-05-13 Thread Benjamin Peterson
Change by Benjamin Peterson : -- pull_requests: +6474 ___ Python tracker ___ ___

[issue33489] Newer externals for windows do not always trigger rebuild

2018-05-13 Thread Terry J. Reedy
New submission from Terry J. Reedy : I added a new 2.7 worktree, with 2.7 checked out to track origin (my github fork). I built Python with tcl/tk 8.5.15. Whoops, my fork had 2.7 at 2.7.13+. Fetch upstream, merge upstream/2.7, and rebuild. I noticed that tcl/tk 8.5.19

[issue33488] github pull request template does not satisfy markdownlint

2018-05-13 Thread Benjamin Peterson
Benjamin Peterson added the comment: New changeset f1ee4a2e52d98698ec2c0a0e8acb6cd3bbc9bf1f by Benjamin Peterson in branch '3.7': [3.7] bpo-33488: Satisfy markdownlint for the pull request template. (GH-6786)