Larry Hastings added the comment:
Argh. I lost 1.5 day's worth of work on revision 6 of this patch last
night, due to me being tired and over-aggressively cleaning my working
directories. I will have to reconstruct it from memory, hopefully
Tuesday. (I basically know what I did, and going
Roundup Robot added the comment:
New changeset 779e5511d803 by Terry Jan Reedy in branch '2.7':
Issue #16655: Explain why Idle's test_calltips has 'fragile' tests of builtins.
http://hg.python.org/cpython/rev/779e5511d803
New changeset 4a505a901b2e by Terry Jan Reedy in branch '3.3':
Issue
Stefan Behnel added the comment:
Unless there are objections, I'll try to work on a patch that either
documents that PIs are lost, or optionally adds them to the tree when parsing
(depending on how difficult that turns out to be).
Please do. It should not be difficult at all to make the
Terry J. Reedy added the comment:
When I moved tests from CallTips.py to test_calltips.py, I reduced but did not
eliminate the use of builtins. I decided instead to explain in the file why
there is no substitute for the real thing, and how to fix a simple mismatch
like that reported here.
New submission from Dirkjan Ochtman:
See https://bugs.gentoo.org/show_bug.cgi?id=498640:
What compiler versions are supported for 3.3? We noticed an issue when
compiling 3.3.3 with gcc 4.3:
/usr/lib/gcc/x86_64-pc-linux-gnu/4.2.4/../../../../x86_64-pc-linux-gnu/bin/ld:
Georg Brandl added the comment:
Should be figured out for 3.3.4
--
nosy: +georg.brandl
priority: normal - release blocker
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20324
___
Vajrasky Kok added the comment:
Here is the patch for longobject. There are two sites which I couldn't convert.
The first is the constructor which is complicated. The other one is __round__
which clinic explicitly does not support.
--
Added file:
New submission from Tal Einat:
In Objects/bytes.c, the strip methods need a self converter so that
they get a PyBytesObject* instead of PyObject*. However, having set
this in bytes.strip and cloning that clinic definition for
bytes.lstrip and bytes.rstrip, it appears that the self converter
New submission from Larry Hastings:
Sorry this is so long--but I wanted to make my point. Here's the tl;dr summary.
The problem: The syntax used for Argument-Clinic-generated text
signatures for builtins means CPython mistakenly identifies
hand-written, unparsable pseudo-signatures as
Ram Rachum added the comment:
Christopher and Serhiy, I would appreciate if you could kindly explain why your
arguments, while applying to my suggestions, do not apply to the following
functions:
- `Path.stat`
- `Path.owner`
- `Path.group`
- `Path.open`
- `Path.chmod`
- `Path.lchmod`
-
Antoine Pitrou added the comment:
FWIW, I agree that shortcuts to easily create or read entire files are useful.
Other path classes (such as Twisted's) often have them.
--
versions: -Python 3.4
___
Python tracker rep...@bugs.python.org
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20197
___
___
Python-bugs-list
Changes by Vajrasky Kok sky@speaklikeaking.com:
--
title: Argument Clinic: self converters are not preserved when clonig functions
- Argument Clinic: self converters are not preserved when cloning functions
___
Python tracker
Larry Hastings added the comment:
Confirmed, and yes it's low priority.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20303
___
___
STINNER Victor added the comment:
Timings on my laptop:
import time, select
t0=time.perf_counter(); select.select([], [], [], 1e-6);
dt=time.perf_counter()-t0; dt
([], [], [])
0.00012494399561546743
t0=time.perf_counter(); select.select([], [], [], 0);
dt=time.perf_counter()-t0; dt
([],
Serhiy Storchaka added the comment:
Christopher and Serhiy, I would appreciate if you could kindly explain why
your arguments, while applying to my suggestions, do not apply to the
following functions:
1. Path.stat() wraps only one function, while Path.read() wraps two functions.
It's
Ram Rachum added the comment:
Serhiy:
Your arguments 1 and 2 are pretty weak. (So what if an import is required? It's
still 2 lines. I thought that Not every two line function are worth to be
added to the stdlib.)
Regarding stat being used much more often than read: I disagree. I've done
New submission from Tal Einat:
It would be useful for conversion of existing function to be able to set the
name of the C variable for arguments which is different than the Python
argument name.
For example, not being able to do so makes converting bytes.translate much
harder. In the
Changes by Jonathan Dowland jon+pyt...@alcopop.org:
--
type: - enhancement
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20328
___
___
New submission from Jonathan Dowland:
It would be great if the 'mailbox' library supported a 'remove' or 'delete'
operation for the mailbox itself as part of its abstract interface. It does
support removing messages, and the sub-classes for Maildir, MH etc. support
removing sub-folders (their
Stefan Krah added the comment:
Do you mean gcc-4.2? The patch uses __GNUC_MINOR__ 3.
On the FreeBSD build slave gcc-4.2 works:
gcc (GCC) 4.2.1 20070831 patched [FreeBSD]
Are you sure that there aren't any gcc patches for this version? It
looks like a gcc/binutils issue to me.
Changes by Daniel Szoska d.szo...@steinreichwald.de:
--
nosy: +Daniel.Szoska
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9291
___
___
Brett Cannon added the comment:
I'm not sure why you think the example code as-is should work. The first entry
on sys.path is the current directory ('' or the absolute path, depending if you
are running from the interpreter prompt or specifying a file on the
command-line). Stripping off
Serhiy Storchaka added the comment:
LGTM. Detecting function matches the WebP specification [1].
[1]
https://developers.google.com/speed/webp/docs/riff_container#webp-file-header
--
stage: - commit review
___
Python tracker rep...@bugs.python.org
Changes by Laurent Mazuel laurent.maz...@gmail.com:
--
nosy: +Laurent.Mazuel
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10614
___
___
Changes by Stefan Krah stefan-use...@bytereef.org:
--
title: gcc-4.3 support on python-3.3 (libmpdec) - gcc-4.2.4 support on
python-3.3 (libmpdec)
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20324
Paul Moore added the comment:
The most obvious solution would be if the onerror argument allowed for retries.
At the moment, all it can do is report issues, not recover. Suppose that
returning True from onerror meant retry the operation. Then you could do
def set_rw(operation, name, exc):
Changes by Jeremy Kloth jeremy.kloth+python-trac...@gmail.com:
--
nosy: +jkloth
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20326
___
___
New submission from Laurent Mazuel:
Hello,
Considering a zip file which contains utf-8 filenames (as uploaded zip file),
the following code fails if launched in a Posix shell.
with zipfile.ZipFile(test_ut8.zip) as fd:
... fd.extractall()
...
Traceback (most recent call last):
File
New submission from Masami HIRATA:
Although PEP 342 says foo(yield 42) is legal, Lib/test/test_grammar.py says:
# Requires parentheses as call argument
def g(): f((yield 1))
check_syntax_error(self, def g(): f(yield 1))
BTW, It seems that foo(yield 42) written in
R. David Murray added the comment:
Conceptually the problem with this is that if you use the object to delete the
folder, you now have an object with no folder...that is,
an invalid object. Do we have any other stdlib examples of objects with a
'delete()' method...looks like bdb has a
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20295
___
___
Glenn Langford added the comment:
Uploading proposed new version of as_completed() for review. Note the following
changes:
- does not install waiters for Futures which are completed
- locks only one Future at a time to improve concurrency (rather than locking
all Futures at once); traverses
Brett Cannon added the comment:
Here is fcntl converted. I didn't do fcntl.ioctl as it has a crazy set of
possible signatures which I simply didn't want to deal with.
--
Added file: http://bugs.python.org/file33591/fcntl_derby.diff
___
Python
R. David Murray added the comment:
See issue 8523 for a discussion of changing the way onerror behaves. I think
it is addressing this same use case, but I didn't reread it in detail.
--
___
Python tracker rep...@bugs.python.org
Paul Moore added the comment:
It's similar. But the problem is that it only returns a list of errors, it
doesn't let you address the error *while the rmtree is in progress*.
The key thing is that if you can fix the problem in onerror, you can avoid
needing to restart the whole tree walk,
R. David Murray added the comment:
If you live in a current-posix world, this might make sense. However, one can
also argue that the filename should be *transcoded* from the tarfile encoding
to the local FS filename encoding, which I believe is what we are currently
doing. Which, if you are
Guido van Rossum added the comment:
You have convinced me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20326
___
___
Python-bugs-list mailing
R. David Murray added the comment:
OK, rereading that issue, I disagree with Tarek and I think that the patch on
that issue is ill-advised as it looks like it changes behavior in a
non-backward-compatible way.
If you changed your set_rw onerror handler to a rm_ro_file error handler, would
Martin d'Anjou added the comment:
How about calling required arguments required arguments?
required arguments:
--reqarg REQARG, -r REQARG
This is required
optional arguments:
-h, --helpshow this help message and exit
--optarg OPTARG, -o OPTARG
R. David Murray added the comment:
Most PEPs are historical documents once they are final, so I don't think there
is actually anything to fix here. But I could be wrong.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
Stefan Krah added the comment:
I've been looking into this, and the only option provided by the Android NDK
involves cross-compiling Python on a regular machine. This would require some
custom commands before and in between the builds, to build a host Python first,
then cross-compile the
berdario added the comment:
yes, sorry... I tried to simplify and generalize it too much (I tried to
avoid creating a new directory in the test, assuming that the same behavior
could make sense by only creating a new module in the current directory)
I'll reupload the correct version of the
New submission from Serhiy Storchaka:
The proposed patch fixes potential fd leaks in various modules.
--
components: Library (Lib)
files: fd_leaks.patch
keywords: patch
messages: 208664
nosy: serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Fix
Paul Moore added the comment:
Looks like that works. At least in my case - I just did
def del_rw(action, name, exc):
os.chmod(name, stat.S_IWRITE)
os.remove(name)
shutil.rmtree(path, onerror=del_rw)
Something more robust might check if name is a directory and os.rmdir
Changes by berdario berda...@gmail.com:
Removed file: http://bugs.python.org/file33582/bbug.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20321
___
___
Antoine Pitrou added the comment:
I've been looking into this, and the only option provided by the
Android NDK involves cross-compiling Python on a regular machine. This
would require some custom commands before and in between the builds,
to build a host Python first, then cross-compile the
Stefan Krah added the comment:
This is the gcc bug:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46861
I rather agree with this comment:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46861#c3
If a distro still maintains 4.2.x, the toolchain should be fixed
(like FreeBSD apparently did).
berdario added the comment:
Ok, the bug is unrelated with timings and the finder caches apparently
--
resolution: invalid -
status: closed - open
versions: -3rd party, Python 2.7
___
Python tracker rep...@bugs.python.org
Brett Cannon added the comment:
Here is pyexpat converted. Couldn't do ErrorString as the 'l' format isn't
supported; issue #20332.
--
Added file: http://bugs.python.org/file33594/pyexpat_derby.diff
___
Python tracker rep...@bugs.python.org
New submission from Brett Cannon:
'l' is long for PyArg_ParseTuple().
--
assignee: larry
components: Build
messages: 208667
nosy: brett.cannon, larry
priority: normal
severity: normal
status: open
title: Argument Clinic does not support the 'l' format
versions: Python 3.4
Serhiy Storchaka added the comment:
In general the patch LGTM, but additional tests can be added.
Needed various tests for imghdr.what() arguments:
* First argument is bytes, int or None.
* First argument is a name of non-existent file.
* First argument is text stream.
* First argument is
New submission from Martin d'Anjou:
Consider the following code:
#!/usr/bin/env python3
import argparse
# create the top-level parser
parser = argparse.ArgumentParser(prog='PROG')
parser.add_argument('--file', help='A filename', required=True)
subparsers =
New submission from Yury Selivanov:
inspect.Signature and inspect.Parameter are immutable structures, and it makes
sense to make them hashable too.
Patch is attached.
--
components: Library (Lib)
files: hashable_signature_01.patch
keywords: patch
messages: 208671
nosy: brett.cannon,
Changes by berdario berda...@gmail.com:
Added file: http://bugs.python.org/file33596/bug_with_invalidatecaches.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20321
___
Yury Selivanov added the comment:
Fair enough.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20334
___
___
Python-bugs-list mailing list
Brett Cannon added the comment:
It actually is a caching issue, but not with the caches in the finder but the
cache *of* finders. Because you inserted LIBDIR before it existed, import
noticed it didn't exist and so put None into sys.path_importer_cache[LIBDIR]
(or imp.NullImporter prior to
Brett Cannon added the comment:
This is a new feature so it can't go into Python 3.4.
--
versions: +Python 3.5 -Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20334
___
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20329
___
___
Serhiy Storchaka added the comment:
What should be correct behavior? Raise an exception, return '[::1]spam' as
hostname, or left all as is?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20271
Changes by Berker Peksag berker.pek...@gmail.com:
--
stage: patch review - committed/rejected
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3073
___
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +bethard, paul.j3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20333
___
___
Berker Peksag added the comment:
Here's a backport of issue 16611 for 2.7.
--
keywords: +patch
nosy: +berker.peksag
stage: - patch review
versions: -Python 3.3
Added file: http://bugs.python.org/file33597/issue19870.diff
___
Python tracker
Georg Brandl added the comment:
OK, thanks for the quick response. Relaxing to normal priority, please close
as required.
--
priority: release blocker - normal
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20324
Dirkjan Ochtman added the comment:
Given this, WONTFIXing this seems reasonable to me, FWIW.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20324
___
Serhiy Storchaka added the comment:
LGTM, but may be write Py3k compatible names at first place?
--
stage: - commit review
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19863
Zachary Ware added the comment:
Larry Hastings wrote:
Does your proposed solution work properly when docstrings are turned
off? Try undefining WITH_DOC_STRINGS. I bet you need to make the
size 1 in that case.
For certain values of properly, yes. It builds with no warnings or errors on
Georg Brandl added the comment:
Thanks.
--
resolution: - wont fix
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20324
___
Serhiy Storchaka added the comment:
LGTM.
I have searched the sources and have not found any use of Py_BuildValue() which
can be used for test. Py_BuildValue() can fail only due to memory error. It can
also can fail with non-UTF8 string, but all uses of Py_BuildValue() with a
string in a
Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: docs@python - mark.dickinson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19863
___
Stefan Krah added the comment:
Larry Hastings rep...@bugs.python.org wrote:
I think we should change the syntax to something people would
never write by accident. Here are some suggestions:
*(
*clinic*(
\01 clinic(
I like the original def (...)\n approach from #19674. If that is not
Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: - zach.ware
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20075
___
___
Stefan Krah added the comment:
See also #20326.
--
nosy: +skrah
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20075
___
___
Python-bugs-list
Serhiy Storchaka added the comment:
What if the __text_signature__ and __doc__ getter will call ast.parse()
(actually compile()) on signature candidate? If it fails, then builtin has no
signature, the __text_signature__ getter returns '', and the __doc__ getter
returns all original docstring.
Tal Einat added the comment:
Never mind my previous comment; I realize it would create to much of a mess.
Sorry for the noise.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20187
___
Tal Einat added the comment:
While converting Objects/stringlib/transmogrify.h as part of issue20180 (Derby
#11), some changes to Objects/bytesobject.c and Objects/bytearrayobject.c were
required. Those changes are included in the relevant patch attached to that
issue.
--
nosy:
Serhiy Storchaka added the comment:
Actually, several other methods also have wrong parameter name, source
instead of string.
--
stage: needs patch - patch review
Added file: http://bugs.python.org/file33598/sre_pattern_string_keyword.patch
___
Changes by Serhiy Storchaka storch...@gmail.com:
Removed file: http://bugs.python.org/file33509/sre_pattern_string_keyword.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20283
___
Tal Einat added the comment:
I've been working on converting unicode, bytes and bytearray, but their files
are separated into Derby groups #10, #11 and #12. Could we create a new issue
and move the relevant files to it?
The files I'd like to move:
From Derby #10 (issue20179):
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +mrabarnett
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20283
___
___
Python-bugs-list
Roundup Robot added the comment:
New changeset da35a4e724e5 by Zachary Ware in branch '3.3':
Issue #20301: Mention the correct KEY_* value as the default
http://hg.python.org/cpython/rev/da35a4e724e5
New changeset 3baee069c5ff by Zachary Ware in branch 'default':
Issue #20301: Merge with 3.3
Zachary Ware added the comment:
Fixed, thanks for the report!
--
assignee: docs@python - zach.ware
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
STINNER Victor added the comment:
Thanks for the review Serhiy.
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20024
___
Serhiy Storchaka added the comment:
Gregory, is it intentional that Modules/_ctypes/libffi/build-ios.sh,
Modules/_ctypes/libffi/libtool-ldflags, Modules/_ctypes/libffi/msvcc.sh and
Modules/_ctypes/libffi/src/arm/gentramp.sh have no executable bits? They all
have executable bits in mainstream
Roundup Robot added the comment:
New changeset 29b4eb47f65e by Victor Stinner in branch 'default':
Issue #20024: Py_BuildValue() now saves/restores the current exception before
http://hg.python.org/cpython/rev/29b4eb47f65e
--
nosy: +python-dev
___
STINNER Victor added the comment:
parsermodule.patch was applied as 91cb83f895cf (Python 3.3) and cd952e92c180
(3.4).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20024
___
Roundup Robot added the comment:
New changeset 7ce7295393c2 by Victor Stinner in branch 'default':
Issue #20311: EpollSelector now also rounds the timeout towards zero, as
http://hg.python.org/cpython/rev/7ce7295393c2
--
___
Python tracker
Roundup Robot added the comment:
New changeset eb7565c212f1 by Serhiy Storchaka in branch '3.3':
Issue #2382: SyntaxError cursor ^ now is written at correct position in most
http://hg.python.org/cpython/rev/eb7565c212f1
New changeset ea34b2b0b8ae by Serhiy Storchaka in branch 'default':
Issue
Tal Einat added the comment:
Attached patch for AC conversion of Objects/bytesobject.c and
Objects/bytearrayobject.c.
This is one patch because there are changes that must be done in
bytes_methods.h and bytes_methods.c that affect both of bytesobject.c and
bytearrayobject.c. Those changes
Zachary Ware added the comment:
This is a duplicate of issue20227; I'm adding you to the nosy of that one, Tal.
--
nosy: +zach.ware
resolution: - duplicate
stage: - committed/rejected
status: open - closed
superseder: - Argument Clinic: rename arguments in generated C?
Changes by Zachary Ware zachary.w...@gmail.com:
--
nosy: +taleinat
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20227
___
___
Python-bugs-list
Tal Einat added the comment:
See nearly complete conversion of Objects/bytearrayobject.c in patch attached
to issue20179.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20181
___
Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: serhiy.storchaka -
stage: patch review - needs patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2382
___
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8260
___
___
Python-bugs-list
Roundup Robot added the comment:
New changeset d55d1cbf5f9a by Terry Jan Reedy in branch '2.7':
Issue #16630: Make Idle calltips work even when __getattr__ raises.
http://hg.python.org/cpython/rev/d55d1cbf5f9a
New changeset 2fe0b2dcc98c by Terry Jan Reedy in branch '3.3':
Issue #16630: Make
Tal Einat added the comment:
+1 for this. It would save a bit of manual conversion work, and would
significantly reduce the size and complexity of the resulting patches.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20227
Terry J. Reedy added the comment:
I added test_attribute_exception with two subtests for each of two nasty
classes. The resulting failures were fixed in CallTips.py by replacing all
ob.__call__ references with one reference inside try_except that saved the
method for possible later use.
Terry J. Reedy added the comment:
THanks for reporting this. #12520 tripled the number of tests in CallTips.py
and #20122 moved them to test_calltips.py. I think they cover this case and the
tests proposed. #16630 added a few more tests and patched CallTips.py to make
them pass. If you find
Larry Hastings added the comment:
If in C you define
static char a[5] = abcde
C suppresses the trailing '\0'. That it continued to work okay was a lucky
break--you must not have looked in many docstrings, or you lucked out and they
happened to be padded with zeroes.
--
Larry Hastings added the comment:
I had to throw it in a struct to prevent gcc from rearranging the variables.
But this demonstrates the problem--when it prints the string, it doesn't stop
at the end.
-
#include stdio.h
typedef struct
{
int a;
char b[8];
int c;
}
1 - 100 of 158 matches
Mail list logo
