R. David Murray added the comment:
lexer.instream.readline(): no, we can't just call that without the seek,
because reading the token that started with # may have caused the line to be
consumed already. I've expanded the comment to explain this.
cygwin: I'd add a comment if I
R. David Murray added the comment:
This should not be applied until after the patch for #12009 is applied, at
which point the test will need to be adjusted. #12009 completely refactors the
netrc test suite.
I wonder if the 'hosts' attribute should be deprecated, but let
R. David Murray added the comment:
Improvements are welcome, the whole thing was a QnD hack that has been evolving
over time...maybe there's some way to reuse unittest's assertEqual machinery?
I also wonder if the whole environment-changed thing should be moved into
unittest so it
Changes by R. David Murray :
--
stage: -> needs patch
type: -> feature request
___
Python tracker
<http://bugs.python.org/issue12314>
___
___
Python-bugs-
Changes by R. David Murray :
--
resolution: -> invalid
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
R. David Murray added the comment:
I agree with Petri, so I'm setting this to a doc issue.
--
assignee: -> docs@python
components: +Documentation
nosy: +docs@python, r.david.murray
stage: -> needs patch
type: -> behavior
___
Python
R. David Murray added the comment:
I think it would be good to mention the numeric levels in the text as well (in
parenthesis, perhaps), unless the rest of the logging docs no longer refer to
the numeric levels.
--
nosy: +r.david.murray
___
Python
R. David Murray added the comment:
Sometimes we use the patch keyword for doc issues where someone has suggested a
specific new wording but not generated an actual patch. That seems to be the
case here.
--
nosy: +r.david.murray
___
Python tracker
Changes by R. David Murray :
--
versions: -Python 3.1
___
Python tracker
<http://bugs.python.org/issue11669>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
fileno and stat won't work on a StringIO object. So StringIO would need to be
special cased to call getvalue.
--
nosy: +orsenthil, r.david.murray
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
According to this ticket it hasn't been applied anywhere yet (a message will be
posted here when it is).
--
nosy: +r.david.murray
versions: +Python 3.3 -Python 3.1
___
Python tracker
<http://bugs.py
R. David Murray added the comment:
But if the len information is available, why not return it?
--
___
Python tracker
<http://bugs.python.org/issue12327>
___
___
Changes by R. David Murray :
--
versions: +Python 3.2 -Python 2.6
___
Python tracker
<http://bugs.python.org/issue12327>
___
___
Python-bugs-list mailin
R. David Murray added the comment:
Is it really necessary to copy this file? I haven't looked at the tests, but
I'm wondering if they are using xxmodule.c only because it already existed.
Could the test instead use a much simpler .c file that it creates on the fly?
-
R. David Murray added the comment:
IIUC, collections.Set is, by definition, the full 'set' API. The fact that the
set builtin implements some additional methods is a convenience.
--
nosy: +r.david.murray, rhettinger
___
Python trac
R. David Murray added the comment:
collections.Set (in python 3.3, collections.abc.Set) is the *definition* of the
minimal API. To construct a class that conforms to the minimal definition is
certainly easier than constructing one that supports all of the methods that
the set builtin does
R. David Murray added the comment:
The code you are suggesting patching is trying its best to return a length. If
that code needs to be fixed to not throw an error when confronted with a
StringIO, then it should do its best to return a length. Your original message
on the ticket did not
R. David Murray added the comment:
I normally build using --prefix, often with --prefix=/dev/null, and I can't
recall any version not building zlib. Of course, I don't uncomment the zlib
entry in Modules/Setup.
As far as I can see the zlib entry is unique in Setup in using pre
R. David Murray added the comment:
Sounds like a plan.
--
___
Python tracker
<http://bugs.python.org/issue12314>
___
___
Python-bugs-list mailing list
Unsub
R. David Murray added the comment:
No need to apologize for the bump. The trick is catching the interest of
someone who feels qualified to judge the patch. I've added a couple people to
nosy who worked on difflib recently. If no one speaks up in the next few days,
it might be time to
Changes by R. David Murray :
--
nosy: +pitrou
___
Python tracker
<http://bugs.python.org/issue12340>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
Well, one way might be to set up and maintain an HP/UX buildbot :)
Other than that, just keep bugging us periodically until someone gets around to
doing it.
--
nosy: +r.david.murray
stage: -> commit review
type: -> compile
R. David Murray added the comment:
Judging from the stack trace, it isn't str.format that's failing, it's tk
failing to display it.
--
nosy: +r.david.murray, terry.reedy
title: characters with ord above 65535 fail conversion with str.format for
'{:c}' in ID
R. David Murray added the comment:
It used to be that the svn version info was pulled from embedded constants in
Makefile.pre.in. I agree that depending on having mercurial (and, presumably,
a repo!) at build time is wrong. These constants need to be pre-computed when
a release tarball is
R. David Murray added the comment:
Indeed, I just confirmed that building with a recent version of mercurial
installed from the release 2.7.2 source tarball from python.org, the make
results in:
abort: repository . not found!
abort: repository . not found!
abort: repository . not found
R. David Murray added the comment:
Fixing the missing dirs for email has been on my todo list for a while but
obviously hasn't gotten done yet. I talked with Antoine about setting up a
buildbot that tested an installed version, but I'm not sure when I'll get
around to tryin
R. David Murray added the comment:
Ah. For subversion there was a similar check, and the values were set to blank
if the subversion binary was not found. For hg it looks like we need to add
additional checks on whether or not there is a repo. (We can probably assume
that if there is a
R. David Murray added the comment:
I would not read [str] as implying a list of strings, FWIW. Nor would I assume
a plural option meant a list if the text says "a string". But I'm just a
bystander here and haven't even looked the docs y
R. David Murray added the comment:
Looks good for linux. Do you have a posix reference that confirms this
interpretation?
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue12
R. David Murray added the comment:
Ah. I wouldn't have expected that. For my information, what was the error you
got?
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Ah, bingo. That was what was tickling at my memory but I couldn't remember
what the exact issue was I was recalling. I forget what I was doing where that
mattered, but as I vaguely remember it there is no portable way to find out
what blocksize st_b
R. David Murray added the comment:
Ah, found it: Issue 10016.
--
___
Python tracker
<http://bugs.python.org/issue12350>
___
___
Python-bugs-list mailin
Changes by R. David Murray :
--
versions: +Python 3.2, Python 3.3
___
Python tracker
<http://bugs.python.org/issue12350>
___
___
Python-bugs-list mailin
R. David Murray added the comment:
Actually, when it is a feature request rather than a bug fix we usually use
'accepted'. It doesn't matter a whole lot, though.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.py
Changes by R. David Murray :
--
title: “pydoc str” works but not “pydoc str.translate” -> "pydoc str" works but
not "pydoc str.translate"
___
Python tracker
<http:/
R. David Murray added the comment:
OK, now I'm really confused. I tried to reproduce this by installing from my
checkout into a work dir, and then running regrtest using that installed
python, and discovered that test_email and its data directory got copied by the
installation pr
R. David Murray added the comment:
Victor, that failure was already reported in issue 12333.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue12
R. David Murray added the comment:
That needs to be ported to the other branches, then.
Ezio, on a completely unrelated note, notice what happened to Ralf's reference.
I think the regexes may need to be reordered.
--
nosy: +ezio.me
R. David Murray added the comment:
Bump, this failure is still happening on the ppc tiger buildbot periodically.
--
nosy: +ned.deily
___
Python tracker
<http://bugs.python.org/issue8
R. David Murray added the comment:
Well, regardless of whether or not I understand what's going on, clearly those
directories in the makefile needed updating, so I did it.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Torsten, can you provide a clear, failing unittest for this?
--
___
Python tracker
<http://bugs.python.org/issue4470>
___
___
R. David Murray added the comment:
Thanks for the report and patch. I'm setting this to test needed since the
final patch will need a unit test.
The idiomatic way to do this kind of check is 'if not argstring or
arg_string[0] not in self.fromfile_prefix_chars):'
--
R. David Murray added the comment:
According to your traceback you should be seeing the error in the first line
(the creation of the SMTP_SSL object). If I run that line at the python prompt
of python2.7.1, I get your connection failure. If I run it using 2.7 tip (or
3.3 tip), the
R. David Murray added the comment:
The patch doesn't work on 2.7. The failures are related to #11700, although
that would seem to indicate that something *is* trying to close the file. I
have no idea what.
--
dependencies: +mailbox.py proxy up
R. David Murray added the comment:
Well, if you want to you could investigate further, and try the patch from
#11700 and see what is left to do here after it has been applied. I'll try to
get 11700 in soon, though.
--
___
Python tracker
R. David Murray added the comment:
It means that objects were garbage collected. The refleak test runs the test
multiple times, and ignores the first N runs to allow the object count to
"settle". But sometimes it either doesn't settle, or later runs end up with
objects
R. David Murray added the comment:
I think that wording is as good as we can do for now. Providing a way to
determine the size of st_blocks blocks should be a separate issue (a feature
request). That enhancement can include an update to these docs, but since it
is an enhancement it will
R. David Murray added the comment:
I'm not sure what the point of your example is, Terry. Is it not fixed in
3.2.1?
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/is
Changes by R. David Murray :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
title: mailbox.py proxy updates -> mailbox.py proxy close method cannot be
called multiple times
type: -> behavior
___
R. David Murray added the comment:
Well, it turns out I was totally wrong about the 11700 dependency. I misread
the errors that were produced by the test suite. Even after fixing 11700 they
are still there: the tests are reading from the closed files. So something
changed in the logic
R. David Murray added the comment:
The second paragraph goes on to cover your point, but I agree that it is not as
clear as it should be.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue12
R. David Murray added the comment:
I forgot that I had this issue open. I committed the makefile part of the
patch in issue 12313. I currently have no way to test the windows part, but it
seems a straightforward modification of the existing code, so I'm going to
comm
Changes by R. David Murray :
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> compile error
___
Python tracker
<http://bugs.python
R. David Murray added the comment:
OK, the invariant make_header(decode_header(x)) == x should once again work for
anything returned by __getitem__.
--
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
R. David Murray added the comment:
Heh, I misstated that invariant, it's only true when x is a Header.
--
___
Python tracker
<http://bugs.python.org/is
Changes by R. David Murray :
--
status: -> open
___
Python tracker
<http://bugs.python.org/issue11873>
___
___
Python-bugs-list mailing list
Unsubscri
Changes by R. David Murray :
--
title: Python 2.7.2 source code build (release) depends on mercurial -> Python
source code build (release) depends on mercurial
___
Python tracker
<http://bugs.python.org/issu
Changes by R. David Murray :
--
keywords: -easy, patch
___
Python tracker
<http://bugs.python.org/issue3216>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
I think we can treat this as a bug. However...
What if there is more than one set of Resent- headers? I think that it is not
possible to guarantee we only look at the most recent set, since the RFC
provides no way to identify a "set". Heuris
R. David Murray added the comment:
Furthermore, Éric is the one who is going to commit the patch, and therefore
these tracker issues should be organized to provide him with maximum
productivity. If he thinks this should be closed, then it should be closed.
If you wish to continue to argue
Changes by R. David Murray :
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.o
Changes by R. David Murray :
--
nosy: +vinay.sajip
___
Python tracker
<http://bugs.python.org/issue12361>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
The file association for .py is pythonw, which does exactly as you say,
intentionally, so as to avoid problems with windows when running a GUI
application. My understanding is that this caters to the most common use case
on Windows: double clicking a .py
Changes by R. David Murray :
--
dependencies: -Error in sys.excepthook on windows when redirecting output of
the script
superseder: -> Error in sys.excepthook on windows when redirecting output of
the script
___
Python tracker
&l
R. David Murray added the comment:
Looks fine except for your changes to the parenthesized defaults. Those should
be '0' and 'False' for 2.7 and 3.x, respectively, since that's what they
areally are.
--
___
Python tracker
R. David Murray added the comment:
Note that the older tradition was to *not* mention the contributor in NEWS
(NEWS was just technical notes), but to mention them in the checkin message
(and What's New, for things that make the What's New cut). However, since we
can't edit c
R. David Murray added the comment:
Ah, I should have known better than to rely on a memory instead of checking,
since I don't use Windows much. My apologies.
--
dependencies: +Error in sys.excepthook on windows when redirecting output of
the script
resolution: invalid -> d
R. David Murray added the comment:
Only if Ralf's patch is applied to all branches. Otherwise the make step
reports "abort: repository . not found!", which most users will ignore but a
few will report here. It looks from Ralf's quoted changeset like it was only
app
R. David Murray added the comment:
As I've said before, I would vote to not have .rej and .orig in .hgignore. You
can always add them to your personal .hgignore, but I know of no way to tell
mercurial "I *don't* want to ignore these files that are in the repo
.hgignore".
R. David Murray added the comment:
I have no idea what you mean by "for distrabution" in this context. The
.hgignore in the checkout causes hg to ignore the files/patterns listed in it
when any hg commands are run against that checkout. And no I can't remove it
in my &quo
R. David Murray added the comment:
This was fixed in issue 4471 by Antoine when he added some tests that call
login. He fixed it by changing _quote to work with strings. Per the
discussion here I'm not sure this is the best fix, but until someone reports a
bug with it it we may as wel
R. David Murray added the comment:
Well, since there's no central server for hg, it can't track based on the
server. Perhaps I confused you by saying that the .hgignore file was "in the
repo". (It is "in the repo" in the sense that any tracked file is in th
Changes by R. David Murray :
--
nosy: +pitrou
___
Python tracker
<http://bugs.python.org/issue12291>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
I reopened the issue because I don't know the answer to your question but
suspect that it is an issue. So it needs to be checked. And the regex should
be cleaned up regardless, for clarity's sake.
--
R. David Murray added the comment:
Isn't 'definition time' vs 'execution time' an artificial distinction? I'm
surprised that the main module is compiled differently than a regular module.
Is that an artifact of the CPython implementation or a part of the langu
Changes by R. David Murray :
--
nosy: +amaury.forgeotdarc, belopolsky -theller
title: find_library should search LD_LIBRARY_PATH on linux -> ctypes
find_library should search LD_LIBRARY_PATH on linux
___
Python tracker
<http://bugs.pyth
R. David Murray added the comment:
Section 3.6.6 says:
Resent fields are strictly informational. They MUST NOT be used in
the normal processing of replies or other such automatic actions on
messages.
Further, since there is no specified order for the headers within a block,
there
R. David Murray added the comment:
Note that that RFC language is clearly directed at automatic processing on
*receipt*, not during sending. The RFC doesn't address automatic processing
during sending, it leaves that the to the SMT
Changes by R. David Murray :
--
nosy: +michael.foord
___
Python tracker
<http://bugs.python.org/issue12376>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
2.6 is in security-fix-only mode.
By inspection the 2.7 and 3.x code have the same issue (though the 3.x code is
very different, there still appears to be a lack of error recovery logic.
Joe, do you have any interest in writing a unit test for this? I
R. David Murray added the comment:
What's the use case? I'm inclined to reject this as not needed.
--
nosy: +r.david.murray
type: behavior -> feature request
versions: +Python 3.3 -Python 2.7, Python 3.1
___
Python tracker
<http:/
R. David Murray added the comment:
As Raymond said in another issue, someone should fix this once and for all by
factoring this check out into a reusable method and making sure it is used
everywhere.
--
keywords: +easy
nosy: +r.david.murray
versions: +Python 3.2, Python 3.3 -Python
R. David Murray added the comment:
Heh. You are assuming we know how it "should" be used :) It's pretty new yet,
so I don't think we've figured that out. But at at least how it works should
be mentioned in the devguide.
---
R. David Murray added the comment:
The test appears to be failing on the buildbots:
http://www.python.org/dev/buildbot/all/builders/x86%20Ubuntu%20Shared%203.2/builds/370
--
nosy: +r.david.murray
status: closed -> open
___
Python tracker
&l
R. David Murray added the comment:
A bytearray is for working with mutable data. We don't support using it in all
places that the non-mutable data types can be used. You can code your example
like this:
barr.center(len(barr) * 4, bytes([barr[-1]]))
I realize that isn't pa
R. David Murray added the comment:
Hmm. I wonder why we have two copies of this method.
Well, given that we do, the best fix for the docs would be to say "for use in
the bytes or bytearray translate method" in both this docstring and the
bytes.maketrans docstring.
-
R. David Murray added the comment:
All right, let's get some other opinions from people who have actually worked
with the bytearray and bytes code (and Terry because he cares about APIs).
--
nosy: +haypo, ncoghlan, pitrou, rhettinger, terry.reedy
status: closed -&
R. David Murray added the comment:
Indeed, the lib/plat- directories should continue to work just fine using
linux3, correct? Or using linux, if we change sys.platform.
(Note: just because we don't import them in the test suite doesn't mean that
user code in the field isn't
R. David Murray added the comment:
Thanks for the patches. I didn't use them, but they were helpful references.
This is in a grey area between a bug and a feature request. The fact is,
though, that for the most part the email module currently doesn't make extra
effort to detect d
R. David Murray added the comment:
Thanks.
--
nosy: +r.david.murray
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by R. David Murray :
--
nosy: +kbk, terry.reedy
___
Python tracker
<http://bugs.python.org/issue12387>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
I have removed the last mentions of template from the docs and docstrings, and
added a comment that it is a private variable despite its name. I do not see a
reason to break backward compatibility just to have a leading underscore in the
name. If there is
R. David Murray added the comment:
Here is a patch that I think describes the algorithm correctly, based on the
comments in the module, with a clarifying parenthetical to cover the
non-obvious consequence of that algorithm.
--
versions: +Python 3.3 -Python 3.1
Added file: http
R. David Murray added the comment:
One question is whether this is a bug fix or a feature request.
Other than that, I'd like to see the test classes collapsed into a single test
class, considering that each one only has a single test in it. Probably
ProxyAuthTests should be refactor
R. David Murray added the comment:
Oh, the bad error message is definitely a bug. The question is whether we can
also add md5-sess support while fixing it. Sounds like Senthil thinks no, in
which case this issue needs to be split into two parts
R. David Murray added the comment:
3.3.
IMO this is in the grey area between feature and bug fix. I think it is
possible to argue that it can be treated as a bug fix, but I think we need
opinions from other developers if we want to try to go that route.
The reason I think it can be argued
R. David Murray added the comment:
Your unit test isn't consistent with the other unit tests in that set, which
makes me suspicious that it isn't testing what we need to test. Also, there
are unit tests for this case further up in the test file
(TestEmptyAndSpaceContainingArgu
R. David Murray added the comment:
Hmm. I don't know that it is really necessary to cater to the particular
failure mode, I was more interested in seeing a unit test that checked the
correct behavior: that a syntax error is raised (by capturing the output using
the tools in script_h
Changes by R. David Murray :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.o
R. David Murray added the comment:
For the most part the patch looks good to me, too. My one concern is the
encoding. tokenize detects the encoding...is it possible for the doctest
fragment to be detected to be some encoding other than utf-8?
--
nosy: +benjamin.peterson
2201 - 2300 of 12936 matches
Mail list logo