Guido van Rossum added the comment:
Please apply the fix and make sure it gets cherry-picked.
I agree on dynamic test recovery but don't find it a high priority.
--
___
Python tracker
<http://bugs.python.org/is
Guido van Rossum added the comment:
I'd prefer not to add a bad hack to support such a broken platform. If it
doesn't have semaphores, not much Python code will run.
Maybe the dummy_threading module can be used instead?
--
___
Python trac
Guido van Rossum added the comment:
For now, can we just add to the asyncio docs that timeouts shouldn't exceed one
day? Then we can fix it later without breaking expectations.
--
___
Python tracker
<http://bugs.python.org/is
Guido van Rossum added the comment:
Thanks, LGTM. Please commit and add to issue 20648.
--
___
Python tracker
<http://bugs.python.org/issue20668>
___
___
Pytho
Guido van Rossum added the comment:
LGTM. Please commit in both repos and add to the list of cherrypicks for
Larry.
On Tue, Feb 18, 2014 at 1:40 AM, STINNER Victor wrote:
>
> STINNER Victor added the comment:
>
> If I add a sleep of 1 second before stdin.write(), the test always
Guido van Rossum added the comment:
This one needs to be CP'ed together with Yuri's UNIX sockets one.
changeset: 89257:f81106f36656
tag: tip
user: Guido van Rossum
date:Tue Feb 18 10:24:30 2014 -0800
summary: asyncio: Make tests pass
Guido van Rossum added the comment:
For the record I'm against it, but I don't have time to explain until after 3.4
has been released.
--
nosy: +gvanrossum
___
Python tracker
<http://bugs.python.o
Guido van Rossum added the comment:
It should not be changed after the release either.
On Tue, Feb 18, 2014 at 11:57 AM, Serhiy Storchaka
wrote:
>
> Serhiy Storchaka added the comment:
>
> This is sad. Because after a release, change it will be
Guido van Rossum added the comment:
Sorry, forget everything I said. (That includes the LGTM, sadly -- as I wrote
earlier I don't know this code very well.)
--
___
Python tracker
<http://bugs.python.org/is
Guido van Rossum added the comment:
TBH I don't think typos or pep8 cleanups ought to be CP'ed...
On Tuesday, February 18, 2014, Larry Hastings
wrote:
>
> Larry Hastings added the comment:
>
> 2e8a142dbccc just missed today's cherry-picking sessi
Guido van Rossum added the comment:
Actually I am trying to quell the tide. :-). But nobody sent me a review
request for those...
On Tuesday, February 18, 2014, Larry Hastings
wrote:
>
> Larry Hastings added the comment:
>
> Perhaps, but if they request cherry-picking for rev
Guido van Rossum added the comment:
Thanks for checking!
On Tuesday, February 18, 2014, Larry Hastings
wrote:
>
> Larry Hastings added the comment:
>
> For what it's worth,
>
> Lib/asyncio/*
> Lib/test/test_asyncio/*
> Doc/library/async*
>
> are all exact
Guido van Rossum added the comment:
That one is not critical. It can go in sfter 3.4 is released.
On Wednesday, February 19, 2014, Vajrasky Kok
wrote:
>
> Vajrasky Kok added the comment:
>
> > It is slowing down. I think we now only have some documentation updates
> left.
&
Guido van Rossum added the comment:
Please wait until after 3.4 release. Not critical.
On Wednesday, February 19, 2014, Yury Selivanov
wrote:
>
> Changes by Yury Selivanov >:
>
>
> --
> nosy: +yselivanov
>
> ___
Guido van Rossum added the comment:
I suspect it's just a race in the test. This happened to me recently too when I
ran the tests on my Windows 7 laptop, and upon the second try it passed, so I
just figured it was a race. After all the test does use run_briefly(), which
might as we
Guido van Rossum added the comment:
Agreed. I suppose all tests currently using run_briefly() should be modified to
use run_until() with a lambda and a timeout. The timeout must be (a) large
enough to never fail on a buildbot, yet (b) small enough that when a test is
actually failing we don
Guido van Rossum added the comment:
>
> It seems to me that the StreamReader() limit parameter is for buffer size
> while the io.BytesIO.readline() "n" parameter is for maximum number of
> lines to be retreived, I guess.
>
You sound confused. The parameter for io.Byte
New submission from Guido van Rossum:
The asyncio package uses the logging module. We should remind users that they
should always configure their logs to go to a file on the local filesystem --
using any kind of network logging will block the event loop.
--
assignee: haypo
messages
Guido van Rossum added the comment:
If you really need network logging you should be able to configure a handler
that puts things in a queue whose other end is serviced by an asyncio task.
There should be no need to mess with the type of the logger object. Anyway, in
3.4 it is what it is
Guido van Rossum added the comment:
Yes, your whole world will be turned upside-down. However, we have to do it one
module at a time -- the approach will be different for each case.
--
resolution: -> wont fix
status: open -> closed
___
Guido van Rossum added the comment:
I'm guessing you weren't ready for learning about metaclasses if you
didn't get the fact that 'name' was the loop control variable in the two
different loops. The example is only 11 lines long!
Still, I'm fine with the two su
Guido van Rossum added the comment:
I am guessing the reason to keep os.popen() (albeit now reimplemented
using subprocess) is that it is a convenient wrapper for a common use case
and also familiar. I see no problem with this. (Indeed the big problem
was with the proliferation of popenN
Guido van Rossum added the comment:
It strikes me as simply a documentation bug. Maybe whoever wrote the
docs just assumed it would work that way. I expect that changing it to
insist on valid input would break some use cases. If you want a
validating API, perhaps a new API could be added that
Guido van Rossum added the comment:
Raymond, I know you see this differently, but personally I don't see a
problem with 2.7 getting additional backports of 3.x features, as long
as they are backwards compatible. While personally I wouldn't spend
effort on this particular backport, I d
New submission from Guido van Rossum :
I am using an edge case of multipart/mixed and find that the
multipart/mix parser in the email package is broken. See attached
example. Similar code using cgi.FieldStorage (!) works fine.
The problem happens through the following combination of factors
Guido van Rossum added the comment:
Older Python versions too.
--
versions: +Python 2.5
___
Python tracker
<http://bugs.python.org/issue6681>
___
___
Python-bug
Guido van Rossum added the comment:
A slot in every type object for this purpose seems wasteful; the large
majority of types won't have a free list. (Remember, each user-defined
class allocates a full type structure on the heap.)
--
assignee: gvanr
Changes by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<http://bugs.python.org/issue2516>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
Some (supportive) discussion on python-dev at
http://mail.python.org/pipermail/python-dev/2009-August/091324.html .
--
nosy: +gvanrossum
___
Python tracker
<http://bugs.python.org/issue6
Changes by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<http://bugs.python.org/issue6749>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
Please don't introduce this without a PEP.
--
nosy: +gvanrossum
___
Python tracker
<http://bugs.python.org/issue5135>
___
___
Guido van Rossum added the comment:
Wow. You (rohdef) really do sound like you are a time capsule from the
eighties. Tabs would save keystrokes and bandwidth, and are not
confusing? The keystrokes argument is wrong for most editors; the
bandwidth argument doesn't matter due to disk
Guido van Rossum added the comment:
Raymond, please refrain from emotional terms like "bug factory".
I have nothing to say about whether string.capwords() should be removed,
but I want to note that it does a split on whitespace and then rejoins
using a single space, so that strin
Guido van Rossum added the comment:
A doc fix sounds like a great idea.
--
___
Python tracker
<http://bugs.python.org/issue7008>
___
___
Python-bugs-list mailin
Changes by Guido van Rossum :
--
assignee: gvanrossum ->
___
Python tracker
<http://bugs.python.org/issue7008>
___
___
Python-bugs-list mailing list
Unsubscri
Changes by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<http://bugs.python.org/issue7012>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Guido van Rossum :
--
assignee: gvanrossum ->
nosy: -gvanrossum
___
Python tracker
<http://bugs.python.org/issue7008>
___
___
Python-bugs-list mai
Guido van Rossum added the comment:
Before we start pointing fingers or throwing mud, was this bug fix which
apparently breaking setuptools introduced knowing that this would break
setuptools users?
I'm surprised that nobody at all tried to use the release candidate with
any of the l
Guido van Rossum added the comment:
Thanks for the quick action Tarek!!!
--
___
Python tracker
<http://bugs.python.org/issue7064>
___
___
Python-bugs-list mailin
Guido van Rossum added the comment:
So there's no bug here right? It's even documented.
--
nosy: +gvanrossum
resolution: -> rejected
status: open -> closed
___
Python tracker
<http://bugs.p
Guido van Rossum added the comment:
> delay all removals until all iterators are done
+1
--
nosy: +gvanrossum
___
Python tracker
<http://bugs.python.org/iss
New submission from Guido van Rossum :
Following my own (Twitter) advice I downloaded the 2.6.4rc1 release
candidate and installed it on my x86 iMac running OS X 10.5.8, and tried
to run a Google App Engine app with it. Google App Engine prefers Python
2.5, but so far it has always worked
Guido van Rossum added the comment:
FWIW, I've ruled out App Engine SDK 1.2.6 as the source of the regression;
on a MacBook Pro with the same OS 10.5.8 I get a similar traceback with
SDK 1.2.6.
--
___
Python tracker
<http://bugs.py
Guido van Rossum added the comment:
The full traceback had probably 1000 lines (or whatever the limit is); the
piece between two calls to logging.warn() is repeated over and over.
--
___
Python tracker
<http://bugs.python.org/issue7
Guido van Rossum added the comment:
Yes, App Engine uses a PEP 302 style import hook to implement the sandbox.
Could it be that there's a new module dependency introduced by 2.6.3-4?
--
___
Python tracker
<http://bugs.python.org/i
Guido van Rossum added the comment:
The dependency of logging on multiprocessing feels backwards. But it's
not actually a new regression, it seems that was there in 2.6.2.
--
___
Python tracker
<http://bugs.python.org/i
Guido van Rossum added the comment:
The addition of multiprocessing support to logging seems to have happened
in 2.6.2; bad timing for a feature. :-(
--
title: 2.6.4rc1 regression -> logging depends on multiprocessing
___
Python tracker
&l
Guido van Rossum added the comment:
Confirmed, the stack limit error is gone now. Thanks!!
(There's another error, the import of _scproxy from urllib, but that's
easily added to the App Engine SDK's whitelist. I am still concerned
about the amount of change in the 2.6 branch,
Guido van Rossum added the comment:
(I don't know why the tracker reopened the issue when I added a comment.
Anyway, is the fix going into 2.6.4 or will it have to wait for 2.6.5?)
--
___
Python tracker
<http://bugs.python.org/i
Changes by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<http://bugs.python.org/issue1235>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Guido van Rossum :
Removed file: http://bugs.python.org/file8578/unnamed
___
Python tracker
<http://bugs.python.org/issue1294>
___
___
Python-bugs-list mailin
Guido van Rossum added the comment:
I don't think this is a good idea.
--
resolution: -> rejected
status: open -> closed
___
Python tracker
<http://bugs.python
Changes by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<http://bugs.python.org/issue1286>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
I think the code originally wasn't meant to support this feature (honor
embedded newlines when replace_whitespace=False). I'm thinking that we
could add it though. Maybe Mark is interested in getting this into 2.7
and 3.2? I imagine it n
Guido van Rossum added the comment:
I don't think it would hurt to put it back, would it? I think the "remove
this in 3.1" note was made when we expected 3.1 to be happening 1.5 years
after 3.0 rather than a few months.
--
no
Guido van Rossum added the comment:
Smells like a feature to me.
"Bug" == "coding error"
"Feature" == "change in (documented or intended) behavior"
I'm fine with this going into 2.7 / 3.2.
--
___
Pyth
Guido van Rossum added the comment:
Can someone provide a code sample to make this argument more
understandable for those of us who don't compare points by angular order
for a living... :-)
I'm not sure what the 2to3 example (I presume you mean msg59937) shows
except that convers
Changes by Guido van Rossum :
Removed file: http://bugs.python.org/file15463/unnamed
___
Python tracker
<http://bugs.python.org/issue1771>
___
___
Python-bugs-list mailin
Changes by Guido van Rossum :
Removed file: http://bugs.python.org/file15473/unnamed
___
Python tracker
<http://bugs.python.org/issue1771>
___
___
Python-bugs-list mailin
Guido van Rossum added the comment:
Python's sort implementation is carefully written to only use the "<"
comparison, ever. So a cmp really isn't the most natural way to specify
a comparison. (This should really be documented somewhere -- I know know
it because T
Guido van Rossum added the comment:
I don't see a bug here. The value returned by imp.get_magic() matches the
first bytes of a bytecode file. That's all that matters.
Of course, we could remove the global variable and just use the defined
constant; but that's just an opti
Guido van Rossum added the comment:
What maintenance issues are you anticipating?
___
Python tracker
<http://bugs.python.org/issue1580>
___
___
Python-bugs-list mailin
Guido van Rossum added the comment:
On Thu, Feb 26, 2009 at 1:01 PM, Tim Peters wrote:
> Is it worth it? To whom ;-) ? It was discussed several times before on
> various Python mailing lists, and nobody was willing to sign up for the
> considerable effort required (both to update G
Guido van Rossum added the comment:
IIUC the problem is that a read() syscall on the pty after the other end
has been closed raises an error instead of reading 0 bytes? Isn't that
a bug in the pty implementation? For lots of devices (e.g. sockets,
pipes) a short non-empty read just means
Guido van Rossum added the comment:
Agreed. Multiple threads trying to read interactive input from a
keyboard sounds like a bad idea anyway.
___
Python tracker
<http://bugs.python.org/issue1
Guido van Rossum added the comment:
That may be how it works, but how do you expect to deal with it?
___
Python tracker
<http://bugs.python.org/issue5380>
___
___
Pytho
New submission from Guido van Rossum :
[Found by a Googler who prefers to remain anonymous]
This might be easier to trigger on a 64-bit:
PyObject *PyUnicode_DecodeUTF7Stateful(...)
{
...
Py_ssize_t startinpos;
...
while (s < e) {
...
utf7Error:
outpos
Guido van Rossum added the comment:
> Well, as I suggested, in FileIO.read(): when receiving errno=5 on a
> read() call and if S_IFIFO() returns true, clear errno and return an
> empty string.
> The question is whether a genuine EIO error ("low level IO error")
Guido van Rossum added the comment:
Hm, I'm not sure why Adam brought up readline. The behavior is
certainly guaranteed (I put that guarantee in myself long ago :-) and it
should be fixed. I have no opinion about the proposed patch, since I
cannot test this and have long lost suffi
Guido van Rossum added the comment:
I tried that, and it was more subtle than that in corner cases.
Another argument against it is that on Windows the system input routine
doesn't correctly round unless 17 digits of precision are given. One of
Tim Peters's responses should have e
Guido van Rossum added the comment:
It looks like it was fixed in 2.6 by adding an assignment to startinpos
to this block:
else if (SPECIAL(ch,0,0)) {
startinpos = s-starts; /* <-- This was added */
errmsg = "unexpected special character";
Guido van Rossum added the comment:
I changed my mind on the cross-platform requirement.
___
Python tracker
<http://bugs.python.org/issue1580>
___
___
Python-bugs-list m
Guido van Rossum added the comment:
The idea is to be able (whether you see a use case or not) to use
different tasks lists simultaneously. Messing with globals is the worst
possible API for that. All you need is to add a tasks=None argument to
the loop() signature, rename the global tasks
Guido van Rossum added the comment:
You could solve this with a "reserved" keyword argument _tasks.
Or you could have two different factory methods, call_later_with_tasks()
and call_later().
___
Python tracker
<http://bugs.python.
Guido van Rossum added the comment:
Sounds good to me.
___
Python tracker
<http://bugs.python.org/issue1580>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
What makes you think it is not called python.exe? Maybe you're confused
by the Finder's auto-hiding of externsions?
--
nosy: +gvanrossum
resolution: -> invalid
status: open -> closed
___
Pytho
Guido van Rossum added the comment:
Well, this one is technically a security fix, though I have no idea how
it could be exploited unless you offer your users a facility to execute
arbitrary Python code.
___
Python tracker
<http://bugs.python.org/issue5
Guido van Rossum added the comment:
That sentence however does not refer to the name of the installed Python
binary, but the binary as it is built in the source tree (or a build
directory). And there we chose to call it "python.exe" because we
couldn't call it "python"
Guido van Rossum added the comment:
OK, submitted.
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue5389>
___
__
Guido van Rossum added the comment:
This is specific to the Python tracker, which Rietveld tries to avoid.
You could maintain this as a locally modified version, but a better
approach would be to make just enough changes to upload.py itself so
that you can write the rest of this script as a
Guido van Rossum added the comment:
Please go ahead and finish this. I'm glad this is going in!
--
message_count: 16.0 -> 17.0
nosy: +gvanrossum
nosy_count: 6.0 -> 7.0
___
Python tracker
<http://bugs.python
Guido van Rossum added the comment:
Thanks for contributing. However since you reformatted the whole file
it's kind of hard to see what you changed, which makes review laborious.
Could you try to upload a unified diff relative to the PEP source code
in Subve
Guido van Rossum added the comment:
Thanks! Reviewing now...
--
assignee: georg.brandl -> gvanrossum
___
Python tracker
<http://bugs.python.org/iss
Guido van Rossum added the comment:
Submitted as revision 70312, with some additional changes. I suppose it
could use some more work (e.g. adding the penultimate proposal to the
History section) but I'll leave that to others. (Lie, if you feel like
writing that history section, just add
Guido van Rossum added the comment:
Not sure if that's worth it -- doesn't sound like the people who would
need that feature would mind much using explicit numbering. Let's try KISS.
--
___
Python tracker
<http://bugs.py
Guido van Rossum added the comment:
(I meant that as a reply to the {:{fmt}} example, but it applies to your
later post too. :-)
--
___
Python tracker
<http://bugs.python.org/issue5
Guido van Rossum added the comment:
> About '{:{fmt}}' and other wacky combinations, like '{.__doc__}':
>
> It's much easier and cleaner in the code to allow these cases than it
> would be to disallow them. And I rather like the '{:{fmt}}' exampl
Guido van Rossum added the comment:
Please don't do this. We need stable APIs. Trying to switch the entire
community to use CapWord APIs for something as commonly used as datetime
sounds like wasting a lot of cycles with no reason except the mythical
"PEP 8 conformance". As I sa
Guido van Rossum added the comment:
Can anyone think of a *reason* to put a yield inside a generator
expression? ISTM we could just forbid this syntactically. It seems
insane and hard to read so if someone has a reason they should write it
out using an explicit for-statement.
--
nosy
Guido van Rossum added the comment:
Fine!
--
resolution: -> wont fix
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue5577>
___
___
Guido van Rossum added the comment:
(Almost) nobody uses these any more, so let's close as won't fix. The
SGI modules are removed from 3.0. The few people who still have Irix
could probably care less about the buffer overflows, but they *might*
care about the modules (otherwise the
Guido van Rossum added the comment:
The naming pattern (assertListEqual, assertDictEqual, etc.) is pretty
easy to pick up so I don't think the addition of several new methods is
a big deal. Making assertEqual() do the right thing all the time is
hard if there are subclasses that might ove
Guido van Rossum added the comment:
I would love to use the time machine to kill the 'fail' variants as well
as 'assert_'. However they are in use and we don't want to break
people's tests. Breaking tests is painful. And there are a lots of
tests. So we'
Guido van Rossum added the comment:
At the sprint, Andrew Svetlov, Martin von Loewis and I looked into this
a bit, and discovered that Andrew's Vista copy uses a Russian locale for
the filesystem encoding (despite using English as the language). In
this locale, a-umlaut cannot be repres
New submission from Guido van Rossum :
There's a path in imp.find_module that mixes encodings. The module name
is encoded to char* using UTF-8 by the 's' format passed to
PyArg_ParseTuple(). But the path name is converted (in the loop over
the path in find_module()) to
Guido van Rossum added the comment:
Since Tim Peters is the absolute authority on concurrency in Python,
let's just close this as Rejected rather than letting it sit here forever.
--
nosy: +gvanrossum
resolution: -> rejected
status: open -
Guido van Rossum added the comment:
Looking at this, I think this change is fine. The _Event class itself
holds the condition that it's checking for, and the is_set() method
doesn't acquire the lock, so there's no reason to prefer
e.wait()
if e.is_set():
GOT_IT()
Guido van Rossum added the comment:
Looks good to me. Amaury, can you make sure this lands in 2.6, 2.7, 3.0
and 3.1?
--
assignee: -> amaury.forgeotdarc
nosy: +gvanrossum
resolution: -> accepted
___
Python tracker
<http://bugs.p
Guido van Rossum added the comment:
> Antoine Pitrou added the comment:
> Wouldn't it be simpler to make assertRaises return the exception and let
> the calling code match it as it feels like?
Hm, that sounds awfully familiar. I can't recall if there was ever a
good re
Guido van Rossum added the comment:
Thanks Andrew! Committed to 3.0.2 as 70756.
Should be merged into 3.1, but should *not* be backported to 2.x.
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/
4401 - 4500 of 5563 matches
Mail list logo