Chris Angelico added the comment:
Another good option is read/write without the 'fd' suffix. Either works, I'd
prefer the shorter one but by a small margin.
--
nosy: +Rosuav
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24536
Changes by Christian Heimes li...@cheimes.de:
--
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24534
___
___
Nick Coghlan added the comment:
As others have noted, we're not going to change this default in the standard
CPython executable (due to the degree of disruption involved), and the -S and
-I switches already effectively turn it off (by disabling site module
processing entirely)
However, it
New submission from Jonathan Slenders:
As discussed on python-ideas, os.pipe should return a structsequence instead of
a plain tuple.
To be decided is the naming for the read and write end.
Personally, I'm in favour of using readfd/writefd.
our_pipe = pipe()
os.write(our_pipe.writefd,
Martin Panter added the comment:
Opened Issue 24541 related to this latest change. The test and documentation
are still inconsistent, even if the test passes.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24400
New submission from Martin Panter:
Revision 0b7c313851ca highlights an inconsistency in the test case at the top
of the TestPredicates class. After removing isawaitable() in Issue 24400, there
are now actually eighteen is* functions. The comment in the test case still
says there are sixteen
Yury Selivanov added the comment:
This has already been fix (see issue24400). I've also updated the comment (16
- 18). Let's keep the test as is.
--
nosy: +yselivanov
resolution: - fixed
stage: - resolved
status: open - closed
___
Python tracker
Roundup Robot added the comment:
New changeset a5c6eaa7d733 by Yury Selivanov in branch '3.5':
Issue #24541: Update comment in test_inspect.test_eightteen
https://hg.python.org/cpython/rev/a5c6eaa7d733
--
nosy: +python-dev
___
Python tracker
Changes by Yury Selivanov yseliva...@gmail.com:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24487
___
___
Kevin Norris added the comment:
If I were designing pathlib from scratch, I would not have a separate Path
class. I would instead do something like this:
In pathlib.py:
if os.name == 'nt':
Path = WindowsPath
else:
Path = PosixPath
Alternatively, Path() could be a
Martin Panter added the comment:
Reopening. The patch still applies to the current code (e.g. revision
df310e5ac015, 30 June). It changes eight references of “async()” that still
exist in this revision.
--
status: closed - open
___
Python tracker
Yury Selivanov added the comment:
Thanks, Martin!
--
resolution: out of date - fixed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24487
___
___
Roundup Robot added the comment:
New changeset 1b3be273e327 by Yury Selivanov in branch '3.5':
Issue #24487: Rename async() - ensure_future() in asyncio docs.
https://hg.python.org/cpython/rev/1b3be273e327
New changeset 3dc2a113e8a7 by Yury Selivanov in branch 'default':
Merge 3.5 (Issue
New submission from Samuel Hoffman:
Realizing the _ssl module does not import very many constants, I think it might
be worth while to reimplement and document SSL_OP_NO_TICKET (0x4000) as
it's another one of the enable this for improved security at a cost options
like SSL_OP_SINGLE_DH_USE
R. David Murray added the comment:
Look like you forgot to adjust test_inspect for the removal. eg:
http://buildbot.python.org/all/builders/AMD64%20Windows8.1%20Non-Debug%203.x/builds/54
--
nosy: +r.david.murray
status: closed - open
___
Python
Yury Selivanov added the comment:
+1 for readfd/writefd. I think 'fd' suffix is necessary to make it explicit
that those aren't file objects.
--
nosy: +yselivanov
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24536
Yury Selivanov added the comment:
Here's a patch, please review.
--
keywords: +patch
stage: - patch review
Added file: http://bugs.python.org/file39839/ospipe.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24536
Changes by Yury Selivanov yseliva...@gmail.com:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24400
___
___
Yury Selivanov added the comment:
Look like you forgot to adjust test_inspect for the removal. eg:
My bad. Thanks, David!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24400
___
Roundup Robot added the comment:
New changeset 0b7c313851ca by Yury Selivanov in branch '3.5':
Issue #24400: Fix failing unittest
https://hg.python.org/cpython/rev/0b7c313851ca
New changeset 8c85291e86bf by Yury Selivanov in branch 'default':
Merge 3.5 (Issue #24400)
Martin Panter added the comment:
Okay but what about the documentation? It still claims sixteen.
# This test is here for remember you to update Doc/library/inspect.rst
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24541
Martin Panter added the comment:
The technical bit of Issue23883_support_check__all__.v3.patch looks pretty
good. Mainly some grammar suggestions for the documentation.
Issue23883_test_gettext.v2.patch looks fine; just depends on check__all__()
being added.
Couple of comments about the APIs
Changes by Ed Maste carpedd...@gmail.com:
--
nosy: +emaste
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24520
___
___
Python-bugs-list mailing
Changes by koobs koobs.free...@gmail.com:
--
keywords: +needs review
nosy: +koobs
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24520
___
___
New submission from Guido van Rossum:
See https://github.com/python/asyncio/issues/251. I'm on vacation and may or
may not find the time to actually fix this (it needs a unittest written).
--
components: asyncio
messages: 245986
nosy: gvanrossum, haypo, yselivanov
priority: normal
Changes by Gerrit Holl topjakl...@gmail.com:
--
type: - crash
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24538
___
___
Python-bugs-list
New submission from Gerrit Holl:
`shutil.copystat` fails on [panfs](https://en.wikipedia.org/wiki/Panasas#PanFS)
if the source file lacks u+w, because setting extended attributes results in a
`PermissionError`. This leads to higher end functions such as
`shutil.copytree` to fail. More
Cyd Haselton added the comment:
Your question about -fPIE brings up a question: How should the differences
between Android 5 and previous versions be handled in regards to this issue?
Other than mandatory -fPIE, there are changes to Android[s libc that may make
patches for python on Android
Gerrit Holl added the comment:
Perhaps the solution would be some kind of flag, at least for copytree and
possibly others, on what to do when attributes cannot be completely copied — a
bit like numpys options to raise, warn, or ignore?
--
___
R. David Murray added the comment:
There are a couple of related open issues. I think there is an stdlib problem
here, but Im not sure what the solution is.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
Cyd Haselton added the comment:
On June 30, 2015 9:20:45 AM CDT, Ryan Gonzalez rep...@bugs.python.org wrote:
Ryan Gonzalez added the comment:
On June 30, 2015 8:14:34 AM CDT, Cyd Haselton rep...@bugs.python.org
wrote:
Cyd Haselton added the comment:
Your question about -fPIE brings up a
Jonathan Slenders added the comment:
Niki Spahiev made a valid argument saying that the following code is common:
if not hasattr(src, 'read'): src = open(src)
This will break if we name it 'read'/'write' like the methods of a file object.
--
___
Ryan Gonzalez added the comment:
On June 30, 2015 8:14:34 AM CDT, Cyd Haselton rep...@bugs.python.org wrote:
Cyd Haselton added the comment:
Your question about -fPIE brings up a question: How should the
differences between Android 5 and previous versions be handled in
regards to this issue?
Matthias Bussonnier added the comment:
Note that I've signed the CLA, and it has been taken into account, as now I
have a small star agains my name (if it was the limiting factor, we never know).
--
___
Python tracker rep...@bugs.python.org
Ethan Furman added the comment:
'read'/'write' is sufficient.
+1 for the proposal.
--
nosy: +ethan.furman
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24536
___
Roundup Robot added the comment:
New changeset 30b676e8b21f by Yury Selivanov in branch '3.5':
Issue #24528: Improve error message for awaits in comprehensions
https://hg.python.org/cpython/rev/30b676e8b21f
New changeset 9598bc916186 by Yury Selivanov in branch 'default':
Merge 3.5 (Issue
Changes by Yury Selivanov yseliva...@gmail.com:
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24528
___
Marc-Andre Lemburg added the comment:
On 30.06.2015 20:52, Min RK wrote:
Thanks for the feedback, I thought it might be a long shot. I will go back to
removing the *use* of the feature everywhere I can find it, since it is so
problematic and rarely, if ever, desirable.
Could you please
Min RK added the comment:
Could you please post an example of where the feature is problematic ?
setuptools/easy_install is the major one, which effectively does `sys.path[:0]
= pth_contents`, breaking import priority. This has been known to result in
adding
Min RK added the comment:
Thanks for the feedback, I thought it might be a long shot. I will go back to
removing the *use* of the feature everywhere I can find it, since it is so
problematic and rarely, if ever, desirable.
it's an essential feature that has been documented for a very long
New submission from Matthew Havard:
The documentation from json.dumps says this about skipkeys:
If ``skipkeys`` is false then ``dict`` keys that are not basic types
(``str``, ``int``, ``float``, ``bool``, ``None``) will be skipped
instead of raising a ``TypeError``.
However, that
STINNER Victor added the comment:
Le lundi 29 juin 2015, Yury Selivanov rep...@bugs.python.org a écrit :
You should develop using asyncio debug mode:
Maybe we should promote this check to the production mode?
asyncio must be slow. The check has a cost, I prefer to keep it only in
debug
Lars Gustäbel added the comment:
Martin, I followed your suggestion to raise ReadError. This needed an
additional change in copyfileobj() because it is used both for adding file data
to an archive and extracting file data from an archive.
But I think the patch is in good shape now.
New submission from Dana Christen:
I'm using the C API to embed the Python interpreter (see the attached example).
Everything works fine until I try to run the resulting executable on a machine
without a Python installation. In that case, the call to Py_Initialize fails
with the following
Yury Selivanov added the comment:
isawaitable(), however, should continue using abc.Awaitable, since it only
checks for __await__ presence on the type (or should we just drop it?)
I'd really remove it. It's not referring to an actual type, so it doesn't fit
the purpose of the inspect
Min RK added the comment:
Just because a feature can be misused doesn't make it a bad feature.
That's fair. I'm just not aware of any uses of this feature that aren't
misuses, hence the patch.
Perhaps you could submit a fix for this to the setuptools maintainers instead.
Yes, that's
On 01.07.2015 00:16, Min RK wrote:
Just because a feature can be misused doesn't make it a bad feature.
That's fair. I'm just not aware of any uses of this feature that aren't
misuses, hence the patch.
I don't remember the details of why this feature was added,
but can imagine that it was
Nick Levinson added the comment:
Thank you. I didn't know enough to understand the relevance of blueman.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24535
___
Marc-Andre Lemburg added the comment:
On 30.06.2015 22:49, Min RK wrote:
Could you please post an example of where the feature is problematic ?
setuptools/easy_install is the major one, which effectively does
`sys.path[:0] = pth_contents`, breaking import priority. This has been known
Ben Darnell added the comment:
Yes, I can switch use the ABC instead, and I agree that it doesn't make sense
to have the inspect method if it's going to be equivalent to the ABC.
I'm happy with the outcome here but AFAIK the original issue still stands: the
Awaitable ABC is unusual in that
Roundup Robot added the comment:
New changeset e20c197f19d6 by Yury Selivanov in branch '3.5':
Issue #24400: Remove inspect.isawaitable().
https://hg.python.org/cpython/rev/e20c197f19d6
New changeset 800bf6a0e0d5 by Yury Selivanov in branch 'default':
Merge 3.5 (Issue #24400)
51 matches
Mail list logo