New submission from Dmitry Kazakov:
This
if there are any values in iterable that are note bytes-like objects
should be
if there are any values in iterable that are not bytes-like objects
Here's a micropatch...
--
assignee: docs@python
components: Documentation
files:
Paul Moore added the comment:
I'm not a heavy user of concurrent.futures, so I don't have a strong opinion.
I've never felt a need for this function, though, so I guess I'm -0. A recipe
in the docs would be good, though.
--
___
Python tracker
Serhiy Storchaka added the comment:
As a workaround specify mode='r' for GzipFile.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18323
___
___
Changes by Serhiy Storchaka storch...@gmail.com:
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23637
___
Roundup Robot added the comment:
New changeset ba271f116f94 by R David Murray in branch '3.4':
#24216: fix typo
https://hg.python.org/cpython/rev/ba271f116f94
New changeset 91ac3040a28d by R David Murray in branch 'default':
Merge: #24216: fix typo
https://hg.python.org/cpython/rev/91ac3040a28d
Terry J. Reedy added the comment:
Assuming that the details are correct, which I cannot check on Windows :-(,
this looks good to me. If the tkinter doc is expanded and reorganized, that
section might move but I cannot see a better place for it now. Martin, have you
used that code (with
Barry A. Warsaw added the comment:
An increase of 13 characters doesn't seem so bad.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6598
___
___
R. David Murray added the comment:
Thanks, Maciej.
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24211
___
Serhiy Storchaka added the comment:
Currently the maximal length of local part is
14+1+len(str(2**16))+1+len(str(10**5-1)) == 26
With the patch it will be
len(str(2**31*100))+1+len(str(2**16))+1+len(str(2**64)) = 39
If encode all three numbers with hexadecimal, it will be
New submission from Jeffrey Armstrong:
While compiling on Linux/x86 with Open Watcom, I've run into the following at
link time in Modules/mmapmodule.c:
./Modules/mmapmodule.c(1223): Error! E1011: Symbol 'O_RDWR' has not been
declared
The constant isn't defined because fcntl.h isn't included.
Changes by Jeffrey Armstrong jeffrey.armstr...@approximatrix.com:
--
resolution: - wont fix
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20596
___
Jeffrey Armstrong added the comment:
Indeed, I agree. Let's try this new patch.
--
Added file:
http://bugs.python.org/file39408/mmapmodule.py3.5.0a3.HAVE_FCNTL_H.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24217
R. David Murray added the comment:
Thanks, Dmitry.
--
nosy: +r.david.murray
resolution: - fixed
stage: - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24216
Skip Montanaro added the comment:
The defaults for the Dialect class are documented:
https://docs.python.org/2/library/csv.html#dialects-and-formatting-parameters
I think the problem is mostly that csv.Dialect must be subclassed. You can't
use it directly, and if you subclass it as MiK did,
Christian Heimes added the comment:
I think it's more portable with a #ifdef HAVE_FCNTL_H block around the include.
--
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24217
Jeffrey Armstrong added the comment:
I thought, while I'm here reporting another bug, I'd bump this once more.
There is a patch here, and it corrects clearly broken code. Should I mark this
as won't fix?
--
___
Python tracker
Terry J. Reedy added the comment:
The idea, obviously, is for python -m idlelib to also work on all systems.
When this was requested as part of the discussion on another issue, I thought
there was no idlelib.__main__ because __main__ did not work on 2.7. Someone
corrected me and the
New submission from R. David Murray:
Now that I've committed issue 24211, we can also add SMTPUTF8 support to
smptlib's send_message command. See attached patch.
--
files: send_message_smtputf8.patch
keywords: patch
messages: 243413
nosy: r.david.murray
priority: normal
severity:
Changes by R. David Murray rdmur...@bitdance.com:
--
components: +email
nosy: +barry
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24218
___
___
Changes by Jakub Wilk jw...@jwilk.net:
--
nosy: +jwilk
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16314
___
___
Python-bugs-list mailing list
Serhiy Storchaka added the comment:
I'm not sure that this is right place to document these functions (they are
methods of undocumented class), but the patch looks correct to me.
If you will commit the patch, Terry, note that tkinter should be replaced with
Tkinter in 2.7.
--
Zachary Ware added the comment:
I'll try to have a look at this later today, but unfortunately can't
guarantee anything (I don't know C, MSI, or _msi very well).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21931
Changes by Maciej Szulik solt...@gmail.com:
--
nosy: +maciej.szulik
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24218
___
___
Python-bugs-list
Nick Coghlan added the comment:
I'm now +1 on a recipe in the docs, -0 on a concrete method on Executor
that delegates to the map() method.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24195
Alex Lord added the comment:
There was a bunch of things wrong with that patch.
In addition to the issues you brought up in the review I was mixing up what the
actual problem is
REPLACE INTO is an alias for INSERT OR REPLACE. INSERT OR REPLACE was correctly
setting the lastrowid values but
Martin Panter added the comment:
For the record, I have used this code in 2.7 in the past (though I mainly use
Python 3). The Python 2 version of the changes look fine to me.
--
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
All lines need to be wrapped to 80 columns.
The idea behind the loop is this:
sql = {} INTO test(id, unique_test) VALUES (?, ?)
self.cu.execute(sql.format('INSERT OR REPLACE, (1, foo)
for statement in [INSERT OR REPLACE, REPLACE]:
with
Terry J. Reedy added the comment:
Here is what I meant, but I cannot test on Windows. Mark, do you have *nix
currently?
--
Added file: http://bugs.python.org/file39412/has_key.diff
___
Python tracker rep...@bugs.python.org
New submission from Raymond Hettinger:
The siftup() and siftdown() routines rearrange pointers in a list. The
generated code repeats the list object to ob_item lookup for each access. This
patch does that lookup only once per iteration. It cleans up the code by
replacing the
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
Added file: http://bugs.python.org/file39414/time_heapify.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24221
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
resolution: remind - postponed
stage: - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2571
___
R. David Murray added the comment:
Thanks Maciej.
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24218
___
R. David Murray added the comment:
Thanks everyone.
--
resolution: - fixed
stage: commit review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13866
___
Zachary Ware added the comment:
I committed a presence test for Tix in #21337 that should catch problems with
loading Tix after this patch. There shouldn't be any problem though, we
actually install Tix into Tcl/Tk whereas TIX_LIBRARY is (to my understanding)
for helping Tcl/Tk find a Tix
Martin Panter added the comment:
I suggest this is a duplicate of Issue 15112. The same problem also happens
with nargs=*, and that issue apparently has a patch to handle both cases.
For the record, this is the resulting error, and a demo that it works if the
option comes before the
Carol Willing added the comment:
Matthias, Did you wish to mark this issue for additional Documentation and as a
behavior change request (warning of signature change until 3.6) in addition to
Devguide?
I've summarized below some info from researching Matthias' original message. I
see three
Terry J. Reedy added the comment:
Tom, please sign the PSF Contributor Agreement
https://www.python.org/psf/contrib/
https://www.python.org/psf/contrib/contrib-form/
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10685
Roundup Robot added the comment:
New changeset c7d82a7a9dea by R David Murray in branch 'default':
Issue #13866: add *quote_via* argument to urlencode.
https://hg.python.org/cpython/rev/c7d82a7a9dea
--
nosy: +python-dev
___
Python tracker
Mark Lawrence added the comment:
Sorry, Windows 8.1 only.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10598
___
___
Python-bugs-list mailing
New submission from Bob Alexander:
Here is simple example of failure to parse arguments that should parse OK. In
the following little program, the second from last line contains an aargument
sequence that parses OK, but the last line should but doesn't.
import argparse
ap =
Zachary Ware added the comment:
As a prerequisite for #20035, I've added the simplest test possible that just
makes sure Tix is available on Windows. I only added it on default as I don't
have the facilities to test it properly on 2.7 and 3.4 on Windows (which is the
only platform actually
Martin Panter added the comment:
I did a rough merge with current “default” (3.5 pre-release) branch so that I
can have a closer look at this issue; see xmlbomb_20150518.patch for the
result. There are some bits with Argument Clinit that need perfecting:
* Unsure how to convert the
Martin Panter added the comment:
Closed Issue 24223 as a duplicate. I understand the patch here also fixes the
case of an --option before an optional positional argument using nargs=?; is
that right?
--
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 30795a477f85 by R David Murray in branch 'default':
#24218: Add SMTPUTF8 support to send_message.
https://hg.python.org/cpython/rev/30795a477f85
--
nosy: +python-dev
___
Python tracker
Roundup Robot added the comment:
New changeset 4a254750ad20 by R David Murray in branch 'default':
#24211: Add missing docs for SMTPUTF8 policy instance.
https://hg.python.org/cpython/rev/4a254750ad20
--
___
Python tracker rep...@bugs.python.org
Terry J. Reedy added the comment:
After looking at these results in relation to the qualitative 'speed'
documentation, I cannot think of any any enhancement appropriate for turtle doc
audience. So I withdraw my request.
--
resolution: - works for me
stage: - resolved
status: open -
R. David Murray added the comment:
Looks to like there is nothing left to do here, so I'm closing it.
--
resolution: - fixed
stage: commit review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
New submission from Terry J. Reedy:
C:\Users\Terrypy -2 -m idlelib.idle -c print True
*** Error in script or command!
Traceback (most recent call last):
File pyshell#0, line 1
print True
^
SyntaxError: invalid syntax
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
priority: normal - low
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24221
___
___
Roundup Robot added the comment:
New changeset eeea91d0506b by Zachary Ware in branch 'default':
Issue #21337: Add bare-bones Tix test
https://hg.python.org/cpython/rev/eeea91d0506b
--
nosy: +python-dev
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 16d0e3dda31c by Zachary Ware in branch 'default':
Issue #23488: Fix a syntax error on big endian platforms.
https://hg.python.org/cpython/rev/16d0e3dda31c
--
___
Python tracker rep...@bugs.python.org
Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23757
___
Roundup Robot added the comment:
New changeset c79530e08985 by Raymond Hettinger in branch '3.4':
Issue #23757: Only call the concrete list API for exact lists.
https://hg.python.org/cpython/rev/c79530e08985
--
___
Python tracker
Changes by Arnon Yaari wiggi...@gmail.com:
--
nosy: +wiggin15
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12485
___
___
Python-bugs-list mailing
Changes by Arnon Yaari wiggi...@gmail.com:
--
nosy: +wiggin15
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20362
___
___
Python-bugs-list mailing
Changes by Arnon Yaari wiggi...@gmail.com:
--
nosy: +wiggin15
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20769
___
___
Python-bugs-list mailing
Terry J. Reedy added the comment:
My succinct version:
Return the dictionary implementing the current module namespace. For code
within functions, this is set when the function is defined and remains the same
regardless of where a function is called.
--
versions: +Python 3.5 -Python
Changes by Terry J. Reedy tjre...@udel.edu:
--
resolution: - out of date
stage: - resolved
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20895
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
stage: patch review - resolved
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19846
___
___
Changes by Zachary Ware zachary.w...@gmail.com:
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21931
___
Zachary Ware added the comment:
It's a simple enough fix, so I went ahead and committed it. Thanks for the
patch!
I've created issue24224 to track the addition of tests to test_msilib (for more
than just this issue).
--
resolution: fixed -
stage: resolved - patch review
status:
New submission from Zachary Ware:
test_msilib doesn't test much and could do with good expansion.
Marked for 3.5, but 3.4 and 2.7 are just as barren if whoever takes this feels
particularly ambitious.
--
components: Tests, Windows
keywords: easy
messages: 243458
nosy: steve.dower,
Matthias Bussonnier added the comment:
Hi Carol,
Matthias, Did you wish to mark this issue for additional Documentation and as
a behavior change request (warning of signature change until 3.6) in addition
to Devguide?
Hum, probably, I'm not super familiar with this bug tracker and it
Roundup Robot added the comment:
New changeset 41281737d71a by Zachary Ware in branch '3.4':
Issue #21931: Fix error handling in msilib.FCICreate().
https://hg.python.org/cpython/rev/41281737d71a
New changeset cb0d1d86215e by Zachary Ware in branch 'default':
Closes #21931: Merge with 3.4
Changes by Maciej Szulik solt...@gmail.com:
--
nosy: +maciej.szulik
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24211
___
___
Python-bugs-list
Maciej Szulik added the comment:
David one small nit in docs, but otherwise the patch looks good, applies
cleanly and works as expected.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24211
Changes by Jeffrey Armstrong jeffrey.armstr...@approximatrix.com:
Removed file: http://bugs.python.org/file39407/mmapmodule.py3.5.0a3.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24217
___
Roundup Robot added the comment:
New changeset 9f0d5e33230f by R David Murray in branch 'default':
#24211: Add RFC6532 support to the email library.
https://hg.python.org/cpython/rev/9f0d5e33230f
--
nosy: +python-dev
___
Python tracker
Terry J. Reedy added the comment:
Since test files should no longer have test_main, test_trace should no longer
look for one. (And indeed, test_trace itself should be converted, which looks
trivial.) The offending line is
cmd='from test import test_pprint; test_pprint.test_main()'):
The use
New submission from Matthias Bussonnier:
The `ast.Call/_ast.Call` signature, changed on May 5th
(https://hg.python.org/cpython/rev/95886) which causes some error in package
using ast
(https://bitbucket.org/pytest-dev/pytest/issue/744/pytest-assert-rewriting-broken-with-python#).
There does
Roundup Robot added the comment:
New changeset b6121a4afad7 by Raymond Hettinger in branch '2.7':
Issue #23757: Only call the concrete list API for exact lists.
https://hg.python.org/cpython/rev/b6121a4afad7
--
nosy: +python-dev
___
Python tracker
Serhiy Storchaka added the comment:
Raymond's patch LGTM. Here is updated patch with tests.
--
assignee: - rhettinger
keywords: +patch
stage: needs patch - commit review
Added file: http://bugs.python.org/file39404/fix_list_to_tuple_2.diff
___
Roundup Robot added the comment:
New changeset 247f003b42ea by Serhiy Storchaka in branch '2.7':
Issue #15809: IDLE shell now uses locale encoding instead of Latin1 for
https://hg.python.org/cpython/rev/247f003b42ea
--
nosy: +python-dev
___
Python
New submission from Dmitry Kazakov:
One of the integers under Some examples of integer literals is repeated twice:
7 21474836470o1770b100110111
3 79228162514264337593543950336 0o3770x1
79228162514264337593543950336
Roundup Robot added the comment:
New changeset df81f9b67736 by R David Murray in branch 'default':
#20098: add mangle_from_ policy option.
https://hg.python.org/cpython/rev/df81f9b67736
--
nosy: +python-dev
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
Thanks, Milan.
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20098
___
R. David Murray added the comment:
At this point (3.5) we have settled on having process_message be required to
have a **kwargs parameter in its signature in order to support new features
(ex: enable_SMTPUTF8). So the solution here would be to use inspect.signature
to find out if
Roundup Robot added the comment:
New changeset e8dd65c7fb6b by Terry Jan Reedy in branch '2.7':
Issue #22155: Add File Handlers subsection with createfilehandler to Tkinter
https://hg.python.org/cpython/rev/e8dd65c7fb6b
New changeset 1efcb7b6ec8a by Terry Jan Reedy in branch '3.4':
Issue
Terry J. Reedy added the comment:
Martin, thanks for the nice patch.
--
assignee: docs@python - terry.reedy
resolution: - fixed
stage: patch review - resolved
status: open - closed
type: - behavior
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 8bd9da5635fe by Terry Jan Reedy in branch '3.4':
Issue #22155: News entry.
https://hg.python.org/cpython/rev/8bd9da5635fe
New changeset 850cbd54cc73 by Terry Jan Reedy in branch 'default':
Issue #22155: News entry.
Maciej Szulik added the comment:
David one small comment regarding typo in smtplib.py, but most importantly I'd
suggest adding additional test case to cover the path (the newly added one)
where you get a UnicodeEncodeError upon encoding from or to with UTF8 and then
failing to find SMTPUTF8
R. David Murray added the comment:
Oh, right, that's what I get for doing this at the end of a long chain of patch
reviews :). I added that code after I'd written the test and forgot to go back
and test it.
--
___
Python tracker
Ram Rachum added the comment:
Okay, let me understand what the opinions of people are about this feature.
Ram Rachum: +1
Antoine Pitrou: -1
Nick Coghlan: Is that a +1 or a +0?
Ethan and Paul, you've participated but I didn't get your opinion on whether
you want to see this feature go in.
If
Martin Panter added the comment:
Here is a patch which moves the documentation from the FAQ into a new section
at the bottom of the main “tkinter” reference manual page, and updates it. I
also mentioned lack of Windows support. Let me know what you think!
--
keywords: +patch
stage:
Serhiy Storchaka added the comment:
Here is a patch that changes the generating of message IDs:
1. The datetime is taken with higher precision, 2 decimal digits after dot.
2. The datetime is written as Unix time in 0.01 second units, not as
mmddHHMMSS. This is more compact and faster.
3.
Steve Dower added the comment:
There's an MSBuild property that I set for release builds (either
BuildForRelease or ReleaseBuild, not Configuration) that should exclude the
preprocessor directive so we don't include an unnecessary full path. I can make
that change later if you don't want to
Skip Montanaro added the comment:
In your Mon class, you've left the doublequote parameter unset (it defaults to
None). It completely overrides the default dialect. When no Dialect class is
given, the default is csv.excel. Note that doublequote is set to True in
csv.excel. In your second
Mik added the comment:
Ok Thanks.
But perhaps the documentation of csv.Dialect would be updated with the default
parameters. If all attribute may be specified this would be indicated in the
doc.
--
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
Looking at my mail store, it looks like one more more mail clients use a GUID
for the messageid, which is 36 characters. So 39 doesn't seem so bad. There's
even one that is 74 characters long. There are also shorter ones, though, so
compactifying the
R. David Murray added the comment:
Yes, I think the documentation should be improved.
--
assignee: - docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python, r.david.murray
resolution: not a bug -
stage: - needs patch
status: closed - open
title: doublequote are not
90 matches
Mail list logo