Changes by Raymond Hettinger raymond.hettin...@gmail.com:
--
nosy: +rhettinger
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17582
___
___
Raymond Hettinger added the comment:
I think this looks promising.
Don't worry too much about the modest timing improvement. For the most part,
we should almost always take steps to eliminate work that is known to be
unnecessary. The timings serve as a guide but it would be easy for us to
Raymond Hettinger added the comment:
I also teach students and prefer the current behavior. It is egregious for
apps to assume that they are the only thing running and to hide everything else
(i.e. docs viewed through a browser). This isn't the norm for most apps I work
with either.
Serhiy Storchaka added the comment:
How long time the test run (if there is enough memory)? May be we should
decrease memory requirements by using shared values?
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
Łukasz Langa added the comment:
Awesome! Thank you for your patch, Kasia. For the record, I left the non-magic
Mock behavior untouched since Kasia rightfully points out that in this case
m.configure_mock() also does not work.
--
keywords: -needs review
resolution: - fixed
status:
Raymond Hettinger added the comment:
I don't agree that this is a bug that should be fixed. It adds code that will
likely never get called or needed (i.e. there has never been a request for this
in the decade long history of desciptors and it seems like a made up
requirement to me.
I
Alex Shkop added the comment:
Removed unnecessary assertWarns() contexts and fixed spelling errors.
--
Added file:
http://bugs.python.org/file38982/issue18383_assert_warns_and_dups_v2.patch
___
Python tracker rep...@bugs.python.org
Berker Peksag added the comment:
Thanks Piotr.
--
nosy: +berker.peksag
resolution: - fixed
stage: - resolved
status: open - closed
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23943
Roundup Robot added the comment:
New changeset 8e5592a43d65 by Łukasz Langa in branch '3.4':
Issue #23310: Fix MagicMock's initializer to work with __methods__.
https://hg.python.org/cpython/rev/8e5592a43d65
New changeset dd8f48ff9480 by Łukasz Langa in branch 'default':
Merge 3.4 (#23310)
Serhiy Storchaka added the comment:
I can't test the patch on UNIX, this branch of the code is executed only on
Windows.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23908
___
Roundup Robot added the comment:
New changeset 9f2bd9939d4b by Berker Peksag in branch '3.4':
Issue #23943: Fix typos. Patch by Piotr Kasprzyk.
https://hg.python.org/cpython/rev/9f2bd9939d4b
New changeset 4fb2075952a4 by Berker Peksag in branch 'default':
Issue #23943: Fix typos. Patch by Piotr
Roundup Robot added the comment:
New changeset 4d1236b180be by Łukasz Langa in branch 'default':
Issue #23310: Fix MagicMock's initializer to work with __methods__.
https://hg.python.org/cpython/rev/4d1236b180be
--
nosy: +python-dev
___
Python
Raymond Hettinger added the comment:
Unassigning because I don't have an interest in going further down this path.
I used this technique with zip() because tuple reuse nicely benefited the
common case of for a,b,c in zip(alist, blist, clist):
On another thread, I did see mention of
Serhiy Storchaka added the comment:
I think we can ignore this flaw. 'pydoc None' works, and any solution for None
aliases could break existing code that use private pydoc functions.
Here is a patch with a test.
--
stage: test needed - patch review
Added file:
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: -haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23193
___
___
Python-bugs-list
STINNER Victor added the comment:
The default (and standards-violating) behavior of the Windows CRT is to kill
the process for a bad file descriptor, instead of just setting errno to EBADF.
Ah, if the issue is the behaviour of the MSVCRT on EBADF, Steve Dower can help
you. He wrote the
STINNER Victor added the comment:
I can't test the patch on UNIX, this branch of the code is executed only on
Windows.
If you write a test which ensures that a path with a null character raises a
TypeError, it should pass on UNIX on your PC. Then builbots will ensure that it
also pass on
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: -haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9951
___
___
Python-bugs-list
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: -haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22980
___
___
Python-bugs-list
STINNER Victor added the comment:
@Benjamin: since you are the maintainer of Python 2.7, I would like your
opinion on this issue.
Python older than 3.5 does not handle EINTR for you: if you register a signal
handler for a signal, the signal handler does not raise an exception, and a
syscall
Serhiy Storchaka added the comment:
Added few comments on Rietveld.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23935
___
___
Serhiy Storchaka added the comment:
map doesn't use PyObject_CallFunctionObjArgs(). If compare results for map with
results for filter and list.sort, seems that PyTuple_New() is responsible for
24% and PyObject_CallFunctionObjArgs() adds other 14%.
--
priority: normal - low
Serhiy Storchaka added the comment:
May be don't add __doc__ if sys.flags.optimize = 2?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23900
___
Changes by Łukasz Langa luk...@langa.pl:
--
stage: needs patch - resolved
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23310
___
___
STINNER Victor added the comment:
avoid memset in small tuple creation
I don't understand this title because there is no call to memset() in the patch.
Can you try to modify PyTuple_New() to use memset() instead of a dummy loop?
Adding a _PyTuple_New() which doesn't initialize the memory
New submission from andig2:
Setup: IE default browser, FF37 preferred browser and configured in BROWSER env
variable
Test:
import webbrowser
url = http://localhost;
webbrowser.open(url + '?XDEBUG_SESSION_START=sublime.xdebug')
Behaviour:
1st run: FF opens - ok
2nd run: FF opens new tab -
Serhiy Storchaka added the comment:
I afraid not that builbots will fail, but that that test could pass without the
fix. Tests should be tested without applying the fix. Writing right tests can
require several iterations with running tests on patched and unpatched the os
module.
Here is a
Serhiy Storchaka added the comment:
Instead of the loop you can use writelines():
f.writelines([b'\0' * bs] * (size // bs))
It would be nice to add a comment that estimate why os.ftruncate() or
seek+write can't be used here. At least a link to this issue with short
estimation.
Chris Abiad added the comment:
Running through each of the demos, it seems as though only 'penrose' and 'tree'
have console/shell output.
'penrose' output is a list of information about the current rendering and has a
format like:
Calculation:0.0009 s
Drawing: 0.0074 s
Together: 0.0083
Heather McCartney added the comment:
You're welcome! Thanks for the feedback, I've signed the PSF Contributor Form.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21400
___
Carol Willing added the comment:
Reviewed the issue history during the PyCon sprints. Thanks to all for the
thoughts on this issue. I am closing this issue since any additional actions on
this subject would be best filed as a new issue.
--
nosy: +willingc
resolution: - out of date
Joe Jevnik added the comment:
I am not sure yet; I don't have access to a machine that can run the test.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5438
___
andig2 added the comment:
And finally: FF returns exit code 1 when its started and already running. That
situation is not handled by the GenericBrowser Popen logic.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23945
New submission from Cristi Fati:
In WinPE environment (at least this is where I found it, but there might be
cases where this issue could be spotted in normal Windows), some folders might
have some of the FILETIME structures reset (because of RAMDrive?). When the
conversion between WIN style
Carol Willing added the comment:
Thanks Heather for filing the PSF contribution form as well as the patch. I
hope you will continue improving Python's documentation in the future :)
--
___
Python tracker rep...@bugs.python.org
andig2 added the comment:
Here is a matrix of the test results depending on which browser is open upon
starting the test script:
No browser open: FF opens tab (ok)
IE open: new tab in FF (ok), nothing in IE (ok)
FF open: new tab in FF (ok), new tab in in IE (NOT ok)
Both open: new tab in FF
andig2 added the comment:
Looking further in webbrowser.py, FF uses GenericBrowser:
class GenericBrowser(BaseBrowser):
Class for all browsers started with a command
and without remote functionality.
...
def open(self, url, new=0, autoraise=True):
cmdline =
Carol Willing added the comment:
Reviewed this issue's history at PyCon sprint. The devguide now provides
results if searched for 'server side' or 'clone'. This issue is resolved.
If additional search terms are required in the index, it is recommended that a
new issue with specific terms be
andig2 added the comment:
It boils down to Popen.wait() returning 1 if FF is already open resulting in
GenericBrowser.open() returning False.
That part was last touched here
https://github.com/python/cpython/commit/a456db5e058f955f235fe7a51e8c111d0a8ecf4e
--
Claudiu Popa added the comment:
Will do, thanks.
--
assignee: - Claudiu.Popa
stage: - needs patch
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23917
___
Steve Dower added the comment:
I'll soon have Windows and Ubuntu machines with 448GB RAM each - anything
special I should test here?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5438
Serhiy Storchaka added the comment:
The patch mostly LGTM. Most changes are trivial, but the patch also fixes few
bugs. It is nice that the patch adds empty lines after blocks, this makes
copying the code to interactive interpreter easier. I left comments on Rietveld.
The most questionable
R. David Murray added the comment:
I believe BJ is saying the same thing that Paul is, that SUPPRESS could omit
the subparser from the list of subparsers. I haven't checked, but I believe
this means that the proposed patch doesn't do anything other than what omitting
the help would do, so a
Dustin J. Mitchell added the comment:
From my read of this bug, there are two distinct tasks mentioned:
1. make PyTokenizer_* part of the Python-level API
2. re-implement 'tokenize' in terms of that Python-level API
#1 is largely complete in Andrew's latest patch, but that will likely need:
Changes by Berker Peksag berker.pek...@gmail.com:
--
stage: patch review - resolved
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12652
___
___
Carol Willing added the comment:
Thanks to all for the feedback. Since this issue covers a sufficiently broad
set of thoughts, it would be better suited for a PEP. Closing this issue with
the recommendation to submit a PEP if interested.
--
nosy: +willingc
resolution: - not a bug
Joe Jevnik added the comment:
The tests need to be run with higher memory caps, this can be set with the -M
flag, like python -m test -M 64G test_bigmem. I am under the impression that
these tests don't get run all the time so we might want to run that suite.
--
Raúl Cumplido added the comment:
Forget my previous comment. Done with Linux and had the initial behavior again:
s = socket.socket()
s.type
SocketType.SOCK_STREAM: 1
s.settimeout(2)
s.type
2049
--
___
Python tracker rep...@bugs.python.org
A.M. Kuchling added the comment:
No further information from poster; please comment if you still wish to pursue
this. Closing this issue as obsolete.
If I'm interpreting the uname -a output correctly, this is HP-UX 11.11,
released in 2000 according to
New submission from Rob King:
Hello, everyone.
I've noticed an issue that could be just a documentation inaccuracy or a
genuine, minor bug in cgi.FieldStorage.
The documentation for the module states:
You can test for an uploaded file by testing either the filename attribute or
the file
New submission from Carol Willing:
It would be helpful to update the conf.py file and dependencies to use a
modified 'pydoctheme' for the devguide style. The benefits would be improved
readability, a consistent look with the stdlib docs, and collapsible sidebars.
Attached is a screenshot of a
Zachary Ware added the comment:
My bad, you're right. Thanks!
--
resolution: - out of date
stage: needs patch - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17751
Steve Dower added the comment:
Just for fun, I attached the memory usage graph from the Windows run. You can
see the long pauses in the middle for the two changed tests, and the graph only
goes up to 57GB despite the limit being set at 440GB.
--
Added file:
Ulaga Nathan Mahadevan added the comment:
I agree. I posted it as an enhancement request. Steven explained the
expected message clearly. Thanks.
On Tue, Apr 14, 2015 at 12:58 PM, Steven D'Aprano rep...@bugs.python.org
wrote:
Steven D'Aprano added the comment:
The error message isn't wrong,
Changes by Kasia Jachim katarzyna.jac...@gmail.com:
--
nosy: +lukasz.langa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23078
___
___
Demian Brecht added the comment:
Although it's likely not going to be a high frequency issue (but would be
painful to track down when it does become an issue), by doing an
int(float(expires)), you're losing the sub-second portion of the expiry. Why
not something like this:
try:
expires =
Demian Brecht added the comment:
Changing to behavior type as crash is generally used to indicate an
interpreter issue.
--
nosy: +orsenthil
stage: - patch review
type: crash - behavior
___
Python tracker rep...@bugs.python.org
A.M. Kuchling added the comment:
Closing as obsolete.
--
nosy: +akuchling
resolution: - wont fix
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6006
Berker Peksag added the comment:
Thank you Alex.
--
nosy: +berker.peksag
resolution: - fixed
stage: needs patch - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23811
A.M. Kuchling added the comment:
Closing this issue; judging by the comments, the fix should be pushed off to
setuptools/distribute/whatever.
--
nosy: +akuchling
resolution: - wont fix
stage: - resolved
status: open - closed
___
Python tracker
Zachary Ware added the comment:
One benefit to having a different theme for the devguide is that it makes clear
that you're not in the standard library docs anymore. I agree that that
screenshot looks quite a bit better, though.
--
nosy: +zach.ware
Gregory P. Smith added the comment:
fyi - i didn't update the 2.7 docs. just 3.4 and 3.5. if some committer wants
to, feel free.
--
nosy: +gregory.p.smith
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22046
Changes by Gregory P. Smith g...@krypto.org:
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22046
___
Zachary Ware added the comment:
Larry: Here's your sample:
/*[clinic input]
_winapi.CreateProcess
application_name: Py_UNICODE(nullable=True)
command_line: Py_UNICODE(nullable=True)
proc_attrs: object
Ignored internally, can be None.
thread_attrs: object
Ignored
A.M. Kuchling added the comment:
#17898 has been closed with basically the same fix as Mark's patch, so I'll
close this issue.
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
New submission from Steve Dower:
There are some situations where the stdlib imports modules that could be
absent, notably import readline in site.py. This import is expected to fail
in some situations, but because sys.path is fully configured it can be
importing arbitrary code.
To limit
Vincent Alquier added the comment:
Martin: You're right, it's the same issue, and only related to python2's old
style classes. Sorry for the useless noise.
Demian: My problem is `len(obj)` raises an Using AttributeError in python2
(with obj being old style class instance). It's python 2.X
Roundup Robot added the comment:
New changeset 22790c4f3b16 by Berker Peksag in branch '2.7':
Issue #23811: Add missing newline to the PyCompileError error message.
https://hg.python.org/cpython/rev/22790c4f3b16
--
___
Python tracker
Changes by Gregory P. Smith g...@krypto.org:
--
assignee: - gregory.p.smith
nosy: +gregory.p.smith
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9859
___
Antoine Pitrou added the comment:
Therefore, I'm finally closing this as won't fix :)
--
resolution: - wont fix
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21039
___
Roundup Robot added the comment:
New changeset 3e8047ee9bbb by Gregory P. Smith in branch '3.4':
issue22046: mention that zipfile can raise NotImplementedError on unsupported
https://hg.python.org/cpython/rev/3e8047ee9bbb
New changeset 4b9deb7e6f2b by Gregory P. Smith in branch 'default':
Eric V. Smith added the comment:
Thanks for your review, Berker. I've updated the code with most of your
suggestions, although some of them were mooted by some restructuring I did.
A couple of questions/issues:
- I'm not sure where we stand on keyword-only arguments. I certainly agree that
Dustin J. Mitchell added the comment:
New:
- rename token symbols in token.h with a PYTOK_ prefix
- include an example of using the PyTokenizer functions
- address minor review comments
--
Added file: http://bugs.python.org/file38999/issue3353-2.patch
A.M. Kuchling added the comment:
No activity on this issue since 2007; HP-UX 11.23 was released in 2003. Closing
as outdated.
--
nosy: +akuchling
resolution: - out of date
stage: - resolved
status: open - closed
___
Python tracker
Joe Jevnik added the comment:
Has the test passed or is it still hung? Also, I guess this means that the
tuple is over-allocating, maybe I need to revise the memory assumptions to
account for the growth algorithm.
--
___
Python tracker
Berker Peksag added the comment:
Thanks for the patch! I've reviewed it on Rietveld:
http://bugs.python.org/review/9859/
--
nosy: +berker.peksag
stage: needs patch - patch review
versions: +Python 3.5 -Python 3.2
___
Python tracker
Ofer Schwarz added the comment:
There's actually no advantage for a weakref here, since the exception object
already holds an indirect reference to all the raising frame's locals through
__traceback__.
--
nosy: +blackfawn
___
Python tracker
Antoine Pitrou added the comment:
I beg to disagree :) Pathlib tries to find a compromise between
user-friendliness and power, but it's definitely more on the user-friendliness
side than, say, the os module APIs. In other words, I don't think it's a
problem if not all details of OS semantics
Antoine Pitrou added the comment:
It is a new feature.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22631
___
___
Python-bugs-list mailing
Larry Hastings added the comment:
Benjamin, would you like CAN_RAW_FD_FRAMES in the next 2.7?
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22631
___
Joe Jevnik added the comment:
I think that the idea is not to get as close to the limit as possible but just
to hard cap the memory usage of the test suite so that it doesn't get
oom-killed. twouters, does this sound correct? Also, I think that this means
that the new decorator is reporting a
New submission from Rob King:
The cgi module has a global variable, 'maxlen', that specifies the maximum
length of a POST request. By default, this limit is 0, meaning an unlimited
POST request size.
Having an unlimited default opens up CGI scripts to resource-exhaustion
attacks. Setting the
Changes by Rob King jk...@deadpixi.com:
--
versions: +Python 2.7, Python 3.2, Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23952
___
New submission from Ulaga Nathan Mahadevan:
data = ['David',50,91.1,(2012,12,21)]
print (Data = ,data)
name,shares,price,date,value = data
print(Name = ,name, , no of shares = ,shares,, unit price = ,price,,
date of purchase = ,date)
After running the script
Data = ['David', 50, 91.1, (2012,
Steve Dower added the comment:
It's passed. There was no memory usage increase from that point, but after
about 20 minutes it completed successfully.
All the tests passed on Ubuntu, but again, they didn't really fill up RAM -
50GB looks like the high point.
--
Skip Montanaro added the comment:
I'm not sure I understand the problem is. You have a list containing four
values, and try to unpack it into five individual objects. There is no element
in the list data to assign the name value.
--
nosy: +skip.montanaro
A.M. Kuchling added the comment:
The subsequent discussion didn't arrive at a consensus. Most commenters wanted
to drop --disable-unicode support and thought these patches were too extensive
for 2.7, but Serhiy noted that running the tests with --disable-unicode turns
up problems in the
Larry Hastings added the comment:
Isn't this a new feature? 2.7 is maybe fair game, but 3.4 arguably is not.
It should be odd to have a field marked in the documentation as New in 3.4.4.
--
___
Python tracker rep...@bugs.python.org
Changes by A.M. Kuchling a...@amk.ca:
--
nosy: +akuchling, fdrake
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22211
___
___
Python-bugs-list
Daniel Stutzbach added the comment:
With regret, I have not had time to work on patches and am unlikely to have
time in the near future.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9859
Marin Dantchev added the comment:
@Zachary - while that is doable, ctypes.macholib.dyld seems to be available in
the dev version of 3.5.
import ctypes.macholib.dyld
ctypes.macholib.dyld.__file__
'/home/morkov/dev/cpython/Lib/ctypes/macholib/dyld.py'
from ctypes.macholib.dyld import
Steven D'Aprano added the comment:
The error message isn't wrong, just hard to understand. In your example, when
it says need more than 4 values to unpack it means that it got four values on
the right hand side, and needs five (which is more than four) to match what is
on the left.
When the
Changes by Michael Schurter m...@schmichael.com:
--
nosy: -schmichael
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20669
___
___
Thomas Kluyver added the comment:
6a following in-person review with Gregory:
- Reapplied to the updated codebase.
- Docs: mention the older functions near the top, because they'll still be
important for some time.
- Docs: Be explicit that combined stdout/stderr goes in stdout attribute.
-
Ned Deily added the comment:
The wording of the patch looks good to me other than one issue: which hook to
use. I notice Nick's suggestion is to use the hg commit hook. But in the
comments for the checkwhitespace hook itself, pretxncommit is the suggested
hook. The difference between them,
Larry Hastings added the comment:
That's AMAZING, you mean the web site knew how to do that THE WHOLE TIME?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23944
___
A.M. Kuchling added the comment:
The markup is not a huge deal, I think. I also did some bits of rewording that
can probably be ignored. The TRACE_REFS question is the only important one, I
think.
--
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 752ea0acc37e by Gregory P. Smith in branch '3.4':
issue9014: Include more formatting on :c:type:`PyObject` etc.
https://hg.python.org/cpython/rev/752ea0acc37e
New changeset 78a2d1169be1 by Gregory P. Smith in branch 'default':
issue9014: Include
Michael McFadden added the comment:
I have a patch for moving script_helper as R. David suggested.
Here it is.
--
nosy: +flipmcf
Added file:
http://bugs.python.org/file39003/iss9517_move_script_helpers_py.patch
___
Python tracker
Jeff Ramnani added the comment:
So, the devguide has been updated since this issue was opened. The Quick Start
section now has a link to build documentation, which includes information about
build dependencies. Is this sufficient to call this bug closed?
--
nosy: +jramnani
1 - 100 of 275 matches
Mail list logo