Stefan Krah added the comment:
Thanks for the report and the patch. I used another approach that still
validates the digits in the coefficient tuple even if it is not used.
decimal.py allows any coefficient:
Decimal((0, ('n', 'a', 'n'), 'F'))
Decimal('Infinity')
_decimal raises:
Changes by Stefan Krah stefan-use...@bytereef.org:
--
resolution: - fixed
stage: - committed/rejected
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15882
___
Andrew Moffat added the comment:
Sorry about that, I refactored out the string at the last minute and typed
testing123 instead of the original testing from the trace.
I'm not sure I follow the problem exactly. So you're saying that the failing
version uses _cancel sys calls, and that
Vitaly added the comment:
I wrote a C-language program to reproduce this issue on Mac OS without Python.
It reproduces the issue in both increasing and decreasing order of initial read
sizes, and it reliably reproduces it for each KB from 128KB to 1024KB ; the
Python version reproduced the
Stefan Krah added the comment:
Even though it's documented, the function is probably a new feature
and should go into 3.4.
Meanwhile, you can call struct.calcsize(format). Any non-trivial
implementation of PyBuffer_SizeFromFormat() would likely do the same.
--
components: +Interpreter
Vitaly added the comment:
The C-language program for reproducing the same issue is attached as
test_fork_pipe_error.cpp
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
___
Vitaly added the comment:
I used g++ to compile test_fork_pipe_error.cpp on both Mac OS and on Linux.
EINVAL showed up only on Mac OS.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
Charles-François Natali added the comment:
The reason I said above that those might be red-herring discoveries is this:
if I insert a short time.sleep(0.001) delay before the outer pipe-read loop,
the EINVAL errors don't get triggered either.
That's interesting.
So it really seems that
Changes by Charles-François Natali neolo...@free.fr:
--
type: crash - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
___
___
Vitaly added the comment:
By the way, the existing code in subprocess.Popen (at least on 2.6.7) reads the
pipe incorrectly: It doesn't loop to read all the data until EOF -- it only
loops over EINTR until it gets a single successful os.read() call. However,
since this is a pipe read (not a
Senthil Kumaran added the comment:
Hi Eduardo,
I tested further and do observe some very strange oddities.
On Mon, Sep 10, 2012 at 10:45 PM, Eduardo A. Bustamante López
rep...@bugs.python.org wrote:
Also, I'm aware that you shouldn't normally worry about setting a specific
user-agent to
Vitaly added the comment:
I filed this issue with apple: Problem ID: 12274650:
https://bugreport.apple.com/cgi-bin/WebObjects/RadarWeb.woa/64/wo/VE1RGG9qEL5OS9KdzFSDHw/19.66
--
___
Python tracker rep...@bugs.python.org
Vitaly added the comment:
Apple bug report URL correction:
https://bugreport.apple.com/cgi-bin/WebObjects/RadarWeb.woa/64/wo/VE1RGG9qEL5OS9KdzFSDHw/17.66
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
Vitaly added the comment:
Sorry, I don't know why the URL comes out all messed up. I can't seem to find
the correct syntax for this.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
Charles-François Natali added the comment:
By the way, the existing code in subprocess.Popen (at least on 2.6.7) reads
the pipe incorrectly: It doesn't loop to read all the data until EOF -- it
only loops over EINTR until it gets a single successful os.read() call.
However, since this is
New submission from Vitaly:
subprocess.Popen (at least on 2.6.7) reads the pipe incorrectly: It doesn't
loop to read all the data until EOF -- it only loops over EINTR until it gets a
single successful os.read() call. However, since this is a pipe read (not a
real file read), the system
Vitaly added the comment:
Filed http://bugs.python.org/issue15918 for the incorrect pipe read logic in
subprocess.Popen.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
___
Senthil Kumaran added the comment:
This is a feature request. urlencode tries to prepare the data in the form
items are submitted. It has been like form is list of key,value pairs and
that's what is reflect in the current behavior. That said, I am not -1 if see
if some frameworks adopting
Senthil Kumaran added the comment:
2012/9/10 Jesús Cea Avión rep...@bugs.python.org:
Ping!.
Guess, it is still for 3.4.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11943
___
Senthil Kumaran added the comment:
Thanks!. Verified the patch, it is fine and can be applied directly on
3.2 and 3.3 branch. Tested it as well. On 2.7, some changes would be
required.
--
nosy: +orsenthil
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 8a40bc71c072 by Senthil Kumaran in branch '3.2':
Fix issue #15899: Make the unicode.rst doctests pass. Patch by Chris Jerdonek.
http://hg.python.org/cpython/rev/8a40bc71c072
New changeset 1d9e89f6abec by Senthil Kumaran in branch 'default':
merge.
Ronald Oussoren added the comment:
What's wrong with working around this bug by reading a smaller amount? How much
data is there supposed to be?
BTW. URLs for reports in Apple's tracker are fairly useless as bugreports are
private (only you and Apple's engineers can see the report). The bug
Senthil Kumaran added the comment:
I shall make the changes 2.7, keeping this open for that.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15899
___
Chris Jerdonek added the comment:
Thanks, Senthil!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15899
___
___
Python-bugs-list mailing list
Chris Jerdonek added the comment:
Adding Eli to this because he did some work on the HOWTO for this module in
issue 14814.
--
nosy: +eli.bendersky
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15888
Changes by Łukasz Langa luk...@langa.pl:
--
assignee: - lukasz.langa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15803
___
___
Python-bugs-list
Changes by Jesús Cea Avión j...@jcea.es:
--
nosy: +jcea
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15917
___
___
Python-bugs-list mailing list
Jesús Cea Avión added the comment:
We need something like this.
I can not review, I am not familiar with mercurial hooks internals.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15917
New submission from Chris Jerdonek:
On hg.python.org, it seems like the entries on the log page don't
always link to the corresponding revision, for example some of the rows in--
http://hg.python.org/cpython/shortlog/1d9e89f6abec
This seems to happen whenever the revision description begins
Chris Jerdonek added the comment:
Andrew, do you think my changes to the patch are adequate given the response on
python-dev to your question?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15533
Christian Heimes added the comment:
+1 for the feature
--
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15917
___
___
Roundup Robot added the comment:
New changeset 4f2811e5e462 by Christian Heimes in branch 'default':
Issue #15900: Fixed reference leak in PyUnicode_TranslateCharmap()
http://hg.python.org/cpython/rev/4f2811e5e462
--
nosy: +python-dev
___
Python
Christian Heimes added the comment:
Yes, 3.2 and earlier are not affected.
Georg, I'm assigning the bug to you so you can decide if you like to cherry
pick the fix.
--
assignee: - georg.brandl
nosy: +georg.brandl
resolution: - fixed
stage: - committed/rejected
New submission from Chris Jerdonek:
This issue is to make the doctests in howto/regex.rst pass using vanilla
doctest. After this issue, 10 out of the 17 HOWTO's will pass with vanilla
doctest.
Patch attached.
--
assignee: docs@python
components: Documentation
files:
Roundup Robot added the comment:
New changeset 4754c4a710e6 by Christian Heimes in branch 'default':
Issue #15895: Fix FILE pointer leak in PyRun_SimpleFileExFlags() when filename
points to a pyc/pyo file and closeit is false.
http://hg.python.org/cpython/rev/4754c4a710e6
--
nosy:
André Anjos added the comment:
A question concerning this patch: is this going to be applied only to 3.3 or to
2.7 as well? Python-2.7.x also does not have this functionality which would be
interesting to get.
--
nosy: +anjos
___
Python tracker
Christian Heimes added the comment:
The bug was a 3.3 regression, possibly introduced with the merge of importlib.
I forgot to tick the 3,3regression keyword.
I've applied the fix. I'm leaving this ticket open and assigned to Georg for
3.3.0.
--
keywords: +3.3regression
resolution:
New submission from Christian Heimes:
Starting with Python 3.3 the select module access the uninitialized tv.tv_usec
member of a timeval struct. I don't see the point of initializing the local
variable long tv_usec from tv.tv_usec. The comment above the code states that
long tv_usec is
Brett Cannon added the comment:
I don't think sys.modules is the right place about a warning regarding
importlib.invalidate_caches(); this has nothing to do with sys.modules but
instead import itself. Probably something more along those lines would be more
fitting.
This is also mentioned the
Brett Cannon added the comment:
I'm going to guess this is a shortcoming of pdb when it comes to frozen modules
as I can get to the source using inspect (which pdb leans on)::
len(inspect.findsource(_frozen_importlib)[0])
1761
len(inspect.findsource(importlib._bootstrap)[0])
1761
Which are
Chris Jerdonek added the comment:
As an aside, I noticed a doctest affected by this in the urllib HOWTO:
url_values = urllib.parse.urlencode(data)
print(url_values)
name=Somebody+Herelanguage=Pythonlocation=Northampton
http://docs.python.org/dev/howto/urllib2.html#data
(search for the
Michael Foord added the comment:
Yep, interesting idea. In Python 3 you can't import unittest.mock without
importing unittest, so there could even be a default testcase to fall back
on. For the external release I'd rather not have unittest as a dependency, but
allowing a testcase to be
Michael Foord added the comment:
The only issue is that the testcase would have to be propagated to child mocks
too.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15860
___
New submission from Chris Jerdonek:
This issue is to make the doctests in howto/urllib2.rst pass using vanilla
doctest.
Patch attached.
--
assignee: docs@python
components: Documentation
files: issue-doctest-howto-urllib-1.patch
keywords: easy, patch
messages: 170305
nosy: cjerdonek,
Changes by Chris Jerdonek chris.jerdo...@gmail.com:
--
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15922
___
___
Roundup Robot added the comment:
New changeset c3539eb02470 by Christian Heimes in branch 'default':
Issue #15895: my analysis was slightly off. The FILE pointer is only leaked
when set_main_loader() fails for a pyc file with closeit=0. In the success case
run_pyc_file() does its own cleanup
Jesús Cea Avión added the comment:
Yes, 3.4.
I would hate to rush, in two years, because this issue was neglected during 18
months :)
No reason for not starting now.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11943
Antoine Pitrou added the comment:
Le mardi 11 septembre 2012 à 13:52 +, Jesús Cea Avión a écrit :
No reason for not starting now.
There's no point in being pushy, though. If you want to start, the
best thing is to work on the patch and update it.
--
Ezio Melotti added the comment:
Attached a set up script to reproduce a test environment for the hook.
Create an empty dir and run ``sh setup.sh`` in it. This will:
1) create a 'c1' subdir which is a cpython-like repo with the branches 2.7,
3.1, 3.2, default;
2) download and set up the
Eric Snow added the comment:
The relationship between invalidate_caches() and sys.modules is definitely
tenuous. However, my rationale was that people would look for an explanation
on why modifying sys.modules was not working as expected. The sys.modules doc
entry was the one that seemed to
Changes by Éric Araujo mer...@netwok.org:
--
nosy: +eric.araujo
versions: -Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15860
___
___
Éric Araujo added the comment:
2.7 and 3.2 are stable versions which only get bug fixes. 3.3 is in release
candidate stage, so this new feature can only go into 3.4. See the devguide
and PEPs for more info about the process we follow.
--
versions: +Python 3.4 -Python 3.3
Barry A. Warsaw added the comment:
Okay, this bug is clearly caused by the patch applied for issue 12776. Here's
the patch set url: http://hg.python.org/cpython/rev/74f6d87cd471
Now, if you look at this, I think this changes the semantics for non-string
default with a type converter, but the
Richard Oudkerk added the comment:
I get the same hang on Linux with Python 3.2.
For Windows the documentation does warn against starting a process as a side
effect of importing a process. There is no explicit warning for Unix, but I
would still consider it bad form to do such things as a
Richard Oudkerk added the comment:
Here is a reproduction without using multiprocessing:
create.py:
import threading, os
def foo():
print(Trying import)
import sys
print(Import successful)
pid = os.fork()
if pid == 0:
try:
t = threading.Thread(target=foo)
Changes by Richard Oudkerk shibt...@gmail.com:
--
type: crash - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15914
___
___
Vitaly added the comment:
What's wrong with working around this bug by reading a smaller amount? How
much data is there supposed to be?
This makes sense for working around the issue. Even in the blocking-read case,
such as in subprocess.Popen, attempting to read a 1MB chunk of data in a
Vitaly added the comment:
Also, attempting to read a 1MB chunk of data in a single os.read call forces
os.read() to unnecessarily allocate a huge !MB buffer (even if only for a short
lifetime). Using something like 4KB read calls in a loop is more practical
(and looping is necessary anyway).
Antoine Pitrou added the comment:
2.6 doesn't receive bug fixes anymore. Can you at least test with a recent 2.7?
--
nosy: +gregory.p.smith, pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15918
Antoine Pitrou added the comment:
I might be wrong, but the logic in your hook looks a bit complicated. Wouldn't
it be simpler to find all topological heads in the new csets (a topological
head is a cset without any child), and check that none of them is on a 3.*
branch?
Finding topological
Ezio Melotti added the comment:
I might be wrong, but the logic in your hook looks a bit complicated.
This might be true. The logic I followed is that once a cset is merged, it
becomes a parent of another cset (either in the same branch or in the next
one). So, if the cset is in 3.x and is
Changes by Vitaly vitaly.krugl.nume...@gmail.com:
--
versions: +Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15896
___
___
Richard Oudkerk added the comment:
Python 3.2 has extra code in _PyImport_ReInitLock() which means that when a
fork happens as a side effect of an import, the main thread of the forked
process owns the import lock. Therefore other threads in the forked process
cannot import anything.
Roundup Robot added the comment:
New changeset 6cdc72f4d82c by Benjamin Peterson in branch 'default':
remove useless and defined initialization (closes #15921)
http://hg.python.org/cpython/rev/6cdc72f4d82c
--
nosy: +python-dev
resolution: - fixed
stage: patch review -
Chris Jerdonek added the comment:
I'm not sure of all the implications of this, but it seems like this is a
relevant piece of information from the docs:
type= can take any callable that takes a single string argument and returns
the converted value:
(from
Ethan Furman added the comment:
RC2 has just been released. Any chance of this getting in to the final
release? Nobobdy has pointed out any problems with the last update...
--
___
Python tracker rep...@bugs.python.org
Chris Jerdonek added the comment:
Along the lines of my previous comment, I notice that the following str type
check was removed in the patch for issue 12776:
-if isinstance(action.default, str):
-default = self._get_value(action, default)
-
Richard Oudkerk added the comment:
It looks like the problem was caused be the fix for
http://bugs.python.org/issue9573
I think the usage this was intended to enable is evil since one of the forked
processes should always be terminated with os._exit().
--
Roundup Robot added the comment:
New changeset 957e1eef3296 by R David Murray in branch '3.2':
#14617: clarify discussion of interrelationship of __eq__ and __hash__.
http://hg.python.org/cpython/rev/957e1eef3296
New changeset c8d60d0c736b by R David Murray in branch 'default':
Merge #14617:
R. David Murray added the comment:
I rewrote the section a bit differently than you did in your patch...if you
think my changes are not an improvement please let me know.
--
nosy: +r.david.murray
resolution: - fixed
stage: - committed/rejected
status: open - closed
Gregory P. Smith added the comment:
fyi - i suspect Python 3.2 and the backport of that to 2.x
http://code.google.com/p/python-subprocess32/ do not have this issue.
but you didn't give enough information in the bug report for me to know which
pipe and which read call you're talking about to
Chris Jerdonek added the comment:
Here is a type of test case that I think should be considered for addition (to
confirm that the code doesn't double-convert strings in at least one case).
Maybe there is already a test case like this:
class MyString(str): pass
def convert(s):
return
R. David Murray added the comment:
turtle uses it because that was the file that I made work when I was playing
with 'make doctest'. I think being able to use the testsetup directive would
be good. It could also them be used (I think!) to put resource directives in
the docs that would
Changes by R. David Murray rdmur...@bitdance.com:
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15917
___
___
Roundup Robot added the comment:
New changeset 6fea947edead by Christian Heimes in branch 'default':
Updates NEWS for issue #15895
http://hg.python.org/cpython/rev/6fea947edead
--
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
I believe you've identified the broken part of the change, Chris. So to
restore previous behavior we need to add that back correctly.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
Changes by R. David Murray rdmur...@bitdance.com:
--
priority: high - deferred blocker
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15906
___
___
Vitaly added the comment:
My bug report refers to the following code block in subprocess.py. The problem
is the same in 2.6.7 and 2.7.3:
=== From subprocess.py in Python 2.7.3 Source Distribution:
# Wait for exec to fail or succeed; possibly raising exception
#
Ethan Furman added the comment:
R. David Murray wrote:
I rewrote the section a bit differently than you did in your patch...if you
think my changes are not an improvement please let me know.
This line is incorrect:
A class which defines its own :meth:`__hash__` that explicitly raises a
Vitaly added the comment:
Sorry, there is no traceback. The issue was identified in code review.
'man 2 read' states:
===
The system guaran-
tees to read the number of bytes requested if the descriptor references a
normal file that has that many bytes left before the end-of-file,
Ethan Furman added the comment:
Ethan Furman wrote:
Ethan Furman added the comment:
R. David Murray wrote:
I rewrote the section a bit differently than you did in your patch...if you
think my changes are not an improvement please let me know.
This line is incorrect:
A class which
Vitaly added the comment:
The prior 'man 2 read' quote was from Mac OS X;
On amazon (centos) Linux, 'man 2 read' makes the same claim, albeit in
different verbiage:
===
It is not an error if this number is smaller than the number of bytes
requested; this may happen for example because fewer
Gregory P. Smith added the comment:
Yeah, sounds like _eintr_retry_call alone isn't appropriate here in 2.7.
I'll fix it.
In practice I doubt this matters much as this error string is likely to be
less than one page (depends on pathnames involved) but it is still
technically incorrect as
Ethan Furman added the comment:
Can we also get this committed before 3.3.0 final?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15209
___
___
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +georg.brandl
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15209
___
___
Barry A. Warsaw added the comment:
On Sep 11, 2012, at 04:15 PM, Chris Jerdonek wrote:
I haven't thought about this very long, but what would happen if the type
conversion is only called on arguments and default arguments that are
strings, and otherwise left alone?
I thought about that, and
Vitaly added the comment:
In practice I doubt this matters much as this error string is likely to be
less than one page (depends on pathnames involved) but it is still
technically incorrect as written.
Agreed. Thank you.
--
___
Python tracker
R. David Murray added the comment:
To repeat: there is no change to be made for 3.3. 3.3.0 will go out the door
with the pre-12776 behavior. So any backward compatibility concerns that apply
to 2.7 and 3.2 also apply to 3.3. Thus I suggest we restore the string check,
and consider an
Barry A. Warsaw added the comment:
On Sep 11, 2012, at 07:00 PM, R. David Murray wrote:
To repeat: there is no change to be made for 3.3. 3.3.0 will go out the door
with the pre-12776 behavior. So any backward compatibility concerns that
apply to 2.7 and 3.2 also apply to 3.3. Thus I suggest
New submission from Barry A. Warsaw:
After a fresh clone, configure make fails due to a fairly obvious bug in the
code. Note that if you `hg revert --all`, configure make will succeed,
probably because the timestamps get updated enough to fool make.
@resist[~/projects/python:1043]% hg
Changes by Christian Heimes li...@cheimes.de:
--
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15923
___
___
New submission from Joshua Landau:
http://www.python.org/about/ section Python plays well with others, last
paragraph, link extension modules links to
http://www.python.org/doc/ext/intro.html, a 404 page.
http://www.python.org/doc/ext/ redirects to http://docs.python.org/extending/
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
versions: +Python 2.7 -Python 2.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5766
___
___
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
versions: +Python 3.4 -Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1005895
___
___
Changes by Brian Curtin br...@python.org:
--
nosy: -brian.curtin
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1005895
___
___
Python-bugs-list
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
keywords: +easy
versions: +Python 3.3 -Python 3.1
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8964
___
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti
versions: +Python 3.4 -Python 3.1
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11385
___
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
dependencies: +Remove -mno-cygwin from distutils
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15315
___
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
components: +Interpreter Core -None
nosy: +ncoghlan
stage: - needs patch
versions: +Python 3.3, Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10445
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
status: open - pending
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13990
___
___
1 - 100 of 116 matches
Mail list logo