Change by A.M. Kuchling :
--
pull_requests: +5798
___
Python tracker
<https://bugs.python.org/issue28626>
___
___
Python-bugs-list mailing list
Unsubscribe:
A.M. Kuchling added the comment:
The plan for updating things looks good, and I think the ordering of
tasks is good -- rearranging the APIs is a pretty simple first step,
and it matches other chapters in the Library Reference, which list
more important modules first and more specialized
A.M. Kuchling added the comment:
New changeset ced350b1952857a9b9c68ec7e2786358bcb61050 by Andrew Kuchling in
branch 'master':
bpo-28626: rearrange discussion of output formatting to encourage f-strings
(#6036)
https://github.com/python/cpyt
Change by A.M. Kuchling :
--
keywords: +patch
pull_requests: +7921
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issu
Change by A.M. Kuchling :
--
assignee: docs@python -> akuchling
nosy: +akuchling
___
Python tracker
<https://bugs.python.org/issue20906>
___
___
Python-
A.M. Kuchling added the comment:
Corrected in the unicode-howto-update branch being developed for issue #20906.
--
assignee: docs@python -> akuchling
nosy: +akuchling
___
Python tracker
<https://bugs.python.org/issu
A.M. Kuchling added the comment:
It looks like the problem on OpenIndiana with #2489 was fixed. As of May 25th,
the 3.x build on OpenIndiana is a green.
http://buildbot.python.org/all/builders/x86%20OpenIndiana%203.x/builds/5962
So, 2) is a misunderstanding and 1) was carried out in #2489
New submission from A.M. Kuchling:
pty.py contains some code that tries to do 'import sgi' and then does something
SGI-specific. sgimodule.c was dropped in 3.0alpha1, so this code is now
useless.
The attached patch removes it; I'm posting the code review largely in case
s
A.M. Kuchling added the comment:
I can't make sense of unget_wch. I used this test program:
import curses
def main(stdscr):
stdscr.addstr(0,0, "Key")
stdscr.refresh()
curses.ungetch(0x0149)
while True:
ch = stdscr.get_wch()
stdscr.adds
Changes by A.M. Kuchling :
--
resolution: -> fixed
stage: commit review -> committed/rejected
status: open -> closed
versions: -Python 2.7, Python 3.3
___
Python tracker
<http://bugs.python.org/
A.M. Kuchling added the comment:
This patch seems obviously correct to me, so it should be applied.
It looks like there's already a test in test_uuid.py that exercises
_netbios_getnode() if the netbios module is importable, so there's no test to
add, though maybe we need a buildbo
A.M. Kuchling added the comment:
Matt's patch looks good to me. It removes two module-level dicts, but they're
marked as internal, so that's OK. There's already a test case that exercises
html.escape(), so I don't think any additional tests are needed.
--
Changes by A.M. Kuchling :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue14015>
___
___
Python-bugs-list mailing list
Unsubscribe:
A.M. Kuchling added the comment:
Continuing my tour of the howtos, here's a patch making many of the changes
discussed here and on issue13997. Changes made:
* state that python3 source encoding is UTF-8, and give examples
* mention surrogateescape in the 'tips and tricks
A.M. Kuchling added the comment:
Updated version of my patch, which adds two more todo items and handles Ezio's
review comments:
* Switch from Greek examples to French, and remove non-Latin-1 characters.
* Change language for bytes.decode to "but supports a few more possible
A.M. Kuchling added the comment:
Here's a proposed patch that touches the Sphinx documentation and a docstring
in codecs.py. The text is slightly revised from my current revisions to the
Unicode howto.
help(open) says "See the documentation for codecs.register for a list of the
A.M. Kuchling added the comment:
Confirmed. I think this is a general typographic issue in the PDF
documentation. If I cut-and-paste the datetime.isoformat(sep='T') line from
the PDF, the single quotes are pasted as U+2019 RIGHT SINGLE QUOTE.
Looking at the library.tex file buil
A.M. Kuchling added the comment:
Here's an updated version of the patch. It at least compiles and runs on
MacOS, but I don't have a Solaris installation to check whether it still fixes
the problem on Solaris 9.
Note that, according to
http://en.wikipedia.org/wiki/Solaris_(operat
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue18179>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue18155>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue11352>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue18113>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
stage: needs patch -> committed/rejected
___
Python tracker
<http://bugs.python.org/issue18113>
___
___
Python-bugs-list mai
A.M. Kuchling added the comment:
Thanks for the bug report! I've committed a fix to the 2.7 and 3.3 branches
that retrieves the existing userptr and Py_DECREFs it if it's not null. This
seems to fix the leak.
--
assignee: -> akuchling
resolution: -> fixed
status
A.M. Kuchling added the comment:
serhiy.storchaka: good point! I wonder if, for strict correctness, we should
only incref obj (the new object) if set_panel_userptr() returns OK and not an
error code. I've attached a new version of the patch that does this check, and
also adds a test.
A.M. Kuchling added the comment:
As far as I can tell, there are no other outstanding suggestions for howto
updates, so I'll now close this item. Feel free to re-open or file a new item
if there are further improvements that can be made.
--
resolution: -> fixed
stage: commi
A.M. Kuchling added the comment:
Thanks for your report! I've corrected the error on the 3.3 and 3.4 branches.
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/is
A.M. Kuchling added the comment:
Thanks for your bug report and patch, but I agree that we can't change this.
The repeat() function really does take 'object' as the keyword argument:
>>> from itertools import *
>>> list(repeat(times=3, object='abc'
A.M. Kuchling added the comment:
This bug is entirely too cryptic. What exactly is the problem you're trying to
fix?
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/is
A.M. Kuchling added the comment:
Mike Hoy: a later patch (http://bugs.python.org/issue14626) changed the API in
the os module so that the wrappers for *at() functions went away; fchmod() now
takes several different forms of argument and calls the correct function under
the hood
A.M. Kuchling added the comment:
I didn't see any other constants that aren't close to the function they're used
with. Can this issue be closed, or are there other changes to make?
--
___
Python tracker
<http://bugs.pyt
A.M. Kuchling added the comment:
I've looked through the matches. "I/O" and the -I command-line switch are
false positives. Many references in the FAQ ("How do I do X?"), but those
don't need to be fixed.
I think personalized references are most probl
A.M. Kuchling added the comment:
I believe the most recent 2 commits fix the segfault problem, so I'll now close
this again. Please re-open if there are further issues with the bugfix.
--
resolution: -> fixed
stage: commit review -> committed/rejected
status: ope
A.M. Kuchling added the comment:
Closing this issue after a week. Mike Hoy: thanks for your patch.
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
A.M. Kuchling added the comment:
Bumping version to 3.4. I'll send a note to python-dev about this issue.
--
versions: +Python 3.4 -Python 3.3
___
Python tracker
<http://bugs.python.org/i
New submission from A.M. Kuchling:
I read through the 3.3 regex howto and have made various edits in the attached
patch.
* describe how \w is different when used in bytes and Unicode patterns.
* describe re.ASCII flag to change that behaviour.
* remove a personal reference ('I gene
A.M. Kuchling added the comment:
Slightly revised version that modifies the discussion of when to pre-compile a
regex and when to not bother. I don't think this is a very important issue, so
I don't think it needs a long discussion.
--
Added file: http://bugs.python.org
Changes by A.M. Kuchling :
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue18562>
___
___
Python-bugs-list
Changes by A.M. Kuchling :
--
stage: patch review -> committed/rejected
___
Python tracker
<http://bugs.python.org/issue18562>
___
___
Python-bugs-list mai
A.M. Kuchling added the comment:
Seydou: thanks for the patch! I've applied it to 3.4.
--
nosy: +akuchling
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.pyth
A.M. Kuchling added the comment:
Here's a patch implementing traceback.clear_tb_frames(). (Feel free to
bikeshed about the name.)
One more substantial question: the top frame of the traceback is possibly still
running. Currently the code skips it by doing an initial 'tb =
A.M. Kuchling added the comment:
Revised version of the patch: catches RuntimeError instead of skipping the
first frame; adds versionadded tag; adds entry to NEWS and whatsnew files.
--
Added file: http://bugs.python.org/file31692/clear-tb-frames-2.txt
A.M. Kuchling added the comment:
I'm happy to change the function name, though I'll note that the traceback
module does have print_tb(), format_tb() and extract_tb().
I'm OK with both of Victor's suggestions but personally slightly prefer
traceback.clear_frames(tb).
R
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue14944>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
nosy: +akuchling
___
Python tracker
<http://bugs.python.org/issue14945>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
nosy: -akuchling
___
Python tracker
<http://bugs.python.org/issue15802>
___
___
Python-bugs-list mailing list
Unsubscribe:
A.M. Kuchling added the comment:
+1 to reducing the number of notes, and to a security HOWTO. (Christian: if
you need writing help, please let me know; I'd be happy to help.)
--
nosy: +akuchling
___
Python tracker
<http://bugs.py
Changes by A.M. Kuchling :
--
nosy: -akuchling
___
Python tracker
<http://bugs.python.org/issue20906>
___
___
Python-bugs-list mailing list
Unsubscribe:
A.M. Kuchling added the comment:
The bug got so complicated, with so many variations and failure modes, that I
just lost track of what to do next.
Let's reboot. Problem #1 is that the single-file mailboxes subvert their own
fcntl locking by writing to a temp. file and renaming; Da
A.M. Kuchling added the comment:
Here's a patch for some of Julian's suggested modules. I went through Julian's
list and included code links for the ones that a) weren't packages or only
written in C, b) only had one Python file (which excluded os.path =
posixpath.py/n
A.M. Kuchling added the comment:
Here's a patch fixing the first return in get_matching_blocks() and updating
the docs.
I didn't change get_matching_blocks() to return a list again, assuming that we
didn't want to do that. (Raymond doesn't say to do so, at leas
A.M. Kuchling added the comment:
Thanks for your patch! I took it and added some more text describing what junk
is, and clarifying that junk affects what's matched but doesn't cause any
differences to be ignored.
--
nosy: +akuchling
stage: needs patch -> patch review
Add
A.M. Kuchling added the comment:
Terry: yes, the code for _decimal.c doesn't look especially helpful. (_csv.c
is a bit more straightforward, but still isn't highly commented.) The problem
is that decimal.py does have long docstrings & discussions, so it's certai
Changes by A.M. Kuchling :
--
resolution: -> fixed
stage: commit review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by A.M. Kuchling :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
A.M. Kuchling added the comment:
Derek: thanks for your patch! However, did you run the test suite for the
shutil module to verify that your change is correct?
(The developer guide discusses running tests at
http://docs.python.org/devguide/runtests.html )
--
nosy: +akuchling
A.M. Kuchling added the comment:
Yes, tests are only run after a change is committed and pushed into
Mercurial; this is done by BuildBot https://www.python.org/dev/buildbot/ .
So it's a good idea to run tests before submitting a patch or committing a
change. No matter how trivial a c
Changes by A.M. Kuchling :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
A.M. Kuchling added the comment:
Are we likely to actually apply this change to the 3.1 and 3.2 branches, given
that even the later 3.3 branch is now in security-fix mode? If we're not going
to change 3.1 or 3.2, this issue can just be c
A.M. Kuchling added the comment:
Are we going to apply a fix for this to 2.7?
--
___
Python tracker
<http://bugs.python.org/issue16042>
___
___
Python-bugs-list m
A.M. Kuchling added the comment:
Version 5 of the patch looks fine; Vajrasky, I think you can go ahead and
commit it.
(I didn't understand RDM's comment about pydoc using DumbWriter; in 3.4, it
doesn't seem to me that pydoc does. What am I missing?)
--
n
A.M. Kuchling added the comment:
BTW, this change is also relevant to 2.7. Previously I wouldn't have bothered
to commit it to 2.7 since the branch was largely closed, but now we're talking
about some corporate maintainer eventually going through and backporting fixes
to the newl
A.M. Kuchling added the comment:
I dislike footnotes and prefer to put things in the text whenever possible.
The text for this change struck as relevant enough -- it notes a property of
the *stmt* parameter -- that it doesn't belong in a footnote. (I'd be happy to
just make it a
A.M. Kuchling added the comment:
Thanks for the patch!
--
assignee: -> akuchling
nosy: +akuchling
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
versions: +Python 3.5 -Python 3.3
___
Python tr
Changes by A.M. Kuchling :
--
versions: +Python 3.5 -Python 3.2
___
Python tracker
<http://bugs.python.org/issue10481>
___
___
Python-bugs-list mailing list
Unsub
A.M. Kuchling added the comment:
Thanks for your patch!
--
nosy: +akuchling
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by A.M. Kuchling :
--
keywords: +easy
___
Python tracker
<http://bugs.python.org/issue17380>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by A.M. Kuchling :
--
versions: +Python 3.5 -Python 3.4
___
Python tracker
<http://bugs.python.org/issue14910>
___
___
Python-bugs-list mailing list
Unsub
A.M. Kuchling added the comment:
No traffic on bug in 2 years, and not clear if there's anything to fix.
Closing.
--
nosy: +akuchling
resolution: -> invalid
stage: -> committed/rejected
status: open -> closed
___
Python
A.M. Kuchling added the comment:
Patch LGTM to me too; someone can probably just commit it.
--
nosy: +akuchling
stage: patch review -> commit review
___
Python tracker
<http://bugs.python.org/issu
New submission from A.M. Kuchling:
io.py contains the following to declare ABCs for some of its classes:
class IOBase(_io._IOBase, metaclass=abc.ABCMeta):
pass
(and similarly for RawIOBase, BufferedIOBase, TextIOBase).
_io._IOBase has an extensive docstring, but IOBase doesn't. (P
Changes by A.M. Kuchling :
--
stage: needs patch -> patch review
___
Python tracker
<http://bugs.python.org/issue20103>
___
___
Python-bugs-list mailing list
Un
A.M. Kuchling added the comment:
Patch to add impl_detail() looks fine to me.
--
versions: +Python 2.7, Python 3.5 -Python 3.2
___
Python tracker
<http://bugs.python.org/issue1704
A.M. Kuchling added the comment:
I agree with Chris's logic; the incorrect size seems to be a curses/ncurses bug
that has gotten fixed somewhere along the line.
You also aren't able to shrink the terminal to be smaller than the size of a
derived window created with derwin(), but y
A.M. Kuchling added the comment:
Because Nick both originally created this ticket and also added the new tests
(http://hg.python.org/cpython/rev/51dddfead80a/), I'll assume he knew what he
was doing when he named those methods, and simply forgot to close this issue.
--
ass
Changes by A.M. Kuchling :
--
components: +IO
versions: +Python 2.7, Python 3.5
___
Python tracker
<http://bugs.python.org/issue21225>
___
___
Python-bugs-list m
A.M. Kuchling added the comment:
Committed to 2.7, 3.4, and default. Thanks for your patch!
--
nosy: +akuchling
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bug
A.M. Kuchling added the comment:
Committed to 2.7, 3.4, and default. Thanks for your patch!
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/iss
A.M. Kuchling added the comment:
Thanks for your patch!
--
assignee: rhettinger -> akuchling
nosy: +akuchling
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://
A.M. Kuchling added the comment:
I thought the discussion of the return code was rather complicated and tried
rewriting it; new patch attached. Is it an improvement? The new version also
specifies the function's parameters.
--
nosy: +akuchling
Added file: http://bugs.pytho
A.M. Kuchling added the comment:
Thanks for your patch!
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
title: os.popen documentation is probably wrong -> os.popen documentation is
probably wrong
__
A.M. Kuchling added the comment:
The attached patch against trunk (3.4) exercises the multiple-authentication
functionality; the SMTP server advertises four different authentication methods,
and the code will try CRAM-MD5 first, which will fail, but LOGIN succeeds.
If this patch is OK, or if
A.M. Kuchling added the comment:
It seems to me that this issue was fixed for Python 3.3 by commit 2dd046be2c88
for #2377, which made importlib the implementation of __import__().
Python/import.c now raises a ValueError if level < 0, and the docstring has
been corrected to have 'lev
A.M. Kuchling added the comment:
Patch looks good with two small changes.
* "for consistency's sakes" is used in 2 places; this should be "sake", not the
plural "sakes".
* "Normally the *path* argument specified to functions": 'specif
A.M. Kuchling added the comment:
Closing, since the original reporter believes the problem was fixed.
--
nosy: +akuchling
resolution: -> out of date
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Changes by A.M. Kuchling :
--
versions: +Python 3.4 -Python 3.3
___
Python tracker
<http://bugs.python.org/issue9584>
___
___
Python-bugs-list mailing list
Unsub
A.M. Kuchling added the comment:
Help is still needed to debug the failing test_glob.py on Windows.
I just tried this patch against 3.4trunk. The first hunk of the glob.py patch
doesn't apply because 'for name in glob1(None, basename): yield name' was
changed to 'y
A.M. Kuchling added the comment:
Patch seems clearly correct; compiles & works on MacOS. We could just apply
this.
--
nosy: +akuchling
versions: +Python 3.4 -Python 3.3
___
Python tracker
<http://bugs.python.org/is
A.M. Kuchling added the comment:
Installation bug; nothing to fix.
--
nosy: +akuchling
resolution: -> invalid
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
A.M. Kuchling added the comment:
Duplicate of #9974. The tokenizer is choking on lines that end in '\'. So
even this can't be parsed:
a = 1 + \
2
--
nosy: +akuchling
superseder: -> tokenizer.untokenize not invariant with
A.M. Kuchling added the comment:
I have no idea.
--
___
Python tracker
<http://bugs.python.org/issue4318>
___
___
Python-bugs-list mailing list
Unsubscribe:
A.M. Kuchling added the comment:
I looked at this a bit and made a revised version of the patch that doesn't add
any line continuations when the token is ENDMARKER. It works on the example
program and a few variations I tried, though I'm not convinced that it'll work
f
A.M. Kuchling added the comment:
MAL suggested adding "a comment explaining why this can be done". If Alexander
or someone wants to do that, great! Otherwise, there seems no other reason to
leave this issue open.
--
nosy: +akuchling
A.M. Kuchling added the comment:
Ping! I'd like to change the function name to clear_frames() and then commit
this. Antoine or anyone, want to disagree with using clear_frames() as the
name?
--
___
Python tracker
<http://bugs.py
A.M. Kuchling added the comment:
The patch requires a little adjusting to apply against 2.6.
--
nosy: +akuchling
Added file: http://bugs.python.org/file31772/smtplib-2.6.patch
___
Python tracker
<http://bugs.python.org/issue16
A.M. Kuchling added the comment:
Here's a final proposed version of the patch for 2.6 that adds a test. Changes
made:
* code now raises SMTPResponseException instead of a new SMTPLineTooLong
exception; bwarsaw deemed that adding a new exception class was changing the
module API.
A.M. Kuchling added the comment:
For 2.6 I'll make a revised version of Giampaolo's patch that doesn't add a new
exception class.
Rationale: Adding a new exception class changes the API of the module, which
we'd like to avoid. If someone is writing 2.6 code that
A.M. Kuchling added the comment:
I took Serhiy's suggestion and just moved up the 'if size' check in the loop.
--
___
Python tracker
<http://bugs.pyt
A.M. Kuchling added the comment:
2.6 version of the patch. Changes from Giampaolo's version of the patch:
* 2.6 didn't have FTP over TLS, so the patch changes slightly to adapt.
* Removed the LineTooLong exception class and just raise Error instead. (This
repeats the message text
301 - 400 of 596 matches
Mail list logo
