STINNER Victor added the comment:
I will try to take a look next week.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22560
___
___
Serhiy Storchaka added the comment:
The patch also fixes issue814253.
If there are no objections I'll commit it soon.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9179
___
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 rep...@bugs.python.org
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).
Serhiy Storchaka added the comment:
If there are no objections I'll commit the patch.
--
assignee: - serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22410
___
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
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
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
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
Akira Kitada added the comment:
Guido is with me :)
On Wed, Oct 22, 2014 at 2:16 AM, Guido van Rossum guido at python.org 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
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
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
Serhiy Storchaka added the comment:
In issue22716 proposed to add a reference to the object missing an attribute.
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18156
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:
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
--
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
mbasti added the comment:
Thank you for your fast reply!
So I will file a fedora bug.
Thanks again.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22717
___
Changes by R. David Murray rdmur...@bitdance.com:
--
resolution: - third party
stage: - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22717
___
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.
New submission from Andreas Kostyrka:
import datetime, pprint
pprint.pformat({datetime.datetime.now(): 1, None: 1})
Traceback (most recent call last):
File stdin, line 1, in module
File /usr/lib/python2.7/pprint.py, line 63, in pformat
return PrettyPrinter(indent=indent, width=width,
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 rep...@bugs.python.org
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
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
Changes by Aaron hosfor...@gmail.com:
--
title: os.path.isfile os.path.exists but in while loop - os.path.isfile
os.path.exists bug in while loop
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22719
Éric Araujo added the comment:
IIRC test___all__ can be used for this.
--
nosy: +eric.araujo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22665
___
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 rep...@bugs.python.org
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
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
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 at
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
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 rep...@bugs.python.org
http://bugs.python.org/issue22719
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
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 rep...@bugs.python.org
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
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
eryksun added the comment:
What do you get for os.stat?
bak_path = rC:\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
Aaron added the comment:
Interesting. It continues to reuse the last one's stats once the path is no
longer valid.
bak_path = rC:\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_uid=0,
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 = rC:\Users\EAARHOS\Desktop\Python Review\baseExcel.py
print(os.stat(bak_path))
nt.stat_result(st_mode=33206,
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:
*
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 rep...@bugs.python.org
http://bugs.python.org/issue22722
Buck Golemon added the comment:
Proposed patch attached.
--
keywords: +patch
Added file: http://bugs.python.org/file37006/link-color.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22723
Changes by Berker Peksag berker.pek...@gmail.com:
--
stage: needs patch - resolved
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22715
___
___
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 rep...@bugs.python.org
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.
--
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berker.peksag, georg.brandl
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22723
___
Georg Brandl added the comment:
Sounds good to me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22723
___
___
Python-bugs-list mailing list
Jacopo Nespolo added the comment:
a better patch
--
Added file: http://bugs.python.org/file37007/argparse.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22672
___
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 rep...@bugs.python.org
http://bugs.python.org/issue22672
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
Changes by Ned Deily n...@acm.org:
--
stage: - patch review
versions: +Python 3.5
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22672
___
___
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
Changes by Matt Frank matthew.i.fr...@intel.com:
--
nosy: +WanderingLogic
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20306
___
___
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.
--
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
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
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.
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 rep...@bugs.python.org
http://bugs.python.org/issue22596
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +neologix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22697
___
___
Python-bugs-list mailing
Terry J. Reedy added the comment:
Currently, in 3.4 Win 7, tk 8.6, Alt O enter (or Alt O O O enter, etc) is
required to get the Format menu to drop down. Following that with Alt O does
select comment out. Alt O O enter (with or without holding down Alt) opens
the Option menu. Having to hit
Changes by Terry J. Reedy tjre...@udel.edu:
--
versions: +Python 3.5 -Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13319
___
___
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
Changes by Benedikt Morbach benedikt.morb...@googlemail.com:
--
versions: +Python 3.5, Python 3.6 -Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22724
___
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
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
Antoine Pitrou added the comment:
Hmm... is it important?
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22721
___
___
Martin Panter added the comment:
See also Issue 10976, discussing passing bytes() strings to json.loads()
--
nosy: +vadmium
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10390
___
Changes by Martin Panter vadmium...@gmail.com:
--
nosy: +vadmium
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19837
___
___
Python-bugs-list
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 rep...@bugs.python.org
http://bugs.python.org/issue10976
Changes by Martin Panter vadmium...@gmail.com:
--
nosy: +vadmium
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17909
___
___
Python-bugs-list
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
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 rep...@bugs.python.org
Changes by Berker Peksag berker.pek...@gmail.com:
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22723
___
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:
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
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
#
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berker.peksag
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10116
___
___
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
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti
versions: +Python 3.4, Python 3.6 -Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22725
___
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
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
Terry J. Reedy added the comment:
#22726 Add help to both dialogs
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3068
___
___
Python-bugs-list
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
Changes by R. David Murray rdmur...@bitdance.com:
--
stage: resolved - commit review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9351
___
___
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,
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
85 matches
Mail list logo