R. David Murray added the comment:
Which is what Barry's library does.
But rather than just deprecating install, I think we should fix the module so
that it supports this directly. That could be as simple as adding a dynamic
translations class.
What does library code that wants to pr
R. David Murray added the comment:
Oh, by the way there's a bit more of the test infrastructure in python3
compared to python2.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Well, the first thing to do would be to write a test that reproduces the
problem. There is test infrastructure in Lib/test/test_imaplib.py, but I will
admit that writing the server side of imaplib tests is not a walk in the park.
You are welcome to take a
R. David Murray added the comment:
I'll also note that if you start from the traceback and look at the code
involved in the exception (keeping in mind that since the exception is caught,
in python2 you lose the original cause, which is therefore in the function
called in the try block),
R. David Murray added the comment:
The documentation is correct, it is the docstrings that need to be changed.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
I'm -1 on this. It doesn't make sense to me to conflate two functions like
that. If you need that functionality often in your application, just write a
small helper function.
--
nosy: +r.david.murray
R. David Murray added the comment:
I find it interesting that I just came across a case where I want this
functionality, involving a generator derived from a possibly-empty sql table.
I'm using Stefan's functional expressi
R. David Murray added the comment:
As a side note, the various warnings about the execution environment being
modified do not appear when tests are run under unittest because those warnings
are generated by regrtest itself.
--
___
Python tracker
R. David Murray added the comment:
support.requires is smart. If the caller's module is __main__, it treats the
resources as set. So you don't have to do anything special in your __main__ to
make the tests runnable directly.
The tests are also run if unittest is called. I don&
R. David Murray added the comment:
Oh, actually I do know why that is. It is because requires is called from
test_main in test_urllibnet, which unittest bypasses. So if you are calling it
in particular tests, then I presume that will be an issue for the unittest case
R. David Murray added the comment:
I don't understand why his keepalive requires the extra spaces...and I'm not
100% sure that accepting them will fix the problem, though in theory it should.
I still lean toward making the spaces fix in imaplib, based on the postel
principle. I
R. David Murray added the comment:
Since this issue has been closed, it would be great if you'd open a new issue
just for the doc change.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Note that it is a known issue that tk (not python's bindings, tk itself) alters
the locale.
--
___
Python tracker
<http://bugs.python.org/is
Changes by R. David Murray :
--
components: +email
nosy: +barry, r.david.murray
versions: +Python 3.3, Python 3.4
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
Yes, the way it is now is normal written english, regardless of what any older
official stylebooks may say :)
--
nosy: +r.david.murray
resolution: -> invalid
stage: -> committed/rejected
status: open -> closed
type: -&
R. David Murray added the comment:
It looks like this bug belongs on the jython tracker at bugs.jython.org. (By
the way, it isn't clear if the problem is in the jython code or the test; the
stdlib Popen works as expected on Windows.)
--
nosy: +r.david.murray
resolution: ->
R. David Murray added the comment:
Sorry, but this is not the kind of change that we make to the codebase. It has
too many unintended consequences. (Large portions of the stdlib pre-date PEP8,
and thus are not PEP8 compliant.) We fix some thing when we modify the code
involved, but other
R. David Murray added the comment:
By the way, this kind of question is more suited to the python-list mailing
list (where you will in the general case get a faster answer anyway :)
--
nosy: +r.david.murray
stage: -> committed/rejected
status: open ->
R. David Murray added the comment:
I am in favor of adding meaningful attributes to stdlib exceptions. I've
always considered the lack of such an API a wart in Python, though an
understandable one (since exceptions started out as simple strings). But yeah,
while I hate to say it, th
R. David Murray added the comment:
I think we tend to forget that that page exists.
Patches welcome.
--
keywords: +easy
nosy: +r.david.murray
stage: -> needs patch
type: -> behavior
___
Python tracker
<http://bugs.python.org/i
R. David Murray added the comment:
This kind of thing is why python3 exists. Presumably some bit of the
elementree code is successfully converting non-ascii into unicode, and then
when that is mixed with the result it is returning, you end up with unicode.
But that is just a guess; you
Changes by R. David Murray :
--
nosy: +eli.bendersky
___
Python tracker
<http://bugs.python.org/issue18268>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
Here is a patch that implements the ones I'm interested in (-o for any option,
and -f as a shortcut for -o FAIL_FAST). It's a good question whether doctest
should support other unittest options, but doing so is more complicated than
supporting
R. David Murray added the comment:
Oh, a documentation update is still needed here, so I guess I'm not really
ready for review yet :(
--
stage: patch review -> needs patch
___
Python tracker
<http://bugs.python.org
Changes by R. David Murray :
Removed file: http://bugs.python.org/file30654/doctest_cli.patch
___
Python tracker
<http://bugs.python.org/issue11390>
___
___
Python-bug
R. David Murray added the comment:
Patch updated with documentation.
--
stage: needs patch -> patch review
Added file: http://bugs.python.org/file30655/doctest_cli.patch
___
Python tracker
<http://bugs.python.org/issu
R. David Murray added the comment:
The python3 email package's handling of 8bit definitely has quirks. (So did
the python2 email package's, but they were different quirks. :)
You can't correctly handle 8bit unless you use message_from_bytes and take the
input from a byte s
R. David Murray added the comment:
If all you are changing is headers (and you con't change the CTE), then when
you use BytesGenerator to re-serialize the message, it is supposed to preserve
the existing CTE/payload. (Whether or not you call get_payload, regardless of
arguments, doe
R. David Murray added the comment:
I will probably commit this tomorrow if there are no objections.
--
___
Python tracker
<http://bugs.python.org/issue11
R. David Murray added the comment:
It would be nice to add a test (guarded by the network resource) that checks
that the generated url in license is not a 404. Would you be interested in
tackling that, Berker?
--
___
Python tracker
<h
R. David Murray added the comment:
Patch updated per Barry's review comments.
--
Added file: http://bugs.python.org/file30667/doctest_cli.patch
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
In python we have a saying that we follow most of the time: if you don't know,
refuse the temptation to guess. So currently this is all working as designed:
you have to know the encoding of the file you are trying to read as unicode.
Adding a &
R. David Murray added the comment:
I just did a clean doc build on 2.7 (on linux) on 2.7 head without error.
Maybe you need to do a 'make clean' and rebuild? Our Makefile ought to have a
target that just throws away the build dir, instead of throwing away the
checked out softwa
R. David Murray added the comment:
Oh, right, the advocacy howto was deleted recently. So it is almost certainly
your stale pickles in the build dir that are the issue.
--
resolution: -> invalid
stage: -> committed/rejected
status: open ->
R. David Murray added the comment:
Automating it could be a sphinx enhancement request. Or maybe a bug report.
--
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
I would prefer that Haypo spend his time contributing code to Python. If
someone else wants to summarize the arguments in the thread for this issue,
that would be great. Absent that, the link to the discussion is sufficient for
the curious.
In any case
R. David Murray added the comment:
Committed. Thanks for the review, Barry.
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
R. David Murray added the comment:
Thanks Berker. I edited your patch to (a) use 'Otherwise' instead of 'By
default', which is a change from the original docstring that I think makes it
clearer, (b) add words about what the local_hostname is used for (the HELO/EHLO
R. David Murray added the comment:
Is the "slower" test on 2.6? io would definitely be slower there, since it is
pure python. 2.7 has the C accelerated version.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.o
R. David Murray added the comment:
It is implemented as a doctest because everything else in that test module is
implemented as a doctest. And clearly this reveals a bug of some sort.
Perhaps the dummy trick needs to be done by doctest itself? (I will need to
read your post over more
R. David Murray added the comment:
Fixed for this test. It would probably be worthwhile to improve
script_helpers, I'll open a new issue for that.
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://b
New submission from R. David Murray:
A look at a random selection of tests that use script_helper indicates that
using universal_newlines=True would either simplify or make no difference to
the usage of the script_helper assert_python functions in the majority of the
tests that use it. Even
New submission from R. David Murray:
See issue 11390 for a discussion, but briefly: there is a bug in the readline
library that causes it, on some systems, to emit a control sequence on stdout
when readline is first initialized and TERM is set to xterm. The bug can be
avoided by setting TERM
Changes by R. David Murray :
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue18293>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
I've had a pep8-ification of this patch sitting on my disk for a while.
Uploading it here so it doesn't get lost. It feels like there is a lot of
redundancy now in the docs. But, it also seems to make sense to provide a wsgi
version of thi
R. David Murray added the comment:
It is implemented as a classmethod, but the "equivalent" code doesn't need to
be part of the class all. I'm not sure what should be done here (say
@staticmethod? Leave the decorator off?). We should probably see what Raymond
thinks
R. David Murray added the comment:
I would (naively) not expect either of these to be serializable. They are not
base data types, which is all the json module handles unless you add extra
handlers yourself.
'crash' means interpreter crash (segfault), by the way.
-
R. David Murray added the comment:
I've checked in the encode version of the method. I'm going to pass on doing
the other inlines, given that the improvement isn't that large. I will,
however, keep the issue in mind as I make other changes to the code, and there
wi
R. David Murray added the comment:
Fixed, thanks.
--
nosy: +r.david.murray
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> behavior
___
Python tracker
<http://bugs.python
R. David Murray added the comment:
A quick look at the VCS history indicates bufsize has been in there for a long
time. The sphinx docs are wrong as well. This is correctly documented in
python3, apparently as part of the conversion from [] notation to keyword
notation in d143eb624cf5
Changes by R. David Murray :
--
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue14360>
___
___
Python-bugs-list mailing list
Unsubscri
R. David Murray added the comment:
This should now be fixed. Calling MIMEApplication with a binary payload and
passing it encode_quopri as the encoder will now actually work.
--
resolution: -> fixed
stage: needs patch -> committed/rejected
versions: +Python 3.4 -Pyth
R. David Murray added the comment:
It's great that this patch was provided. Xuanji, can you submit a contributor
agreement, please?
The patch is missing an update to the documentation.
(Really the patch should have been in a separate issue, as requested, since
this one is about impr
R. David Murray added the comment:
Yes, I think opening a new issue at this point might be a good idea. The
reason is that there are a changes either in place or pending in other issues
that involve the parse_know_args code, so a new patch is probably required
regardless.
I wish I had time
R. David Murray added the comment:
Based on a quick glance at the code, the problem isn't that you are passing
fileobj, it is that fileobj has been opened for writing, not reading, and you
are attempting to read from it. extrastart (and other attibutes) are only set
if mode starts wi
R. David Murray added the comment:
On the other hand, it seems reasonable to be able to read if you've done a
seek(0), so a more complicated fix may also be appropriate.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
Julian, could you please submit a contributor agreement?
(http://www.python.org/psf/contrib)
--
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
For the record, encode_quopri was fixed in #14360.
--
___
Python tracker
<http://bugs.python.org/issue4768>
___
___
Python-bug
New submission from R. David Murray:
In order to maintain model consistency without exposing the need for
'surrogateescape' to library users, it should be possible to pass binary data
to set_payload and have it do the correct conversion to the expected storage
format for the model.
R. David Murray added the comment:
At 83 it is a judgement call whether to leave it long or wrap it. I wouldn't
compress it. (Wrapping is done by putting a '\' at the end of the line you
want to wrap, if you don'
R. David Murray added the comment:
It means they are keyword-only arguments. This could be mentioned in the text,
with the term 'keyword-only arguments' linked to an appropriate glossary entry
(which appears to need to be added).
--
keywords: +easy
nosy: +r.david.mu
R. David Murray added the comment:
Committed, with slight modifications to the tests. Thanks Vajrasky.
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by R. David Murray :
--
nosy: +ncoghlan
___
Python tracker
<http://bugs.python.org/issue18331>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
"Intentional" is not perhaps the exactly right term, but it is working as
expected. There was a thread about this on python-dev ("PEP 3115 compliant
dynamic type creation" and "adding types.build_class for 3.3"), that ultimatel
Changes by R. David Murray :
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue18236>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
There are two issues that I could find related to these characters, one of them
still open: #18236 and #7643. The latter contains a fairly complete
discussion of the underlying issue, but on a quick read through it is not clear
to me if the linebreak issue
R. David Murray added the comment:
Yes, I am against it. See python-dev for reason (it would no longer be a
'distclean').
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.o
R. David Murray added the comment:
Unfortunately make distclean is now failing on the buildbots.
Which reminds us that there is another place that that target, including the
'find' at issue, is used.
Presumably there is an issue with tests not cleaning up after themselves that
o
R. David Murray added the comment:
Hmm. I may be pointing the finger at the wrong thing here...looking at more
logs I'm now not sure what is up with the buildbots.
--
___
Python tracker
<http://bugs.python.org/is
Changes by R. David Murray :
--
resolution: -> rejected
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
R. David Murray added the comment:
OK, this looks like a false alarm. I still don't understand those @ failures
or the input error on the makefile, but there doesn't seem to be a problem with
this patch.
--
stage: needs patch -> committed/rejected
status: o
R. David Murray added the comment:
pydoc doesn't do anything special other than import the module. Well, there's
some special code in linecache for getting docstrings from C modules, I think,
so that could be something to check.
Does Ubuntu have local patches to numpy?
-
R. David Murray added the comment:
Exactly which installer are you trying to run and where did you download it
from?
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
Why wouldn't you just enable bytes warning and fix all the code to work
correctly?
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/is
Changes by R. David Murray :
--
stage: test needed -> patch review
status: languishing -> open
___
Python tracker
<http://bugs.python.org/issue4079>
___
___
R. David Murray added the comment:
All the buildbots are volunteer hosted and run, so we need someone to volunteer
to run a Windows 8 buildbot. That said, I'm not sure if we are formally
supporting Windows 8 yet or not. Actually, given the lack of a buildbot, I
suppose the answer is
R. David Murray added the comment:
I'm not sure that this belongs in core, but I'll certainly defer to Georg on
that :). If it does go in, the help string should explain what 'devhelp' does,
since it isn't obvious from the name (for a non-GNOME user...I had to goog
R. David Murray added the comment:
I understand your concern, but the API is that 'start' is a directory. The
function does not interrogate the file system, and should not do so. It is
purely a path computation, and as such the current behavior is correct.
--
nosy: +r.da
R. David Murray added the comment:
I believe there are open (and some closed) issues in this tracker related to
mingw support, though many of them are for support for compiling extensions
using mingw, which is actually a harder problem. They will probably provide
clues if nothing else. What
R. David Murray added the comment:
That's a good point.
--
assignee: -> docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python
resolution: rejected ->
stage: committed/rejected -> needs patch
status: closed -> open
title: os.path.relpath gives inc
R. David Murray added the comment:
@Martin: you are probably correct, though I think there was at least one closed
issue that did deal with mingw/msys build. As I recall it wasn't coherent
enough to produce useful patches, but may have some
R. David Murray added the comment:
Windows doesn't support posix signals, sending one just kills the process, as
you have observed. See the documentation for the os.kill function.
--
nosy: +r.david.murray
resolution: -> invalid
stage: -> committed/rejected
status: ope
R. David Murray added the comment:
Please see issue 14425 and then let us know if this is still valid. It's been
a while since I looked at the code and I no longer remember the details, but I
seemed confident in my conclusion at the time... :)
--
nosy: +r.david.m
R. David Murray added the comment:
See also issue 4079, if you are concerned about timeouts in urllib.
--
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
Perhaps we could add something like "Also works, though possibly inefficiently,
on any iterable whose elements support addition". The biggest part of the
sphinx docs for this are about what to use instead, and that doesn't really
seem ap
Changes by R. David Murray :
--
versions: +Python 3.4 -Python 3.1, Python 3.2
___
Python tracker
<http://bugs.python.org/issue18424>
___
___
Python-bugs-list m
R. David Murray added the comment:
OK, I reviewed the issue enough to remember: If socket.setdefaulttimeout is
never called, then the default timeout is None (no timeout). Since
socket.setdefaulttimeout is deprecated, the global default should always be
None. Therefore, if you want to use
Changes by R. David Murray :
--
nosy: +facundobatista
___
Python tracker
<http://bugs.python.org/issue18417>
___
___
Python-bugs-list mailing list
Unsubscribe:
R. David Murray added the comment:
OK, so your vote is to leave the doc string alone (except for the PEP8
changes), right?
--
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
Regardless of the resolution here, the use of getdefaultlocale could be
reported as a bug on the sphinx tacker.
--
nosy: +r.david.murray
___
Python tracker
<http://bugs.python.org/issue18
R. David Murray added the comment:
Thanks, Andrew.
--
nosy: +r.david.murray
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
versions: +Python 3.3
___
Python tracker
<http://bugs.python.or
R. David Murray added the comment:
Thanks, but the patch is incorrect. The model consistently stores its data as
surrogateescaped strings, and this assumption is baked in to other parts of the
code. So the correct fix is to do the surrogateescape encoding at the time the
payload is set.
It
R. David Murray added the comment:
If you want to work on it that would be great. Note that one of the things
that is needed is a bunch more tests of setting various *kinds* of binary
payload, including ones containing non-ascii data, and making sure the right
thing happens when the payload
R. David Murray added the comment:
Ok, pep8 changes committed.
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> behavior
___
Python tracker
<http://bugs.python
R. David Murray added the comment:
Now we need a test :)
--
___
Python tracker
<http://bugs.python.org/issue4079>
___
___
Python-bugs-list mailing list
Unsub
R. David Murray added the comment:
Thanks, Adam. I'm afraid it may be a while before Michael (or anyone else)
gets a chance to review it and think about the issue some more.
--
___
Python tracker
<http://bugs.python.org/is
R. David Murray added the comment:
I played around with this for a bit, but I couldn't come up with any test
improvements, or any way to test the bug that is being fixed. So I just
committed it as is. Thanks, Serhiy. And thanks Vajrasky for giving it a try
and figuring out some o
R. David Murray added the comment:
What information there is is here: http://wiki.python.org/moin/BuildBot.
Antoine Pitrou is the current buildbot "master" and will issue you a password
once you are set up, if you decide to do this. You can also talk to us on the
#python-dev i
R. David Murray added the comment:
Thanks, Dmi.
--
nosy: +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:
This is actually two separate bugs, both a bit embarrassing. The first one
(that I just fixed) is that when parsing an encoded word I was only checking
for decimal digits after an '=' (instead of the correct hex digits) when trying
to do robust de
3701 - 3800 of 10554 matches
Mail list logo