paul j3 added the comment:
Changing the test from
if argument_values is not action.default:
to
if argument_values is not action.default and \
(action.default is None or argument_values != action.default):
makes the behavior more consistent. Strings and large ints behave
Roundup Robot added the comment:
New changeset 44ed0cd3dc6d by Senthil Kumaran in branch '2.7':
Fix the dead link of IEEE_854-1987 standard with the Wikipedia entry.
http://hg.python.org/cpython/rev/44ed0cd3dc6d
New changeset 0eef1670f316 by Senthil Kumaran in branch '3.3':
Fix the dead link of
Senthil Kumaran added the comment:
Fixed that in 2.7,3.3 and 3.4
--
nosy: +orsenthil
resolution: - fixed
stage: - committed/rejected
status: open - closed
versions: +Python 2.7, Python 3.3
___
Python tracker rep...@bugs.python.org
Ethan Furman added the comment:
Run the test suite both with and without the patch, and compare the results.
Additional skipped tests, additional failed tests, or less than the number of
expected additional tests signal a problem.
The first two should be automatable, the last depends on the
Dwayne Litzenberger added the comment:
@amk: I'd appreciate it if you did. :)
I ran into this bug while writing some code that converts b... into ... in
PyCrypto's setup.py script (for backward compatibility with Python 2.5 and
below).
--
nosy: +DLitz
Nick Coghlan added the comment:
The one I ran into earlier today tried to merge the entirety of the
3.3.x RC NEWS into the 3.4 NEWS :P
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18967
Charles-François Natali added the comment:
R. David Murray added the comment:
See also issue issue 17409.
The code isn't setting it to RLIM_INFINITY explicitly, though, so this must
mean that OSX is reporting an infinite hard limit when the hard limit is not
in fact infinite. Seems
Ned Deily added the comment:
As a simple check, does the following work on OSX ?
limit = resource.getrlimit(resource.RLIMIT_NOFILE)
resource.setrlimit(resource.RLIMIT_NOFILE, limit)
It doesn't produce an exception.
Does the attached patch solve this?
With the patch, test_selectors no
Roundup Robot added the comment:
New changeset 0e52b9f77dbf by Charles-François Natali in branch 'default':
Issue #18934: Use poll/select-based selectors for multiprocessing.Connection,
http://hg.python.org/cpython/rev/0e52b9f77dbf
--
___
Python
Roundup Robot added the comment:
New changeset 9ba1432fdc5a by Charles-François Natali in branch 'default':
Issue #18963: Fix test_selectors.test_above_fd_setsize on OS X, where the
http://hg.python.org/cpython/rev/9ba1432fdc5a
--
nosy: +python-dev
STINNER Victor added the comment:
Do you mean something like *this*?
this - the patch, pythonfaulthandler_env_var.patch.
Thanks for the patch, I combined it with other changes.
--
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 5c2cf4349adc by Victor Stinner in branch 'default':
Close #18957: The PYTHONFAULTHANDLER environment variable now only enables the
http://hg.python.org/cpython/rev/5c2cf4349adc
--
nosy: +python-dev
resolution: - fixed
stage: -
Roundup Robot added the comment:
New changeset b13cec63b495 by Victor Stinner in branch 'default':
Issue #18904: Improve os.get/set_inheritable() tests
http://hg.python.org/cpython/rev/b13cec63b495
--
nosy: +python-dev
___
Python tracker
STINNER Victor added the comment:
Thanks for the report and the patch. I removed the duplicate test, but I also
added new tests using fcntl and FD_CLOEXEC.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18904
New submission from Charles-François Natali:
Currently, the test suite, as well as processes spawned by the
script_helper.assert_python family, are run with faulthandler enabled.
That's great to debug crashes, but it would be even better if those processes
were started with faulthandler's
Roundup Robot added the comment:
New changeset b7f6f6f59e91 by Victor Stinner in branch 'default':
Issue #18904: test_socket: add inheritance tests using fcntl and FD_CLOEXEC
http://hg.python.org/cpython/rev/b7f6f6f59e91
--
___
Python tracker
Charles-François Natali added the comment:
Anyone? (The patch is trivial, but I'm not familiar with regrtest).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18935
___
STINNER Victor added the comment:
I see two options:
* faulthandler calls killpg(SIGABRT) on timeout to kill child processes (but it
should ignore temporary the signal to not kill itself)
* use a timeout, but shorter than the global timeout, for child processes
Not all tests use
STINNER Victor added the comment:
The patch looks good to me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18935
___
___
Python-bugs-list
Charles-François Natali added the comment:
I knew this wouldn't be so easy with OS X...
http://buildbot.python.org/all/builders/x86%20Tiger%203.x/builds/6916/steps/test/logs/stdio
==
ERROR: test_above_fd_setsize
Roundup Robot added the comment:
New changeset 2982ac8b45bc by Charles-François Natali in branch 'default':
Issue #18935: Fix test_regrtest.test_timeout when built --without-threads (the
http://hg.python.org/cpython/rev/2982ac8b45bc
--
nosy: +python-dev
Roundup Robot added the comment:
New changeset fa735675e485 by Charles-François Natali in branch 'default':
Issue #18963: skip test_selectors.test_above_fd_setsize on older OS X versions.
http://hg.python.org/cpython/rev/fa735675e485
--
___
Python
Antoine Pitrou added the comment:
Indeed the Ubuntu Shared buildbot started failing again. There's probably a
timing-dependent behaviour here (which is why test_is_alive_after_fork() tries
several times, after all).
--
___
Python tracker
Antoine Pitrou added the comment:
I think I've found the answer: the thread is sometimes already stopped by the
time the child is forked, so it doesn't appear in _enumerate() anymore (it left
the _active dict). Therefore its locks are not reset in _after_fork().
Oh, I also get the following
Vajrasky Kok added the comment:
Well, what about None?
$ python3 -c 'import csv; reader = csv.reader(foo, delimiter=None)'
Traceback (most recent call last):
File string, line 1, in module
TypeError: delimiter must be set
English grammatically speaking, we should get this kind of error:
Charles-François Natali added the comment:
Committed.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18935
Roundup Robot added the comment:
New changeset 74dc664ad699 by Antoine Pitrou in branch 'default':
Issue #18808 again: fix the after-fork logic for not-yet-started or
already-stopped threads.
http://hg.python.org/cpython/rev/74dc664ad699
--
___
koobs added the comment:
For reference, this test is successfuly identifying failures on koobs-freebsd
and koobs-freebsd10 buildbots:
==
FAIL: test_is_alive_after_fork (test.test_threading.ThreadTests)
koobs added the comment:
Adding reference to failing tests on koobs-freebsd9 and koobs-freebsd10
buildbots:
==
FAIL: test_is_alive_after_fork (test.test_threading.ThreadTests)
Antoine Pitrou added the comment:
I've identified a possible cause (and fix) for the sporadic test failures in
74dc664ad699. If the solution holds, it should be backported to 3.3 and 2.7.
--
___
Python tracker rep...@bugs.python.org
Terry J. Reedy added the comment:
A possible complication is that at least in part, the same code
(AutoComplete.py) is used for both attribute completion and filename
completion. Attribute completion is always case sensitive.
--
___
Python tracker
Wouter Bolsterlee added the comment:
An additional speedup would be to add a if len(h) == 1 check inside the while
loop, and just yield from the remaining iterator if a single iterable remains.
This would also speed up merges with multiple inputs, as it doesn't do the
whole heapreplace() loop
Roundup Robot added the comment:
New changeset aea58e1cae75 by Victor Stinner in branch 'default':
Issue #18904: test_os and test_socket use unittest.skipIf() to check if fcntl
http://hg.python.org/cpython/rev/aea58e1cae75
--
___
Python tracker
Charles-François Natali added the comment:
Alright, it should be fixed now, thanks for the report.
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
type: - behavior
___
Python tracker rep...@bugs.python.org
Changes by Ben Finney ben+pyt...@benfinney.id.au:
--
nosy: +bignose
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12916
___
___
Python-bugs-list
Changes by Lukas Wunner lu...@wunner.de:
Added file: http://bugs.python.org/file31662/run_setup-py31.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18970
___
New submission from Lukas Wunner:
The principle of least surprise suggests that run_setup() should behave
equivalently to a command line invocation of setup.py. However there are
currently (at least) two issues preventing this:
(a) When calling exec(), both a globals and a locals dict is
Changes by Lukas Wunner lu...@wunner.de:
Added file: http://bugs.python.org/file31664/run_setup-py26.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18970
___
Changes by Lukas Wunner lu...@wunner.de:
Added file: http://bugs.python.org/file31663/run_setup-py27.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18970
___
Changes by Andrea Corbellini corbellini.and...@gmail.com:
--
versions: +Python 3.1, Python 3.2, Python 3.3, Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18454
___
R. David Murray added the comment:
Were you changing something in core/builtins? I'm wondering if just making
some same-between-versions space between the version header/date and the
beginning of that section would be enough to solve 99% of the problems.
--
R. David Murray added the comment:
delimiter must be a 1 character string would cover it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18829
___
New submission from Serhiy Storchaka:
Here is a patch which replaces optparse to argparse in the profile and cProfile
modules.
--
components: Library (Lib)
files: profile_argparse.patch
keywords: patch
messages: 197262
nosy: bethard, georg.brandl, serhiy.storchaka
priority: normal
New submission from Serhiy Storchaka:
Here is a patch which replaces optparse to argparse in the
Doc/includes/email-dir.py and Doc/includes/email-unpack.py scripts.
--
assignee: docs@python
components: Documentation, email
files: email_examples_argparse.patch
keywords: patch
messages:
New submission from Serhiy Storchaka:
Here is a patch which replaces optparse to argparse in the calendar modules.
--
components: Library (Lib)
messages: 197264
nosy: bethard, rhettinger, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Use
Vajrasky Kok added the comment:
David R. Murray said, 'delimiter must be a 1 character string would cover it.'
You mean
$ ./python -c 'import csv; reader = csv.reader(foo, delimiter=)'
should give this error 'delimiter must be a 1 character string'?
Attached the patch to accommodate your
Vajrasky Kok added the comment:
Sorry for typing your name wrongly.
s/David R. Murray/R. David Murray/
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18829
___
A.M. Kuchling added the comment:
Here's a patch implementing traceback.clear_tb_frames(). (Feel free to
bikeshed about the name.)
One more substantial question: the top frame of the traceback is possibly still
running. Currently the code skips it by doing an initial 'tb = tb.tb_next'.
Vajrasky Kok added the comment:
I don't see any patches. You forgot to upload the patch?
--
nosy: +vajrasky
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18973
___
New submission from Serhiy Storchaka:
Here is a patch which replaces optparse to argparse in the
Tools/scripts/diff.py script.
--
components: Demos and Tools
files: diff_argparse.patch
keywords: patch
messages: 197269
nosy: serhiy.storchaka
priority: normal
severity: normal
stage:
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +bethard
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18974
___
___
Python-bugs-list
Serhiy Storchaka added the comment:
You forgot to upload the patch?
Again.
--
keywords: +patch
Added file: http://bugs.python.org/file31670/calendar_argparse.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18973
Vajrasky Kok added the comment:
After contemplating for a while, I am not sure that we should treat empty
string for delimiter with error message: 'delimiter must be an 1-character
string'.
The reason is to keep consistency with quotechar keyword.
[sky@localhost cpython]$ ./python -c 'import
Raymond Hettinger added the comment:
Try this patch.
--
Added file: http://bugs.python.org/file31671/merge.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18962
___
R. David Murray added the comment:
Parsing a csv file with no delimiter would seem to be meaningless, unless I'm
misunderstanding what 'delimeter' controls. So the error messages for
delimiter and quotechar are necessarily different.
--
___
Python
New submission from Jakub Stasiak:
This patch includes:
* making code more PEP8-compatible and refactoring it a bit
* printing number of loops per second when using command line interface
* using thousands separators when printing numbers of loops (also in command
line interface)
* changing
Changes by Akira Kitada akit...@gmail.com:
--
nosy: +akitada
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12317
___
___
Python-bugs-list mailing
paul j3 added the comment:
A possibly unintended consequence to this `seen_non_default_actions` testing is
that default values do not qualify as 'present' when testing for a required
mutually exclusive group.
p=argparse.ArgumentParser()
g=p.add_mutually_exclusive_group(required=True)
Jakub Stasiak added the comment:
Oops, forgot to patch the tests, please find correct patch attached.
--
Added file: http://bugs.python.org/file31673/timeit-v2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18975
New submission from Benedikt Morbach:
At
http://hg.python.org/cpython/file/1043cc2cb0ff/Lib/distutils/command/build_ext.py#l247
build_ext.py compares sys.executable against sys.exec_prefix.
When cross compiling cpython, it notices that the interpreter running the build
is located at
Antoine Pitrou added the comment:
One more substantial question: the top frame of the traceback is
possibly still running. Currently the code skips it by doing an
initial 'tb = tb.tb_next'. Would it be better to catch and ignore the
RuntimeError
from frame.clear()?
Yes, I think it would
New submission from Serhiy Storchaka:
The uu module can be run as command-line tool. In Python 2, uu -t infile
outfile opens input file in text mode, i.e. convert '\r\n' to '\n' on Windows
and '\r' to '\n' on Mac Classic, and uu -d -t infile outfile opens output
file in text mode, i.e.
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +tim.peters
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18975
___
___
Python-bugs-list
Ethan Furman added the comment:
Okay, taking a step back.
It seems that currently inspect is geared towards instances and classes, not
metaclasses. Consequently, so is help.
So, how do we enhance inspect so that help can be metaclass aware?
classify_class_attrs seems like an obvious choice,
Ezio Melotti added the comment:
I personally dislike the , as thousands separator, and if a separator is
added at all I would prefer a space as defined in the SI standard[0].
The PEP8 changes should also me moved to a separate patch IMHO; the other
changes are OK grouped together.
[0]:
New submission from Jason R. Coombs:
In Python 2.x and 3.2, I used to use a Request subclass I created for
overriding the method used:
class MethodRequest(request.Request):
def __init__(self, *args, **kwargs):
Construct a MethodRequest. Usage is the
Senthil Kumaran added the comment:
Hi Jason,
Agree with you. This design change could be valuable in extending
urllib.request.Request
Thanks!
--
nosy: +orsenthil
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18978
Jason R. Coombs added the comment:
I've created a clone in which to draft this work.
--
hgrepos: +208
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18978
___
Antoine Pitrou added the comment:
It seems that currently inspect is geared towards instances and
classes, not metaclasses. Consequently, so is help.
I'm afraid I don't really understand what you're talking about. A
metaclass is just a slightly different kind of class :-)
def
Changes by Jason R. Coombs jar...@jaraco.com:
--
keywords: +patch
Added file: http://bugs.python.org/file31676/6d6d68c068ad.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18978
___
Jakub Stasiak added the comment:
I agree with both notes. Splitting the patch won't be a problem.
As much as I don't fancy , as thousands separator either - I just used what's
in the standard library but I'll think about the best way of putting spaces
there.
--
Changes by Jason R. Coombs jar...@jaraco.com:
Added file: http://bugs.python.org/file31677/2b2744cfb08f.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18978
___
Antoine Pitrou added the comment:
I personally dislike the , as thousands separator, and if a
separator is added at all I would prefer a space as defined in the SI
standard[0].
Then you really want a non-breaking space ;-)
(as a French person who's used to commas as decimal points, count
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +brett.cannon
versions: +Python 3.4 -Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12317
___
Changes by Jason R. Coombs jar...@jaraco.com:
Added file: http://bugs.python.org/file31678/061eb75339e2.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18978
___
Jason R. Coombs added the comment:
I've added tests to capture the new behavior.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18978
___
___
New submission from Serhiy Storchaka:
Here is a patch which replaces optparse to argparse in the uu module.
--
components: Demos and Tools, Library (Lib)
files: uu_argparse.patch
keywords: patch
messages: 197289
nosy: bethard, serhiy.storchaka
priority: normal
severity: normal
stage:
paul j3 added the comment:
I should add that defaults with required arguments (or groups?) doesn't make
much sense. Still there's nothing in the code that prevents it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18943
Tim Peters added the comment:
[Antoine]
Oh, I also get the following sporadic failure
which is triggered by slight change in semantics
with Thread.join(timeout) :-)
==
FAIL: test_various_ops
Roundup Robot added the comment:
New changeset 03ee22236465 by Serhiy Storchaka in branch '3.3':
Fixed tests with Tcl/Tk 8.5 (closes #18964).
http://hg.python.org/cpython/rev/03ee22236465
New changeset 138e086e187d by Serhiy Storchaka in branch 'default':
Fixed tests with Tcl/Tk 8.5 (closes
Antoine Pitrou added the comment:
That's because multiprocessing exits child processes with os._exit(), not
sys.exit().
The fix would be trivial (call threading._shutdown() before os._exit()), but I
don't know if that's something we want to do. After all there are many things
in the Python
Antoine Pitrou added the comment:
Le dimanche 08 septembre 2013 à 17:30 +, Tim Peters a écrit :
Really! In context, the test does:
t.join()
self.assertTrue(not t.is_alive())
Ah, no, the failing test did `t.join(something)`. I removed the timeout
to remove the
Armin Rigo added the comment:
Fwiw I agree with you :-) I'm just relaying a bug report that originates on
PyPy (https://bugs.pypy.org/issue1595).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18943
Roundup Robot added the comment:
New changeset c08e92529f62 by Serhiy Storchaka in branch '3.3':
Fix a typo. (closes #18953)
http://hg.python.org/cpython/rev/c08e92529f62
New changeset 9dc5bdab157e by Serhiy Storchaka in branch 'default':
Fix a typo. (closes #18953)
Serhiy Storchaka added the comment:
Thank you Vajrasky. Actually there was no need to open a new issue for such
minor bug. You can report on the same issue which caused this bug. I'm afraid
there are a lot of more grave syntax and grammatic errors in my Misc/NEWS
entries.
--
Guido van Rossum added the comment:
Here's a combined patch. Hopefully it will code review properly.
--
nosy: +gvanrossum
Added file: http://bugs.python.org/file31680/statistics_combined.patch
___
Python tracker rep...@bugs.python.org
Jason R. Coombs added the comment:
oh. My mistake. I didn't realize the error was in 'distutils.fancy_getopt' and
not in the getopt module itself. Indeed, the problem doesn't exist under getopt
itself, so is specific to distutils and fancy_getopt.
from __future__ import unicode_literals
Guido van Rossum added the comment:
Nice docstrings, but those aren't automatically included in the Doc tree.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18606
___
R. David Murray added the comment:
I think it is highly inadvisable to make changes like this without also adding
tests. We broke compileall by doing this even though we *did* add tests.
--
nosy: +r.david.murray
___
Python tracker
Tim Peters added the comment:
Ah - the test used to do t.join(NUMTASKS)! That's just bizarre ;-)
I believe I can repair that too (well - there was never a _guarantee_ that
waiting 10 seconds would be long enough), but I'll wait until this all settles
down.
join() and is_alive() are too
Serhiy Storchaka added the comment:
I'm afraid the adding any separators will make some people angry. With a comma
or space it no more a valid number in Python and many other languages and can't
be copy/pasted and parsed.
Actually I sometimes use small shell scripts to run python -m timeit
Antoine Pitrou added the comment:
join() and is_alive() are too complicated now, because of the 2-step
dance to check whether the thread is done: we have both an Event
(_stopped) and a lock (_tstate_lock) to check now. The Event doesn't
serve a purpose anymore: it's almost always
R. David Murray added the comment:
I'm with Serhiy on this. So if separators are added, I would say they *must*
be optional. Presumably if they are added they should be locale dependent :)
All of which may well make it more complicated than it is worth.
--
nosy: +r.david.murray
Tim Peters added the comment:
Without _stopped, join() can simply wait to acquire _tstate_lock (with or
without a timeout, and skipping this if _tstate_lock is already None). Etc ;-)
Of course details matter, but it's easy. I did it once, but the tests joining
the main thread failed and I
Antoine Pitrou added the comment:
Without _stopped, join() can simply wait to acquire _tstate_lock (with
or without a timeout, and skipping this if _tstate_lock is already
None). Etc ;-) Of course details matter, but it's easy. I did it
once, but the tests joining the main thread failed
Tim Peters added the comment:
-1 from me, and I'm a comma-loving American ;-)
I'm sure lots of code in the wild parses this output - Serhiy isn't the only
one doing it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18975
Stefan Behnel added the comment:
While refactoring the iterparse() implementation in lxml to support this new
interface, I noticed that the close() method of the XMLPullParser does not
behave like the close() method of the XMLParser. Instead of setting some .root
attribute on the parser
Tim Peters added the comment:
The MainThread class could override is_alive() and join(), then.
I think it will be easier than that, but we'll see ;-)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18808
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
Added file: http://bugs.python.org/file31681/merge2.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18962
___
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
Removed file: http://bugs.python.org/file31671/merge.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18962
___
1 - 100 of 142 matches
Mail list logo
