Change by Guido van Rossum :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
PEP 584 has been approved by the Steering Council (at my recommendation). We
will shortly begin landing PRs related to this.
--
versions: +Python 3.9 -Python 3.8
___
Python tracker
<https://bugs.python.
Guido van Rossum added the comment:
New changeset eb8ac57af26c4eb96a8230eba7492ce5ceef7886 by Brandt Bucher in
branch 'master':
bpo-36144: Dictionary Union (PEP 584) (#12088)
https://github.com/python/cpython/commit/eb8ac57af26c4eb96a8230eba7492c
Guido van Rossum added the comment:
While the main code has been merged now, I propose to keep this issue open
until some other things have happened:
- Documentation
- Add | operators to some dict subclasses in the stdlib
- (What else
Guido van Rossum added the comment:
Markus, your comment does not seem relevant to this issue. Maybe you meant to
add it to a different issue, or you meant to create a new issue?
--
___
Python tracker
<https://bugs.python.org/issue31
Guido van Rossum added the comment:
Yup, great plan.
On Mon, Feb 24, 2020 at 22:29 Brandt Bucher wrote:
>
> Brandt Bucher added the comment:
>
> My current PR plans are:
>
> - Docs. This will include the dict docs and the whatsnew 3.9. I assume we
> have no plans
Change by Guido van Rossum :
--
nosy: +gvanrossum, pitrou
___
Python tracker
<https://bugs.python.org/issue39763>
___
___
Python-bugs-list mailing list
Unsub
Guido van Rossum added the comment:
New changeset d0ca9bd93bb9d8d4aa9bbe939ca7fd54ac870c8f by Brandt Bucher in
branch 'master':
bpo-36144: Document PEP 584 (GH-18659)
https://github.com/python/cpython/commit/d0ca9bd93bb9d8d4aa9bbe939ca7fd
Guido van Rossum added the comment:
@Brandt: you have some more followup PRs planned right? Let's keep this issue
open until you've done all of those.
--
___
Python tracker
<https://bugs.python.o
Guido van Rossum added the comment:
@Antoine: I can sort of see why you consider the small example a red herring,
since it mixes threads and fork. Nevertheless, a simpler version (not using an
executor) would probably work just fine? It's the edge cases where the mixture
becomes expl
Guido van Rossum added the comment:
When I run this on macOS, either with the bug, or with the buggy line commented
out, I get the same hang. When I interrupt it, I get
a separate traceback from each thread. Here's the full session:
Parent 78918
Parent 78919
Child 78918
Parent 78920
Guido van Rossum added the comment:
I think for `|=` the only choice is for it to be essentially an alias to
`.update()`. So that means `cm |= other` becomes `cm.maps[0].update(other)`.
For `|` we are breaking new ground and we could indeed make `cm | other` do
something like `ChainMap
Guido van Rossum added the comment:
I had just come to a different conclusion. Maybe ChainMap should just not
grow `|` and `|=` operators? That way there can be no confusion. `dict() |
ChainMap()` and `ChainMap() | dict()` will fail because ChainMap doesn't
inherit from dict. (Note th
Guido van Rossum added the comment:
OK, assuming `|=` gets the same semantics as update(), can you repeat once
more (without motivation) what the specification for `cm | other` will be?
--
___
Python tracker
<https://bugs.python.org/issue36
Guido van Rossum added the comment:
The key is that os.execl() never returns.
My understanding is that the hang happens when the child code returns (or
raises) in the forked child process, because the semaphore state is cloned by
the fork.
There is no logging here. I just added some print
Guido van Rossum added the comment:
OK, that makes sense, it works similar to ChainMap.copy(), which copies maps[0]
and keeps links to the rest. So in particular `cm | {}` will do the same thing
as cm.copy().
Im not sure if the dict(other) cast is the best way to go about it. Maybe this
Guido van Rossum added the comment:
I didn't see your second reply, with `ChainMap(other, *cm.maps)`.
I'm not so keen on that, because its special behavior can't be mimicked by `|=`.
--
___
Python tracker
<https://bugs.pyt
Guido van Rossum added the comment:
And you're sure that your "toy" example hangs in the at-fork handler.
In that case I can't help you further, I don't have the resources or time to
try out QNX. Your best bet is indeed to change distutils.spawn to not use
fork+exe
Guido van Rossum added the comment:
We already have somewhat different semantics of `|` for Counter, and hence I
think it's fine to give it the most useful semantics for ChainMap given that
class's special behavior. I think we've come up with the right solution there.
Let
Guido van Rossum added the comment:
As I mentioned in my response on the PR, dict *is* required to be ordered since
3.7. So I think it is okay as is.
--
___
Python tracker
<https://bugs.python.org/issue17
Change by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<https://bugs.python.org/issue17422>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
This is as intended.
--
resolution: -> not a bug
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
I’m a bit disappointed, since it looks like this won’t allow implementing the
OP’s classes without using private APIs. The debugging and loggin use cases
aren’t very compelling to me.
--
___
Python tracker
Guido van Rossum added the comment:
But note my response to Antoine at the time, mentioning that implementing
‘as_completed()’ is impossible that way. Antoine then backtracked somewhat.
--
___
Python tracker
<https://bugs.python.org/issue39
Change by Guido van Rossum :
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue39704>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
New changeset be501ca2419a91546dea85ef4f36945545458589 by Brandt Bucher in
branch 'master':
bpo-39702: Relax grammar restrictions on decorators (PEP 614) (GH-18570)
https://github.com/python/cpython/commit/be501ca2419a91546dea85ef4f3694
Guido van Rossum added the comment:
I'm guessing there's some doc update that needs to happen now the code has
landed.
--
___
Python tracker
<https://bugs.python.o
Guido van Rossum added the comment:
Sounds good. If you feel like it you can submit a PR to the devguide, I'm sure
Brett will merge it quickly...
--
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
It should definitely not return {} for objects without __annotations__
attributes -- get_type_hints(42) should raise TypeError.
One could argue that get_type_hints(Bar()) incorrectly returns a dict, but we
want to allow for some duck typing here, and
Guido van Rossum added the comment:
New changeset 8bae21962bab2fac7630982abd73676b89930902 by Brandt Bucher in
branch 'master':
bpo-39868: Update Language Reference for PEP 572. (#18793)
https://github.com/python/cpython/commit/8bae21962bab2fac7630982abd73676b89930902
-
Guido van Rossum added the comment:
New changeset 57c9d1725689dde068a7fccaa7500772ecd16d2e by Brandt Bucher in
branch 'master':
bpo-36144: Implement defaultdict union (GH-18729)
https://github.com/python/cpython/commit/57c9d1725689dde068a7fccaa75007
Guido van Rossum added the comment:
Still waiting for ChainMap -- what else?
--
___
Python tracker
<https://bugs.python.org/issue36144>
___
___
Python-bug
Guido van Rossum added the comment:
Haven’t looked at the code but I welcome the simplification.
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue34
Guido van Rossum added the comment:
New changeset 8f130536926a30237b5297780d61ef4232e88577 by Brandt Bucher in
branch 'master':
bpo-39702: Update the Language Reference (PEP 614) (GH-18802)
https://github.com/python/cpython/commit/8f130536926a30237b5297780d61ef
Guido van Rossum added the comment:
New changeset 4663f66f3554dd8e2ec130e40f6abb3c6a514775 by Brandt Bucher in
branch 'master':
bpo-36144: Update MappingProxyType with PEP 584's operators (#18814)
https://github.com/python/cpython/commit/4663f66f3554dd8e2ec130e40
Guido van Rossum added the comment:
I have a nit on the commit message used here (and also in issue 39885).
*Please* don't use this style of commit message "IDLE context menu clears
selection". That phrasing sounds like it is the description of a bug. Please
always use a phra
Guido van Rossum added the comment:
I guess it's similar to the 'return' vs. 'returns' issue, but I feel much
stronger about it here.
I would have written that as "make the spam module more spammy". Just read
a bunch of commits using e.g. `git log --onelin
Change by Guido van Rossum :
--
versions: -Python 3.8
___
Python tracker
<https://bugs.python.org/issue12782>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
If we introduce a PEG-based parser, we can do this without hacking the
tokenizer. See https://github.com/gvanrossum/pegen/issues/229
I'd propose to aim for Python 3.10 (if the PEG parser happens).
--
nosy: +gvanr
Guido van Rossum added the comment:
I'm going to review the actual code next.
Regarding the omission of parentheses in various contexts, I am all for that,
but I consider it a separate issue (as it only pertains to ast.unparse()). The
fix in https://github.com/python/cpython/pull/
Change by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<https://bugs.python.org/issue39763>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
I propose to close this issue, since (as of Python 3.8) we now have
ast.parse(source, type_comments=True).
--
___
Python tracker
<https://bugs.python.org/issue24
Guido van Rossum added the comment:
If people are looking for a concrete CST that works now, maybe LibCST will
work? https://github.com/Instagram/LibCST
--
___
Python tracker
<https://bugs.python.org/issue33
Guido van Rossum added the comment:
New changeset d648ef10c5c7659ed3c9f34d5c751dc55e2c6007 by Charles Burkland in
branch 'master':
bpo-36144: Update os.environ and os.environb for PEP 584 (#18911)
https://github.com/python/cpython/commit/d648ef10c5c7659ed3c9f34d5c751d
Guido van Rossum added the comment:
New changeset 6d674a1bf456945eb758e85c11484a9f1494f2b4 by Brandt Bucher in
branch 'master':
bpo-36144: OrderedDict Union (PEP 584) (#18967)
https://github.com/python/cpython/commit/6d674a1bf456945eb758e85c11484a
Guido van Rossum added the comment:
I definitely think we should leave Shelf alone, it's a toy class from a
different era.
It makes sense to update the weak dicts; hopefully the | and |= operators can
be implemented in terms of other, more primitive operations, so we will have
assu
Guido van Rossum added the comment:
I stopped following the discussion at some point, but I think this is worth
adding it -- I have seen this done over and over again, and apparently lots of
other people have felt the need too.
I think these names are fine, and about the best we can do
Guido van Rossum added the comment:
That's why for over a decade we've been recommending not to use string
comparisons to compare versions. You have to parse the version and then compare
the numeric values.
--
resolution: -> not a bug
stage: -> resolved
status
Guido van Rossum added the comment:
That's a question for a user forum. There's some code in
Lib/distutils/version.py.
--
___
Python tracker
<https://bugs.python.o
Guido van Rossum added the comment:
Sounds good.
--
___
Python tracker
<https://bugs.python.org/issue39939>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
New changeset f393b2c588559162dc2e77f8079a42e48558870a by Curtis Bucher in
branch 'master':
bpo-36144: Add PEP 584 operators to collections.ChainMap (#18832)
https://github.com/python/cpython/commit/f393b2c588559162dc2e77f8079a42
Guido van Rossum added the comment:
New changeset 25e580a73c163f472fdeb5489bebef85da21655c by Curtis Bucher in
branch 'master':
bpo-36144: Add union operators to WeakKeyDictionary (#19106)
https://github.com/python/cpython/commit/25e580a73c163f472fdeb5489bebef
Guido van Rossum added the comment:
New changeset 8f1ed21ecf57cc8b8095d9d1058af2b9b3ed0413 by Curtis Bucher in
branch 'master':
bpo-36144: Add union operators to WeakValueDictionary584 (#19127)
https://github.com/python/cpython/commit/8f1ed21ecf57cc8b8095d9d1058af2
Guido van Rossum added the comment:
I'm guessing this can be closed?
--
stage: resolved -> patch review
___
Python tracker
<https://bugs.python.org
Guido van Rossum added the comment:
I think you're going too far for some of these.
> 1. metaclasses
This is reasonable.
> 2. __mro__
This is also reasonable. (I wonder if that part of the dir() implementation
predates __mro__?) I'm not sure about honoring mro() in the m
Guido van Rossum added the comment:
FWIW it might be a good idea to look into how PEP 585 could benefit from the
improvements to dir(). Currently, dir(list) and dir(list[int]) are quite
different -- only the former shows list methods like append and insert.
See https://github.com/gvanrossum
Guido van Rossum added the comment:
I guess we should keep this open until Raymond Hettinger has given feedback on
https://github.com/python/cpython/pull/18832 (where we have the option of
changing to Brandt's proposal from
https://github.com/python/cpython/pull/18832#issuecomment-5969
Guido van Rossum added the comment:
The SC approved the PEP!
Łukasz, do you want to submit the PR to update the PEP status?
We should get the implementation reviewed (e.g. by Serhiy) and land it before
alpha 6 goes out, April 22 (we should aim for a week before at least
Guido van Rossum added the comment:
> >>> t = list[int]
>
> A few years ago, I tried to explain to you that it's much more intuitive
> to use builtin types instead of their alter egos from typing (List etc.)
> for [] operator, you said it's not important. I
Guido van Rossum added the comment:
For me, one of the most annoying things about dir() is that it gives all the
dunders. There are many dunders that are just always there (__class__, mostly
__dict__, __doc__, __name__ etc.). I wish it would just not give dunders that
are inherited from
Guido van Rossum added the comment:
Vedran, please stay on topic for this issue.
FWIW I agree that it would be best if dir() showed only those dunders that are
significant. E.g. __eq__ should only be shown if it is overridden by a subclass.
Serhiy did you see my feedback on 3 and 4
Guido van Rossum added the comment:
At least for overriding __dict__, I would presume there would be a matching
override for __getattribute__ (not __getattr__).
Ditto for __class__, e.g. look at GenericAlias in the pep585 implementation PR.
(Here __class__ is not in the list of exceptions
Change by Guido van Rossum :
--
resolution: -> fixed
stage: needs patch -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Guido van Rossum :
--
nosy: -gvanrossum
___
Python tracker
<https://bugs.python.org/issue39704>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
New changeset 48b069a003ba6c684a9ba78493fbbec5e89f10b8 by Guido van Rossum in
branch 'master':
bpo-39481: Implementation for PEP 585 (#18239)
https://github.com/python/cpython/commit/48b069a003ba6c684a9ba78493fbbe
Guido van Rossum added the comment:
The base implementation has landed. We still need docs, and I'm sure that the
alpha and beta release cycle will find small things that need to be improved.
Perhaps the next priority is an update for Doc/whatsnew/3.
Guido van Rossum added the comment:
Hold on, os.DirEntry[str] still doesn't work.
--
___
Python tracker
<https://bugs.python.org/issue39019>
___
___
Pytho
Guido van Rossum added the comment:
Ethan Smith produced a list of types that are Generic in typeshed but not in
the stdlib. So these could be added.
https://github.com/gvanrossum/cpython/pull/1#issuecomment-582781121
--
___
Python tracker
Guido van Rossum added the comment:
New changeset f9dd51e7db27d04e0b716d41a2804d5acbf145d1 by Batuhan Taşkaya in
branch 'master':
bpo-39481: Make os.DirEntry generic (GH-19415)
https://github.com/python/cpython/commit/f9dd51e7db27d04e0b716d41a2804d
Guido van Rossum added the comment:
New changeset 2fa67df605e4b0803e7e3aac0b85d851b4b4e09a by Batuhan Taşkaya in
branch 'master':
bpo-39481: PEP 585 for ipaddress.py (GH-19418)
https://github.com/python/cpython/commit/2fa67df605e4b0803e7e3aac0b85d8
Guido van Rossum added the comment:
New changeset 7c4185d62d4aec486d82c3ad02acd878db2d3537 by Ethan Smith in branch
'master':
bpo-39481: PEP 585 for enumerate, AsyncGeneratorType, mmap (GH-19421)
https://github.com/python/cpython/commit/7c4185d62d4aec486d82c3ad02acd8
Guido van Rossum added the comment:
At this speed I can merge about 3 PRs per hour. I'll be back tomorrow.
--
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
New changeset e3ec44d692d9442e640cf5b2d8708157a65cec3e by Ethan Smith in branch
'master':
bpo-39481: PEP 585 for difflib, filecmp, fileinput (#19422)
https://github.com/python/cpython/commit/e3ec44d692d9442e640cf5b2d87081
Guido van Rossum added the comment:
Check typeshed
--
___
Python tracker
<https://bugs.python.org/issue39481>
___
___
Python-bugs-list mailing list
Unsubscribe:
Guido van Rossum added the comment:
New changeset 0361556537686f857f1025ead75e6af4ca7cc94a by Batuhan Taşkaya in
branch 'master':
bpo-39481: PEP 585 for a variety of modules (GH-19423)
https://github.com/python/cpython/commit/0361556537686f857f1025ead75e6a
Guido van Rossum added the comment:
So a slightly shorter example uses ru''. This is an error because you can't
combine the r prefix and the u prefix (in fact you can't combine anything with
the r prefix).
I declare that this is a bug to report EOF here and the code s
Guido van Rossum added the comment:
Yeah, this seems safe to remove. If somebody's code breaks, that will just help
them remove some dead code.
--
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
Let's give up on this one.
--
resolution: -> wont fix
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.pyth
Guido van Rossum added the comment:
But that is the tokenize.py module. It does not have 100% the same behavior as
the builtin tokenizer.c module. In this case it probably doesn't.
--
___
Python tracker
<https://bugs.python.org/is
Guido van Rossum added the comment:
That may well be correct. Do you want to submit a PR to fix it?
--
___
Python tracker
<https://bugs.python.org/issue25
Guido van Rossum added the comment:
yes
On Sat, Apr 11, 2020 at 04:46 Lysandros Nikolaou
wrote:
>
> Lysandros Nikolaou added the comment:
>
> I have working code that checks if there is a quotation mark right after
> an identifier. Here is an example:
>
> ╰─ ./py
Guido van Rossum added the comment:
New changeset 25a6833f7945f14cad83509ec73954d0ad70bdb1 by Chih-Hsuan Yen in
branch 'master':
bpo-39481: fix test_genericalias on Android (GH-19469)
https://github.com/python/cpython/commit/25a6833f7945f14cad83509ec73954
Guido van Rossum added the comment:
New changeset cecf049673da6a24435acd1a6a3b34472b323c97 by Ethan Smith in branch
'master':
bpo-39481: Make functools.cached_property, partial, partialmethod generic
(#19427)
https://github.com/python/cpython/commit/cecf049673da6a24435acd1a6a3b34
Guido van Rossum added the comment:
New changeset 8ef875028a3644a329c87ce420a73793e315143f by Ethan Smith in branch
'master':
bpo-39481: Make weakref and WeakSet generic (GH-19497)
https://github.com/python/cpython/commit/8ef875028a3644a329c87ce420a737
Guido van Rossum added the comment:
New changeset d01628e411752ee6849f862cae66a1c69fe512b7 by Ethan Smith in branch
'master':
bpo-39481: PEP 585 for dataclasses, mailbox, contextvars (GH-19425)
https://github.com/python/cpython/commit/d01628e411752ee6849f862cae66a1
Guido van Rossum added the comment:
Thanks!
--
--Guido (mobile)
--
___
Python tracker
<https://bugs.python.org/issue39481>
___
___
Python-bugs-list mailin
Guido van Rossum added the comment:
New changeset 9a4b38f66b3e674db94e07980e1cacb39e388c73 by Lysandros Nikolaou in
branch 'master':
bpo-40267: Fix message when last input character produces a SyntaxError
(GH-19521)
https://github.com/python/cpyt
Change by Guido van Rossum :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
New changeset ba1bcffe5cafc1bb0ac6fdf9ecef51e75e342707 by Russell Davis in
branch 'master':
bpo-29255: Wait in KqueueSelector.select when no fds are registered (GH-19508)
https://github.com/python/cpython/commit/ba1bcffe5cafc1bb0ac6fdf9ecef51
Guido van Rossum added the comment:
New changeset ba1bcffe5cafc1bb0ac6fdf9ecef51e75e342707 by Russell Davis in
branch 'master':
bpo-29255: Wait in KqueueSelector.select when no fds are registered (GH-19508)
https://github.com/python/cpython/commit/ba1bcffe5cafc1bb0ac6fdf9ecef51
Change by Guido van Rossum :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Guido van Rossum added the comment:
How ironic, the other issue had to be closed manually. :-)
Reopening this one.
--
resolution: fixed ->
status: closed -> open
___
Python tracker
<https://bugs.python.org/i
Guido van Rossum added the comment:
Yeah, I think help() or pydoc needs to special-case this. (Didn't your other PR
attempt to fix this?)
Note that issubclass(list[int].__class__, type) returns True -- the __class__
attribute in this case is taken from __origin__, while type() return
New submission from Guido van Rossum :
Note: PEP 617 is currently under review by the Steering Council, but if they
approve we'd like to get it into alpha 6, and reviews are welcome (even though
we're still finagling some corner cases of the implementation).
--
messages: 3
Guido van Rossum added the comment:
I am in favor of this. We could promote LibCST, which is based on Parso, which
uses a forked version of pgen2 (the parser in lib2to3). I believe one of these
could switch to a fork of pegen as its parser, so it will be able to handle new
PEG based syntax
Guido van Rossum added the comment:
VIctor, you were very right about longjmp. See
https://github.com/we-like-parsers/cpython/pull/119
--
___
Python tracker
<https://bugs.python.org/issue40
Change by Guido van Rossum :
--
pull_requests: +19012
pull_request: https://github.com/python/cpython/pull/19692
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
New changeset bc28805570ae3e8835a5d502ae9ab15c52449f77 by Guido van Rossum in
branch 'master':
bpo-40334: Use old compiler when compile mode is func_type (GH-19692)
https://github.com/python/cpython/commit/bc28805570ae3e8835a5d502ae9ab1
Change by Guido van Rossum :
--
pull_requests: +19015
pull_request: https://github.com/python/cpython/pull/19695
___
Python tracker
<https://bugs.python.org/issue40
Guido van Rossum added the comment:
Yeah that's why we have a PR out to delete or skip this test. Pablo and
Lysandros will deal with it tomorrow.
On Thu, Apr 23, 2020 at 19:33 STINNER Victor wrote:
>
> STINNER Victor added the comment:
>
> test_peg_generator leaks refere
2401 - 2500 of 5533 matches
Mail list logo