R. David Murray added the comment:
Thank you, that information confirms that this is indeed a duplicate.
--
___
Python tracker
<http://bugs.python.org/issue20
R. David Murray added the comment:
Yeah, let's just close it. Nobody is likely to care enough to try to fix it
properly.
--
resolution: -> out of date
stage: needs patch -> committed/rejected
status: languishing -> closed
___
Python
R. David Murray added the comment:
That is seriously broken on Apple's part. But I guess we have no choice but to
emulate their bug.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
I read that first paragraph as the controlling one, and that says that errno
will be zero if the problem was that the entry could not be found, and non-zero
if some other error occurred.
Raising an error would be a backward incompatible change, so it could
Changes by R. David Murray :
--
nosy: +lemburg
___
Python tracker
<http://bugs.python.org/issue21031>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by R. David Murray :
--
assignee: -> ronaldoussoren
components: +Macintosh
nosy: +ronaldoussoren
___
Python tracker
<http://bugs.python.org/issu
R. David Murray added the comment:
The patch needs .. versionadded:: 3.5 tags for the two new methods, and adding
it to the skeleton whatsnew would be a good idea. The committer can do these,
but if you feel like updating the patch that would be great
R. David Murray added the comment:
I made some review comments. There is one bug with your patch (you dropped
some argument descriptions in one place.)
--
resolution: fixed ->
stage: committed/rejected -> needs patch
status: closed -
R. David Murray added the comment:
Can you think of any circumstance in which getpwnam would be able to read the
file to look for the user, and yet errno is non-zero?
--
___
Python tracker
<http://bugs.python.org/issue4
R. David Murray added the comment:
Ah, I see. Obviously I didn't read it as carefully as I thought I had ;(.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
It's really too bad cert validation fails on that ftp site. It would be nice
to show best practices in that example. We really need that python test server
Benjamin was talking about.
--
___
Python tr
R. David Murray added the comment:
Well, I guess you could sum it up as -0. I personally don't feel the need for
a change, but if the chronology problem is solved and it isn't *harder* to make
the needed NEWS changes, then I'm not
R. David Murray added the comment:
I don't think that changes the picture. If the error code is non-zero, that
means that one of the access methods failed, which means getpwnam can't report
reliably whether or not that user exists according to the system con
Changes by R. David Murray :
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
R. David Murray added the comment:
Thanks, Kammie. I removed the extra whitespace from your fix and simplified
the tests a bit.
--
resolution: -> fixed
stage: test needed -> committed/rejected
status: open -> closed
___
Python track
R. David Murray added the comment:
There also needs to be a way to specify a different NEWS entry from the commit
message first line.
--
___
Python tracker
<http://bugs.python.org/issue18
Changes by R. David Murray :
--
nosy: +r.david.murray
stage: -> patch review
title: tempfile.TemporaryFile() shouldn't have a name attribute -> tarfile does
not handle file __name__ being an int
versions: +Python 3.5 -Python 3.3
___
Pyt
Changes by R. David Murray :
--
title: tarfile does not handle file __name__ being an int -> tarfile does not
handle file .name being an int
___
Python tracker
<http://bugs.python.org/issu
Changes by R. David Murray :
--
nosy: +pitrou, r.david.murray
___
Python tracker
<http://bugs.python.org/issue21041>
___
___
Python-bugs-list mailing list
Unsub
R. David Murray added the comment:
I think this is a doc bug. That object shouldn't be called a sequence, since
it isn't one.
--
___
Python tracker
<http://bugs.python.o
R. David Murray added the comment:
Which is exactly what I mean by saying it is not a sequence. It is
'sequence-like'. Kind of like email Messages are dict-like: they share many
methods and behaviors, but the exact behaviors and semantics are
R. David Murray added the comment:
See the documentation link in msg214670. This isn't a characteristic of
TemporaryFile, it's a characteristic of the Python IO system. So you'd have to
argue that the documented behavior of the io
Changes by R. David Murray :
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue16047>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by R. David Murray :
--
stage: -> committed/rejected
status: open -> closed
title: Can't solve special women's problems? Let us do it. -> spam
___
Python tracker
<http://bugs
R. David Murray added the comment:
What makes you think this is a different issue? It sounds like a duplicate to
me.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue21
R. David Murray added the comment:
Ah, in case I wasn't clear: what unittest failures that you are seeing are not
covered by issue 20939? In order to diagnose this issue, we'll need to know
that :)
--
___
Python tracker
<http://bu
R. David Murray added the comment:
Well, I just looked at one of those tests at random, and it is using an
http://www.python.org url, so it would be covered by 20939. Can you re-run
your tests after applying the patches from that issue
R. David Murray added the comment:
It hung before the changes as well. On the other hand, before the removal of
the timeout value, it at least displayed an error *before* hanging, so I think
absent a proper fix the timeout at least should be restored.
--
nosy: +r.david.murray
R. David Murray added the comment:
I agree that the implementation does not match the documentation in this case.
Especially the part about "the format string used to create this Struct
object". I don't see what having a flag would buy you: it doesn't help you in
writing
R. David Murray added the comment:
Network tests are only run if the 'network' resource is enabled. The devguide
does not suggest setting network, but make test does do so. (I personally
always use -uall when running the tests).
--
R. David Murray added the comment:
Also, the "see below" sentence is missing.
--
___
Python tracker
<http://bugs.python.org/issue20344>
___
___
Python-b
R. David Murray added the comment:
That facility already mostly exists. The bug is that the code in question
doesn't use it.
>>> m['Content-Disposition'].content_disposition
'attachment'
>>> m['Content-Disposition'].params
{'filename
R. David Murray added the comment:
Here's patch.
--
keywords: +patch
Added file: http://bugs.python.org/file34641/is_attachment.patch
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
I was going to open new issue for adding 'value', but looking at the parsing
code I see why I didn't add one. The way the new parser works it really wants
to know the actual structure of the value, it doesn't have a good way to treat
R. David Murray added the comment:
Well, that's why is_attachment exists. I wouldn't be averse to adding
get_content_disposition if nobody objects, though.
The attributes are on the headers because the data really is attributes of the
parsed headers, but the more useful user
R. David Murray added the comment:
OK. If you would be willing to open a feature request for that, that would be
great.
--
___
Python tracker
<http://bugs.python.org/issue21
R. David Murray added the comment:
cygwin is not officially supported. We do accept patches that improve cygwin
support when they are narrowly targted and well motivated.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.
R. David Murray added the comment:
Hmm. This might be related to issue 20731?
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue21
R. David Murray added the comment:
Also, to me 'fill' implies something is being filled, not that something is
being created.
--
___
Python tracker
<http://bugs.python.o
R. David Murray added the comment:
He wasn't, sbt (with help) was. Benjamin may have checked it in or something.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
For whatever it is worth, it looks like rfc 5892 marks U+2026 as disallowed.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue21
R. David Murray added the comment:
Note that any issues here should also be considered for zipfile and shutil.
(Well, shutil can just use the other two once the security is available.) See
issue 20907.
--
nosy: +r.david.murray
___
Python tracker
New submission from R. David Murray:
It looks like something changed on the python.org server-side that is causing
test_urllib2net tests test_urlwithfrag (and, on 3.4,
test_redirect_url_withfrag) to fail because the fragment doesn't appear in the
response url. Which is very puzzling
R. David Murray added the comment:
The URLs in question are:
http://docs.python.org/3.4/glossary.html#term-global-interpreter-lock
http://docs.python.org/2/glossary.html#glossary
Which are now redirecting to https, which would cause a failure regardless of
the fragment issue...perhaps the
R. David Murray added the comment:
I believe it is a python invariant that a == b implies hash(a) == hash(b). I
don't see why the two signatures should be equal. I'm not even sure why the
bound argument shows up in the signature of the partial. That surprises me.
-
R. David Murray added the comment:
There were a bunch of changes to pydoc in 3.4, so I'm not surprised that it
doesn't use DumbWriter any more (possibly as part of the formatter pending
deprecation). I think I was answering why it wasn't deprecated as part of the
html
R. David Murray added the comment:
OK, I didn't even realize that was possible with partial. Now I understand
Yuri's original point. His example is wrong:
>>> def foo(a, b):
...print(a, b)
>>> x2 = partial(foo, 'x')
>>> str(inspect.si
R. David Murray added the comment:
By "didn't know that was possible", I mean binding a positional argument as a
keyword argument in the partial. If nobody else thought that was possible,
maybe can just fix it :)
--
___
Python
R. David Murray added the comment:
We'll have to wait for Nick to chime in, but I'll make a couple of comments.
First, I think this is a bug in partial, so I think we need to decide what, if
anything, to do about that first, before we decide if signature needs to
compensate for
R. David Murray added the comment:
Oh, the error message comes from deep in the guts of python, yes. I'm saying
that the fact that partial lets you write partial(foo, a='bar') when a is a
positional argument is a bug. Even if other people agree with me (and they may
not, &qu
R. David Murray added the comment:
A google for quadruple got me wikipedia as the first hit, with this as the
match text:
"Quadruple may refer to: A 4-tuple, ..."
So, it's tech-speak. The alternative would be to say either "a 4-tuple" or "a
tuple of
R. David Murray added the comment:
Why? Just let the context convey the information. It's not like we are
building a UI here, this is for the programmer.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Although I will grant you that I have to guess at what the bad value
substitution error message is trying to tell me, so that error message could
use some improvement.
--
___
Python tracker
<h
R. David Murray added the comment:
But since the two partial instances are (conceptually, I don't care how they
are implemented) two separate functions, then reasoning by analogy from two
identical functions not comparing equal, I would expect two partial instances
to not compare
R. David Murray added the comment:
Of course, as soon as I hit send, I had second thoughts :). I guess a partial
is a binding, not a function.
--
___
Python tracker
<http://bugs.python.org/issue21
Changes by R. David Murray :
--
nosy: +dstufft
___
Python tracker
<http://bugs.python.org/issue21144>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
Yes, I remember previous discussions of the corporate agreement from Google, so
I'm sure it exists.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
I think it would be helpful for you to email the python-list mailing list.
Hopefully the people there can help you refine your problem report into
something we can tackle. The installer works fine for most people, so the
first thing that we need in order
R. David Murray added the comment:
Sorry, I meant to delete that word 'minimal', since you have told us what you
currently know, but I hit send before I did :)
--
___
Python tracker
<http://bugs.python.o
R. David Murray added the comment:
Antoine, being polite never hurts. Terry is a valuable member of the community
and sure, he sometimes makes mistakes (or trusts the docs too much?). So do
the the rest of us.
--
nosy: +r.david.murray
___
Python
R. David Murray added the comment:
Another problem with that error message is that it seems confusing as to where
the bad value *reference* came from, given that rawval doesn't seem to include
any interpolation point.
--
nosy: +r.david.m
R. David Murray added the comment:
I've been thinking about this more, and I think I will revise my opinion. I
haven't been able to think of a place where knowing that the key is missing in
self._sections would in fact be helpful to a programmer using the module. So
I'm OK
Changes by R. David Murray :
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue21161>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by R. David Murray :
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue21145>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
I added some review comments. Since this is a new feature, the patch also
needs a 'versionchanged' that indicates that ipv6 support was added.
--
stage: needs patch -> patch review
versions: +Python 3.
R. David Murray added the comment:
This issue should have gone back to being a release blocker after the alpha
release to fix the tab-as-indent issue, but obviously that didn't happen (I
forgot about it myself). Please open a new issue requesting a fix for this bug
(that tab doesn'
R. David Murray added the comment:
Can you suggest how to improve the docs?
--
___
Python tracker
<http://bugs.python.org/issue20010>
___
___
Python-bugs-list m
Changes by R. David Murray :
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue21169>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
Testing the documentation examples is a long term goal, which people
occasionally contribute to. I think there is an open issue about using the
Sphinx doctest support, that for a long time was blocked by the documentation
tool chain not using python3. Some
R. David Murray added the comment:
I think this is a specific case of a more general need to improve 'wraps' that
was discussed on python-dev not too long ago.
--
nosy: +r.david.murray
versions: +Python 3.5 -Python 3.4
___
Python trac
R. David Murray added the comment:
Yeah, this is a bit non-obvious, but it is a specific instance of the general
way that argparse handles types. So as far as I can see there really isn't
anything to do here.
--
nosy: +r.david.murray
resolution: -> invalid
stage: ->
R. David Murray added the comment:
This looks good, except that if we are not going to delete that test routine
(and we aren't because we didn't deprecate it :) I think we should instead
replace the usage of Netrc with the ne
R. David Murray added the comment:
Since we don't want the prompting for the password to fail, what we do in the
patch is use the replace error handler so that you get as much as could be
encoded of the prompt. (Note: this approach was reviewed by both Toshio and
Marc Andre.)
Thanks fo
R. David Murray added the comment:
Ok, I'll reopen the issue to do that.
--
status: closed -> open
___
Python tracker
<http://bugs.python.org/issue21169>
___
R. David Murray added the comment:
I think 'please review' was directed at anyone, and yes, using the review link
is one way to do a review, but when there isn't enough line-by-line commenting
to make the code review tool worth using you can just post on the issue. (And
when
R. David Murray added the comment:
I decided to tweak the language slightly, Kushal. If this isn't what you were
looking for, Martin, let me know.
--
status: open -> closed
___
Python tracker
<http://bugs.python.org
R. David Murray added the comment:
Since you are saying that it is "sometimes" necessary to use a different
package, perhaps we should be saying that in the devguide? And providing the
possible names.
--
nosy: +barry, r.david.murray
R. David Murray added the comment:
I've made some review comments.
--
___
Python tracker
<http://bugs.python.org/issue15104>
___
___
Python-bugs-list m
Changes by R. David Murray :
--
stage: needs patch -> patch review
versions: +Python 3.5 -Python 3.2, Python 3.3
___
Python tracker
<http://bugs.python.org/issu
Changes by R. David Murray :
--
Removed message: http://bugs.python.org/msg174146
___
Python tracker
<http://bugs.python.org/issue15916>
___
___
Python-bugs-list m
Changes by R. David Murray :
--
Removed message: http://bugs.python.org/msg174145
___
Python tracker
<http://bugs.python.org/issue15916>
___
___
Python-bugs-list m
R. David Murray added the comment:
Well, with the new website the url now returns a 404, so I guess we can close
this. If someone wants to see it restored as a historical document, that would
be an issue for the web site bug tracker. But, since other essays exist, it
looks like it was a
R. David Murray added the comment:
OK, so the devguide currently has
sudo apt-get build-dep python3
which did something on Glenn's machine, but did not enable him to build the
optional packages. So the question is, what should we put in the devguide as
the correct build-dep to in
R. David Murray added the comment:
The suggestion was to make this a footnote, not a note. Also, it should
probably say that the stmt is executed inside a function, meaning that instead
of being a syntax error it changes the return value of the internal timeit
function.
I understand
R. David Murray added the comment:
This looks good, however we also need a documentation change indicating the new
behavior, including a '.. versionchanged:: 3.5' tag, and an entry in
whatsnew/3.5 in the 'other changes' section.
--
__
R. David Murray added the comment:
Thanks, Sam. I did not apply this to 2.7 because I'm not sure if the
__main__.py is supported there. Can someone check?
--
___
Python tracker
<http://bugs.python.org/is
Changes by R. David Murray :
--
Removed message: http://bugs.python.org/msg216177
___
Python tracker
<http://bugs.python.org/issue15104>
___
___
Python-bugs-list m
R. David Murray added the comment:
Thanks, Sam. I did not apply this to 2.7 because I'm not sure if the
__main__.py is supported there. Can someone check?
--
type: enhancement -> behavior
versions: -Python 3.4, Python 3.5
___
Python
R. David Murray added the comment:
Adding a parameter is an enhancement. Probably not a bad thing to have anyway,
but it can only go in 3.5. Since this isn't something that causes problems for
production code, that seems fine. (The alternative would be to say have a
private class var
R. David Murray added the comment:
OK, if you think it is worthwhile in the text, then sure. But yeah, not as a
..note. And yes, I think we should keep backporting relevant doc patches.
Especially since Google results still land one on the 2.7 docs
R. David Murray added the comment:
I'm guessing they've got a local fix in the release candidate and won't change
even their test code there, so I'd guess the beta. But the jython folks would
really be the ones to ask. Perhaps they will respond here (th
R. David Murray added the comment:
Just as a point of information, when making a patch like this it is best to
change the smallest number of lines possible, without worrying about line
wrapping. This allows us to see just what was changed. The committer can then
reflow the paragraph
R. David Murray added the comment:
Actually I take it back, the patch as a patch also has a couple issues: the
line lengths are not in fact less than 80 characters in any case, and there is
trailing whitespace on several lines.
Could you redo it with just the minimum lines changed and no
R. David Murray added the comment:
Thanks. Could you also change 'Invalid syntax' to 'Unsupported syntax', per
the last bit of the discussion between Terry and I?
--
___
Python tracker
<http://bug
R. David Murray added the comment:
Did you hand test it? Also, you could open a new issue to add tests for the
ftplib cli, and probably improve and document it? It was designed as a test,
but some people may be using it and it might even be actually useful
R. David Murray added the comment:
The patch looks good, but 'splitdoc' needs to remain a valid name for the
function in the pydoc namespace. You could just add 'splitdoc =
inspect.splitdoc' after the import statements. (The reason it needs to remain
valid is for ba
R. David Murray added the comment:
Thanks, Kushal. Sorry it took so long to get this committed :)
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
type: -> behavior
versions: +Python 3.5 -Python 2.7, Python 3.2
R. David Murray added the comment:
If you look at the source code for 2.7, it is clear that patch has been applied.
--
___
Python tracker
<http://bugs.python.org/issue21
R. David Murray added the comment:
To clarify, the commit in that issue is in 2.7. So if there is something else
that isn't in 2.7, it is a different issue.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Thanks, Glenn.
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
4901 - 5000 of 10554 matches
Mail list logo
