[issue22722] inheritable pipes are unwieldy without os.pipe2

STINNER Victor added the comment: os.dup2() is really a special case for inheritable file descriptors. See the PEP 446. -- ___ Python tracker ___

[issue22725] improve documentation for enumerate() (built-in function)

Van Ly added the comment: I don't want to argue. Ask a 12-year old and their English teacher, "Does the second sentence qualify as gobbledygook even if it is technically correct and complete and not verbose?" To be constructive and take on what has been said, an iteration on improving the wor

[issue22672] float arguments in scientific notation not supported by argparse

paul j3 added the comment: This issue has already been raise in http://bugs.python.org/issue9334 argparse does not accept options taking arguments beginning with dash (regression from optparse) There I proposed leaving '_negative_number_matcher' unchanged, but only use it to set '_has_negativ

[issue22725] improve documentation for enumerate() (built-in function)

R. David Murray added the comment: The existing documentation is technically correct, while your replacement leaves some things out, such as the fact that enumerate accepts an iteratable and returns an iterator that is specifically a special 'enumerate object'. (A sequence is an iteratable, bu

[issue9351] argparse set_defaults on subcommands should override top level set_defaults

Changes by R. David Murray : -- stage: resolved -> commit review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue9351] argparse set_defaults on subcommands should override top level set_defaults

R. David Murray added the comment: Yeah, I hesitated a bit about the backports, but didn't visualize the scenario you are suggesting and thought it would be safe. Perhaps it should be backed out of 2.7 and 3.4. For 3.5, do you have any thoughts about how to make namespace= play nicely with t

[issue3068] IDLE - Add an extension configuration dialog

Terry J. Reedy added the comment: #22726 Add help to both dialogs -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue22726] Idle: add help to config dialogs

New submission from Terry J. Reedy: At one time, ConfigDialog was given a do-nothing Help button. It was diabled a couple of years ago. This patch restores it for both config dialogs, displaying a text appropriate for both. There are separate functions in anticipation of dialog-specific con

[issue22721] pprint output for sets and dicts is not stable

Fred L. Drake, Jr. added the comment: Stability in output order from pprint is very useful in doctests (yes, some people write documentation that they test). I think fixing any output stability issues would be very worthwhile. -- ___ Python tracker

[issue22725] improve documentation for enumerate() (built-in function)

Changes by Ezio Melotti : -- nosy: +ezio.melotti versions: +Python 3.4, Python 3.6 -Python 3.3 ___ Python tracker ___ ___ Python-bugs-

[issue22725] improve documentation for enumerate() (built-in function)

New submission from Van Ly: The existing documentation is confusing. — improve wording as follows enumerate(sequence, start=0) Returns pairings of index into sequence[link to glossary.html#term-sequence] with the object at that index in the sequence. — wording as found in v.2.7.5 enumerate

[issue10116] Sporadic failures in test_urllibnet

Changes by Berker Peksag : -- nosy: +berker.peksag ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail

[issue22650] set up and use VM for net access in the test suite

Berker Peksag added the comment: There are two disabled tests in Lib/test/test_urllib2net.py: * OtherNetworkTests.test_cnri: # XXX Following test depends on machine configurations that are internal # to CNRI. Need to set up a public server with the right authentication # configurat

[issue22596] support.transient_internet() doesn't catch connection refused errors

Berker Peksag added the comment: Thanks for the review, Ned. I didn't combine the patch with issue 22289, because I couldn't reproduce that failure easily. -- assignee: -> berker.peksag resolution: -> fixed stage: patch review -> resolved status: open -> closed __

[issue22596] support.transient_internet() doesn't catch connection refused errors

Roundup Robot added the comment: New changeset 76ef82ec80a7 by Berker Peksag in branch '3.4': Issue #22596: support.transient_internet() now also catches https://hg.python.org/cpython/rev/76ef82ec80a7 New changeset 9c35973829e6 by Berker Peksag in branch 'default': Issue #22596: support.transien

[issue22723] visited-link styling is not accessible

Changes by Berker Peksag : -- resolution: -> fixed status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Un

[issue22723] visited-link styling is not accessible

Berker Peksag added the comment: Thanks for the patch, Buck. -- assignee: docs@python -> berker.peksag stage: -> resolved type: -> enhancement versions: +Python 3.4, Python 3.5 ___ Python tracker

[issue22723] visited-link styling is not accessible

Roundup Robot added the comment: New changeset a3481025a742 by Berker Peksag in branch '3.4': Issue #22723: Make link styling more accessible. https://hg.python.org/cpython/rev/a3481025a742 New changeset 56e87f2c6d3c by Berker Peksag in branch 'default': Issue #22723: Make link styling more acce

[issue17909] Autodetecting JSON encoding

Changes by Martin Panter : -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue10976] json.loads() raises TypeError on bytes object

Martin Panter added the comment: Issue 17909 (auto-detecting JSON encoding) looks like it has a patch which would probably satisfy this issue -- nosy: +vadmium ___ Python tracker __

[issue19837] Wire protocol encoding for the JSON module

Changes by Martin Panter : -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue10390] json.load should handle bytes input

Martin Panter added the comment: See also Issue 10976, discussing passing bytes() strings to json.loads() -- nosy: +vadmium ___ Python tracker ___ ___

[issue22721] pprint output for sets and dicts is not stable

Antoine Pitrou added the comment: Hmm... is it important? -- nosy: +pitrou ___ Python tracker ___ ___ Python-bugs-list mailing list Un

[issue22724] byte-compile fails for cross-builds

Benedikt Morbach added the comment: alternatively, introduce a $(PYTHON_FOR_INSTALL), which doesn't set PYTHONPATH. Then run those steps using that, with the Makefile setting PYTHONPATH=$(DESTDIR)$(LIBDEST), which doesn't contain the compiled modules at that point. The first solution looks cl

[issue22724] byte-compile fails for cross-builds

Changes by Benedikt Morbach : -- versions: +Python 3.5, Python 3.6 -Python 3.2 ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue22724] byte-compile fails for cross-builds

New submission from Benedikt Morbach: For cross-builds the byte-compiling and 2to3 Grammar generation fail. This is because PYTHONPATH/sys.path includes 'build/lib.$PLATFORM', in which the compiled .so modules reside. The host python obviously barfs on those, as they are compiled for a differen

[issue9351] argparse set_defaults on subcommands should override top level set_defaults

paul j3 added the comment: A possible problem with this patch is that it overrides any Namespace given by the user. In the test example: parser.parse_args(['X'], namespace=argparse.Namespace(foo=3)) the result is 'foo=2', the setdefault from the subparser. Previously, and with a single

[issue13319] IDLE: Menu accelerator conflict between Format and Options

Terry J. Reedy added the comment: Currently, in 3.4 Win 7, tk 8.6, Alt O (or Alt O O O , etc) is required to get the Format menu to drop down. Following that with Alt O does select comment out. Alt O O (with or without holding down Alt) opens the Option menu. Having to hit makes both poss

[issue13319] IDLE: Menu accelerator conflict between Format and Options

Changes by Terry J. Reedy : -- versions: +Python 3.5 -Python 3.3 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22697] Deadlock with writing to stderr from forked process

Changes by Antoine Pitrou : -- nosy: +neologix ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyt

[issue22596] support.transient_internet() doesn't catch connection refused errors

Ned Deily added the comment: LGTM. Perhaps it can be committed with a revised version of the patch for Issue22289? -- nosy: +haypo, ned.deily ___ Python tracker ___ ___

[issue22289] support.transient_internet() doesn't catch timeout on FTP tests of test_urllib2net

Ned Deily added the comment: There seems to be a small problem with the patch that can be easily fixed by changing the test for "ftp error: timeout()" to "ftp error: timeout(". Otherwise, it seems OK. Note that there is now also a related patch for Issue22596 that modifies the same test. Pe

[issue20306] Lack of pw_gecos field in Android's struct passwd causes cross-compilation for the pwd module to fail

Matt Frank added the comment: Here is shiz's patch extended with the addition to configure.ac. I added the variable HAVE_PASSWD_GECOS_FIELD and the appropriate tests. Luckily this very problem (missing pw_gecos field) is the example used in the autoconf manual (https://www.gnu.org/software/a

[issue22722] inheritable pipes are unwieldy without os.pipe2

Antoine Pitrou added the comment: > This seems objectionable since set_inheritable() code is invoked > twice, where I'd prefer to invoke it zero times (or at most once). Does it make a difference in the grand scheme of things? If you're forking a process, you're usually doing something heavy en

[issue22719] os.path.isfile & os.path.exists bug in while loop

eryksun added the comment: > i.e. the object id is the same after appending Actually, that's wrong. bak_path is a compact string. So the whole object is realloc'd, and the base address (i.e. id) could change. Check PyUnicode_AsUnicode even if the id changes. -- __

[issue20306] Lack of pw_gecos field in Android's struct passwd causes cross-compilation for the pwd module to fail

Changes by Matt Frank : -- nosy: +WanderingLogic ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue22719] os.path.isfile & os.path.exists bug in while loop

eryksun added the comment: When appending to a singly-referenced string, the interpreter tries to reallocate the string in place. This applies to both `s += 'text'` and `s = s + 'text'`. Storing to a temp variable is adding a 2nd reference, so a new string gets allocated instead. If the former

[issue22672] float arguments in scientific notation not supported by argparse

Changes by Ned Deily : -- stage: -> patch review versions: +Python 3.5 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

[issue22672] float arguments in scientific notation not supported by argparse

Ned Deily added the comment: Thanks for the updated patch and test program. Note, to be part of the Python test suite, the test should be in the form of a unittest test and fit into the existing argparse tests here: Lib/test/test_argparse.py. -- nosy: +ned.deily _

[issue22672] float arguments in scientific notation not supported by argparse

Jacopo Nespolo added the comment: test program that checks handling of a bunch of numbers. -- Added file: http://bugs.python.org/file37008/test_argparse.py ___ Python tracker ___

[issue22672] float arguments in scientific notation not supported by argparse

Jacopo Nespolo added the comment: a better patch -- Added file: http://bugs.python.org/file37007/argparse.patch ___ Python tracker ___ ___

[issue22723] visited-link styling is not accessible

Georg Brandl added the comment: Sounds good to me. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue22723] visited-link styling is not accessible

Changes by Berker Peksag : -- nosy: +berker.peksag, georg.brandl ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22665] shutil.__all__ incomplete

Martin Panter added the comment: Looking at Lib/test/test___all__.py, that module just makes sure that the names imported with “from ... import *” are the same as listed in the __all__ attribute. It does no know what the __all__ attribute is meant to contain. -- __

[issue22723] visited-link styling is not accessible

Ned Deily added the comment: +1 It's a fairly subtle change but definitely an improvement IMO (as viewed with vanilla Safari and Firefox settings on OS X). -- nosy: +ned.deily ___ Python tracker _

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

Changes by Berker Peksag : -- stage: needs patch -> resolved ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue22723] visited-link styling is not accessible

Buck Golemon added the comment: Proposed patch attached. -- keywords: +patch Added file: http://bugs.python.org/file37006/link-color.patch ___ Python tracker ___

[issue22722] inheritable pipes are unwieldy without os.pipe2

Buck Golemon added the comment: I notice that dup2 grew an `inheritable=True` argument in 3.4. This might be a good precedent to use here, as a third option. -- ___ Python tracker __

[issue22723] visited-link styling is not accessible

New submission from Buck Golemon: The color needs adjusted such that it has at least 3:1 luminance contrast versus the surrounding non-link text. (See "non-inheritable" https://docs.python.org/3/library/os.html#os.dup) See also: * http://www.w3.org/TR/WCAG20/#visual-audio-contrast-without-col

[issue22719] os.path.isfile & os.path.exists bug in while loop

Aaron added the comment: If I use a separate temp variable, the bug doesn't show, but if I use the same variable, even with + instead of +=, it still happens. >>> bak_path = r"C:\Users\EAARHOS\Desktop\Python Review\baseExcel.py" >>> print(os.stat(bak_path)) nt.stat_result(st_mode=33206, st_ino=8

[issue22719] os.path.isfile & os.path.exists bug in while loop

Aaron added the comment: Interesting. It continues to reuse the last one's stats once the path is no longer valid. >>> bak_path = r"C:\Users\EAARHOS\Desktop\Python Review\baseExcel.py" >>> print(os.stat(bak_path)) nt.stat_result(st_mode=33206, st_ino=8162774324652726, st_dev=0, st_nlink=1, st_ui

[issue22719] os.path.isfile & os.path.exists bug in while loop

eryksun added the comment: What do you get for os.stat? bak_path = r"C:\Users\EAARHOS\Desktop\Python Review\baseExcel.py" print(os.stat(bak_path)) bak_path += '.bak' print(os.stat(bak_path)) bak_path += '.bak' print(os.stat(bak_path)) # This should raise FileNotFoundError

[issue22722] inheritable pipes are unwieldy without os.pipe2

R. David Murray added the comment: pipe2 can also be used to set O_NONBLOCK, so no, that specific API change wouldn't obsolete pipe2. So, a shim for pipe2 would be the way to go, since that's currently the de-facto standard way of passing flags on pipe creation in unix land. (The os module i

[issue22722] inheritable pipes are unwieldy without os.pipe2

New submission from Buck Golemon: In order to make an inheritable pipe, the code is quite a bit different between posixes that implement pipe2 and those that don't (osx, mainly). I believe the officially-supported path is to call os.pipe() then os.setinheritable(). This seems objectionable sin

[issue22720] Obscure error w/ windows online-only file

R. David Murray added the comment: We may not be getting any better error message from the OS. -- components: +Windows -IO nosy: +r.david.murray, steve.dower, tim.golden, zach.ware ___ Python tracker __

[issue22719] os.path.isfile & os.path.exists bug in while loop

R. David Murray added the comment: Looking at the code, it looks like it calls the win32 api directly if path->wide is true, which I'm guessing is the case unless you are using bytes paths in windows? It looks like the critical call, then, is CreateFileA (why A in a _w method I have no idea..

[issue22719] os.path.isfile & os.path.exists bug in while loop

Serhiy Storchaka added the comment: Could we encode both paths to the unicode_internal encoding and check if results are equal? -- nosy: +serhiy.storchaka ___ Python tracker ___

[issue22721] pprint output for sets and dicts is not stable

New submission from Serhiy Storchaka: pprint() sorts the content of sets and dicts in order to get stable output which doesn't depend on iteration order of set or dict, which depend not only from values of elements, but also from set or dict history. But in some cases the output is different f

[issue22719] os.path.isfile & os.path.exists bug in while loop

Steve Dower added the comment: I wonder whether the same thing occurs if you're not appending a new extension each time? There could be some optimisation (from the dark old days of 8.3 filename) that compares "baseExcel" and ".bak" separately and assumes that the name is known. Last I looked

[issue22720] Obscure error w/ windows online-only file

New submission from kg*2: Windows has integrated onedrive into their os. A feature of onedrive is allowing users to browse files that aren't downloaded onto their computer. os.listdir will return the items, but when they are opened for read/writing it throws IOError[22]: invalid mode or file n

[issue22719] os.path.isfile & os.path.exists bug in while loop

R. David Murray added the comment: Interesting bug. The obvious difference between the two cases is that in the += version the address of the string pointing to the filepath doesn't change, whereas when you use a temp variable it does (there's an optimization in += that reuses the same memory

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

Guido van Rossum added the comment: I've applied the second version -- rev 9b715d8246db in the peps repo. -- assignee: -> gvanrossum nosy: +gvanrossum resolution: not a bug -> fixed ___ Python tracker

[issue22665] shutil.__all__ incomplete

Éric Araujo added the comment: IIRC test___all__ can be used for this. -- nosy: +eric.araujo ___ Python tracker ___ ___ Python-bugs-li

[issue22719] os.path.isfile & os.path.exists bug in while loop

Changes by Aaron : -- title: os.path.isfile & os.path.exists but in while loop -> os.path.isfile & os.path.exists bug in while loop ___ Python tracker ___ __

[issue22718] pprint not handline uncomparable dictionary keys, set members well

R. David Murray added the comment: Actually, it applies to datetime, which is is almost unique among python2 datatypes in not being comparable to other types by default. I'm not sure this is worth fixing in 2.7. -- nosy: +r.david.murray ___ Python

[issue22719] os.path.isfile & os.path.exists but in while loop

New submission from Aaron: When using os.path.isfile() and os.path.exists() in a while loop under certain conditions, os.path.isfile() returns True for paths that do not actually exist. Conditions: The folder "C:\Users\EAARHOS\Desktop\Python Review" exists, as do the files "C:\Users\EAARHOS\De

[issue22718] pprint not handline uncomparable dictionary keys, set members well

Andreas Kostyrka added the comment: This also applies to sets. -- title: pprint not handline uncomparable dictionary keys well -> pprint not handline uncomparable dictionary keys, set members well ___ Python tracker

[issue22718] pprint not handline uncomparable dictionary keys well

New submission from Andreas Kostyrka: >>> import datetime, pprint >>> pprint.pformat({datetime.datetime.now(): 1, None: 1}) Traceback (most recent call last): File "", line 1, in File "/usr/lib/python2.7/pprint.py", line 63, in pformat return PrettyPrinter(indent=indent, width=width, d

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

Akira Kitada added the comment: Good point. I looked at some Python files but, as you pointed out, I couldn't find ones what follow the recommendation. In some files there is a blank line after a docstring but it seemed it's not because it "is written as a number of sections". The patch updated.

[issue22717] PySSL segmentation fault

Changes by R. David Murray : -- resolution: -> third party stage: -> resolved status: open -> closed ___ Python tracker ___ ___ Pyth

[issue22717] PySSL segmentation fault

mbasti added the comment: Thank you for your fast reply! So I will file a fedora bug. Thanks again. -- ___ Python tracker ___ ___ Pyt

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

R. David Murray added the comment: Yeah, on reflection (after I posted) I can see why people tend to leave out that blank line, because it is more parallel to not having a blank line after the 'def'. And I don't think I've ever seen anyone do the alternate version of keeping things parallel b

[issue22717] PySSL segmentation fault

STINNER Victor added the comment: Other Fedora patches on Python: https://apps.fedoraproject.org/packages/python/sources I see another *huge* patch on the ssl module, "ssl backport": http://pkgs.fedoraproject.org/cgit/python.git/tree/00196-ssl-backport.patch --

[issue22717] PySSL segmentation fault

STINNER Victor added the comment: > 317 self->ctx->options &= ~(SSL_OP_NO_SSLv2 | SSL_OP_NO_SSLv3); I don't see this line in Python 2.7.8 vanilla: https://hg.python.org/cpython/file/ee879c0ffa11/Modules/_ssl.c It looks like Fedora patched the source code: http://pkgs.fedoraproject.org

[issue18156] Add an 'attr' attribute to AttributeError

Serhiy Storchaka added the comment: In issue22716 proposed to add a reference to the object missing an attribute. -- nosy: +serhiy.storchaka ___ Python tracker ___ __

[issue22717] PySSL segmentation fault

Alex Gaynor added the comment: Are you able to test this against the 2.7 branch from hg? The ssl module received some significant attention for 2.7.9. -- nosy: +alex, christian.heimes, dstufft, giampaolo.rodola, janssen, pitrou ___ Python tracker

[issue22716] Add reference to the object missing an attribute to AttributeError

Serhiy Storchaka added the comment: This is a duplicate of issue18156. -- nosy: +serhiy.storchaka resolution: -> duplicate stage: -> resolved status: open -> closed superseder: -> Add an 'attr' attribute to AttributeError ___ Python tracker

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

Akira Kitada added the comment: Guido is with me :) On Wed, Oct 22, 2014 at 2:16 AM, Guido van Rossum wrote: > I think the argument is about the blank line between the "class" line and > the docstring. I agree with Akira that very few packages follow this style, > and I think we would do well t

[issue22717] PySSL segmentation fault

New submission from mbasti: Hello, I'm getting null pointer dereference which leads to segmentation fault. I have no stable reproducer, but don't hesitate to contact me. Additional info is here: https://fedorahosted.org/freeipa/ticket/4649 python 2.7.8-4.1 (Fedora 21) #0 0x7f3c4a66dde4 i

[issue22716] Add reference to the object missing an attribute to AttributeError

New submission from flying sheep: Trying to replicate a Ruby Gem that raises a “did you mean” error when mistyping a method name, I hit a showstopper: There seems to be no way to get the object which misses an attribute from an AttributeError. I propose the appended patch (it might be complet

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

R. David Murray added the comment: I follow this recommendation. I'm sure there are many classes in the stdlib that do not follow this convention, since much of the stdlib was written before PEP8, and we don't go back and do changesets just to "fix" PEP8 non-conformance in the stdlib. But th

[issue22715] PEP 257: drop the recommendation for a blank line between the class line and the docstring

New submission from Akira Kitada: The PEP 257 says: Insert a blank line before and after all docstrings (one-line or multi-line) that document a class Looking at stdlib and popular open source software, I couldn't find packages that follow this style, so I think this is not really a con

[issue22410] Locale dependent regexps on different locales

Serhiy Storchaka added the comment: If there are no objections I'll commit the patch. -- assignee: -> serhiy.storchaka ___ Python tracker ___ ___

[issue17381] IGNORECASE breaks unicode literal range matching

Serhiy Storchaka added the comment: Does the patch look good now for you Antoine? If there are no objections I'm going to commit it soon. In order to apply 3.4 patch to 2.7 we need either significant modify the patch, or first backport issue19329 changes to 2.7 (it would be easier). -

[issue814253] Grouprefs in lookbehind assertions

Serhiy Storchaka added the comment: The patch for issue9179 fixes this issue too. -- nosy: +serhiy.storchaka superseder: -> Lookback with group references incorrect (two issues?) ___ Python tracker __

[issue9179] Lookback with group references incorrect (two issues?)

Serhiy Storchaka added the comment: The patch also fixes issue814253. If there are no objections I'll commit it soon. -- ___ Python tracker ___ __

[issue22560] Add loop-agnostic SSL implementation to asyncio

STINNER Victor added the comment: I will try to take a look next week. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Uns