Gregory P. Smith added the comment:
any idea if this is still relevant?
--
components: +Cross-Build
nosy: +Alex.Willmer, gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue20
Gregory P. Smith added the comment:
The way I'd go about this is to support some form of periodic checkpoint in the
algorithm where it checks in with code supplied by the difflib user.
Traditionally that'd have been via callbacks, there might be an async style way
to express that these days
Gregory P. Smith added the comment:
The merged PRs disable the test without anything tracking fixing or
re-re-enabling it. Reopening & retitling this issue.
This specific test does not need to use an external server. That has nothing to
do with what the purposes of the test is which is
Gregory P. Smith added the comment:
What do other APIs in widely used languages do with regex terminology? We
appear to be the only popular language who anchors to the start of a string
with an API named "match".
libpcre C: uses "match" to mean what we call "search
Change by Gregory P. Smith :
--
keywords: +patch
pull_requests: +29314
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/31137
___
Python tracker
<https://bugs.python.org/issu
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
versions: +Python 3.11 -Python 3.10
___
Python tracker
<https://bugs.python.org/issue42353>
___
_
Change by Gregory P. Smith :
--
nosy: +mark.dickinson
___
Python tracker
<https://bugs.python.org/issue46639>
___
___
Python-bugs-list mailing list
Unsubscribe:
Gregory P. Smith added the comment:
lib2to3 is going away per https://bugs.python.org/issue40360
but it is behaving as intended above, it does not understand logic. it was
only ever intended to maybe work on python 2-only code. not python 2-and-3
compatible code.
--
nosy
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
nosy: +gregory.p.smith
versions: +Python 3.11 -Python 2.7, Python 3.6
___
Python tracker
<https://bugs.python.org/issu
Gregory P. Smith added the comment:
Thanks to Kumar for contributing Windows compiler flags side of this (the point
of this issue).
--
resolution: -> fixed
stage: patch review -> commit review
status: open -> closed
___
Python tracke
Gregory P. Smith added the comment:
test_peg_generator is significantly less of the long tail on optimized builds
now. CI extremely noisy performance wise (times vary by 2-3x without any
differences anways) so I can't easily judge if this made a notable difference
in windows CI latency
Gregory P. Smith added the comment:
for my purposes those PRs make it reasonable, even though it remains one of our
slowest tests. we can reopen/reuse this issue for any further
test_peg_generator performance improvements as needed.
--
assignee: -> gregory.p.smith
resolut
Gregory P. Smith added the comment:
New changeset 8726067ace98a27557e9fdf1a8e1c509c37cfcfc by Gregory P. Smith in
branch 'main':
bpo-45629: Improve test.support.skip_if_buildbot (GH-31094)
https://github.com/python/cpython/commit/8726067ace98a27557e9fdf1a8e1c509c37cfcfc
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
nosy_count: 2.0 -> 3.0
pull_requests: +29280
pull_request: https://github.com/python/cpython/pull/31094
___
Python tracker
<https://bugs.python.org/issu
Gregory P. Smith added the comment:
New changeset f5ebec4d3e1199ec38b88920cfde8e460e5120dd by Gregory P. Smith in
branch '3.10':
[3.10] bpo-46576: bpo-46524: Disable compiler optimization within
test_peg_generator. (GH-31015) (GH-31089)
https://github.com/python/cpython/commit
Gregory P. Smith added the comment:
New changeset f5ebec4d3e1199ec38b88920cfde8e460e5120dd by Gregory P. Smith in
branch '3.10':
[3.10] bpo-46576: bpo-46524: Disable compiler optimization within
test_peg_generator. (GH-31015) (GH-31089)
https://github.com/python/cpython/commit
Change by Gregory P. Smith :
--
pull_requests: +29273
pull_request: https://github.com/python/cpython/pull/31089
___
Python tracker
<https://bugs.python.org/issue46
Change by Gregory P. Smith :
--
pull_requests: +29274
pull_request: https://github.com/python/cpython/pull/31089
___
Python tracker
<https://bugs.python.org/issue46
Gregory P. Smith added the comment:
New changeset 164a017e13ee96bd1ea1ae79f5ac9e25fe83994e by Gregory P. Smith in
branch 'main':
bpo-46576: bpo-46524: Disable compiler optimization within test_peg_generator.
(#31015)
https://github.com/python/cpython/commit
Gregory P. Smith added the comment:
New changeset 164a017e13ee96bd1ea1ae79f5ac9e25fe83994e by Gregory P. Smith in
branch 'main':
bpo-46576: bpo-46524: Disable compiler optimization within test_peg_generator.
(#31015)
https://github.com/python/cpython/commit
Gregory P. Smith added the comment:
New changeset e8659b47dece5a272111c0af5e340c364a9f807b by Hugo van Kemenade in
branch 'main':
bpo-45173: Keep configparser deprecations until Python 3.12 (GH-30952)
https://github.com/python/cpython/commit/e8659b47dece5a272111c0af5e340c364a9f807b
Change by Gregory P. Smith :
--
pull_requests: +29197
pull_request: https://github.com/python/cpython/pull/31015
___
Python tracker
<https://bugs.python.org/issue46
Gregory P. Smith added the comment:
If a decent parallelism CI systems are not available from github (they seem
stuck at 2-3 threads per
https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners),
an alternative approach could be to shard across multiple
Gregory P. Smith added the comment:
re: slow tests in the first half of the list. the same total amount of time is
going to be spent regardless. In our test suite on a modern fast 16 thread
system, all but 10 tests are completed in parallel within the first 30 seconds.
The remaining ~10
Change by Gregory P. Smith :
--
keywords: +patch
pull_requests: +29194
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/31015
___
Python tracker
<https://bugs.python.org/issu
New submission from Gregory P. Smith :
test_peg_generator is an extremely slow test. This bug tracks any changes to
reduce its runtime and test latency.
--
components: Tests
messages: 412104
nosy: gregory.p.smith
priority: normal
severity: normal
status: open
title
Gregory P. Smith added the comment:
I manually tested this on 3.11 main and it appears to be working.
--
resolution: -> works for me
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Gregory P. Smith :
--
nosy: +ned.deily
___
Python tracker
<https://bugs.python.org/issue46548>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Gregory P. Smith :
--
pull_requests: +29145
pull_request: https://github.com/python/cpython/pull/30966
___
Python tracker
<https://bugs.python.org/issue42
Change by Gregory P. Smith :
--
keywords: +3.11regression
nosy: +gregory.p.smith
priority: normal -> high
___
Python tracker
<https://bugs.python.org/issu
Gregory P. Smith added the comment:
I reworked the PR and went with less specific text and linking to the NIST 800
132 appendix as guidance on how people should determine what is right for them.
there is no one right number. it is application specific.
thanks for everyone's valuable input
Gregory P. Smith added the comment:
New changeset 897ce9018775bcd679fb49aa17258f8f6e818e23 by Illia Volochii in
branch 'main':
bpo-42982: Improve the text on suggested number of iterations of PBKDF2
(GH-24276)
https://github.com/python/cpython/commit/897ce9018775bcd679fb49aa17258f8f6e818e23
Change by Gregory P. Smith :
--
assignee: gregory.p.smith ->
___
Python tracker
<https://bugs.python.org/issue45162>
___
___
Python-bugs-list mailing list
Un
Gregory P. Smith added the comment:
New changeset b50322d20337ca468f2070eedb051a16ee1eba94 by Gregory P. Smith in
branch 'main':
bpo-45162: Revert "Remove many old deprecated unittest features" (GH-30935)
https://github.com/python/cpython/commit/b50322d20337ca468f2070eedb051a
Change by Gregory P. Smith :
--
pull_requests: +29114
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/30935
___
Python tracker
<https://bugs.python.org/issu
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue45162>
___
___
Python-bugs-list mailing list
Un
Gregory P. Smith added the comment:
Per
https://mail.python.org/archives/list/python-...@python.org/thread/GJTREADEXYAETECE5JDTPYWK4WMTKYGR/
we want to revert
https://github.com/python/cpython/commit/1fc41ae8709e20d741bd86c2345173688a5e84b0
for 3.11 to avoid causing pain.
per that thread
Gregory P. Smith added the comment:
Per
https://mail.python.org/archives/list/python-...@python.org/thread/GJTREADEXYAETECE5JDTPYWK4WMTKYGR/
we want to revert this change for 3.11 as it causes adoption pain for little
benefit.
To determine if we should proceed with this in 3.12 or future
Gregory P. Smith added the comment:
As this is a new feature, it would also be reasonable to have specifying
max_tasks_per_child without explicitly specifying a mp_context default to a
safe mp_context.
--
___
Python tracker
<ht
Gregory P. Smith added the comment:
from discussion on the other bug it looks like we should have a way to keep
this; we just need to not allow it when the mp_context to be used is the 'fork'
one.
--
___
Python tracker
<https://bugs.python.
Gregory P. Smith added the comment:
and similarly, the dynamic spawning could be kept when the mp_context is spawn
(and possibly forkserver).
--
___
Python tracker
<https://bugs.python.org/issue46
Gregory P. Smith added the comment:
It sounds like we need to introspect the mp_context= passed to
ProcessPoolExecutor (and it's default when None) to raise an error when
max_tasks_per_child is incompatible with it.
--
___
Python tracker
<ht
Gregory P. Smith added the comment:
while figuring this issue out, it may also make sense to address
https://bugs.python.org/issue44723 as well.
--
___
Python tracker
<https://bugs.python.org/issue46
Gregory P. Smith added the comment:
ablack: the basic auth username:password@ part of the string is not part of a
hostname. What code are you seeing that is trying to send that to a name
resolver rather than stripping the obviously private info up through the @ sign?
--
nosy
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue44723>
___
___
Python-bugs-list mailing list
Unsubscribe:
Gregory P. Smith added the comment:
note that Bodo's own followup issue about the breaking change for python-iconv
was filed as https://bugs.python.org/issue44723
--
___
Python tracker
<https://bugs.python.org/issue37
Gregory P. Smith added the comment:
(note: this might not be the true cause of that issue; though it sounds
potentially related - I haven't investigated far enough yet)
--
___
Python tracker
<https://bugs.python.org/issue37
Gregory P. Smith added the comment:
https://bugs.python.org/issue46508 filed to track fixing the acceptance and use
of garbage codec values regression that this caused.
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.
New submission from Gregory P. Smith :
in 3.8 this was not a valid codec name: "เ_เ_เ_iDnA"
in 3.9 it gets treated as idna and triggers the punycode decoder when passed to
bytes.decode(codec).
Discovered by oss-fuzz.
_Likely_ a consequence of https://bugs.python.org/
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue11322>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Gregory P. Smith :
--
assignee: -> pablogsal
___
Python tracker
<https://bugs.python.org/issue46503>
___
___
Python-bugs-list mailing list
Un
New submission from Gregory P. Smith :
```
>>> v = r"""f'\N '"""
>>> import ast
>>> ast.literal_eval(v)
python: ../gpshead/Parser/string_parser.c:487: fstring_find_literal: Assertion
`s == end || *s == '{' || *s == '}'' failed.
Ab
Change by Gregory P. Smith :
--
keywords: +patch
pull_requests: +29029
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/30847
___
Python tracker
<https://bugs.python.org/issu
Gregory P. Smith added the comment:
https://bugs.python.org/issue44733 for 3.11 attempts to build upon the dynamic
spawning ability and will need reverting unless a non-thread+fork
implementation is provided.
--
___
Python tracker
<ht
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
resolution: fixed ->
___
Python tracker
<https://bugs.python.org/issue44733>
___
___
Pyth
Gregory P. Smith added the comment:
If we want this feature to stay in, it needs an implementation that does not
rely on a multithreaded processing calling fork(). I'm going to have to revert
the existing implementation for bpo-46464.
--
stage: resolved -> needs patch
status: clo
Gregory P. Smith added the comment:
[bpo-46464](https://bugs.python.org/issue46464) describes a deadlock that the
pre-requisite for this feature causes.
Spawning new children directly via fork() is impossible once a thread in the
parent process has been started
Change by Gregory P. Smith :
--
title: Cross-compiling Python -> Cross-compiling CPython - need a buildbot
versions: +Python 3.11 -Python 3.4
___
Python tracker
<https://bugs.python.org/iss
New submission from Gregory P. Smith :
It'd be handy to have a function to determine if there are multiple threads in
the current process or not - at least on POSIXish systems. This would be
useful anytime a library is trying to use os.fork(), as fork() is not safe in a
multi-threaded
Gregory P. Smith added the comment:
For context, the fundamental problem is that os.fork() is unsafe to use in any
process with threads. concurrent/futures/process.py violates this. So long as
its design involves a thread, it can never be guaranteed not to deadlock.
POSIX forbids
Gregory P. Smith added the comment:
fyi - this appears to have caused https://bugs.python.org/issue46464
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue39
Gregory P. Smith added the comment:
off the top of my head that might actually work as I _think_ "empty" things are
required to consume an unused byte of size no matter what meaning sizeof
shouldn't change? Some testing and standards perusing for C99 is in order to
confirm t
Gregory P. Smith added the comment:
The gdbm module's purpose is effectively one of just exposing the underlying C
library APIs to Python as you said.
Consider this a +1 in favor of exposing the new APIs in the Python gdbm module.
I'm not concerned about anyone wanting these in older
Gregory P. Smith added the comment:
I tested my PR branch on 32-bit arm (raspbian bullseye) and the microbenchmark
timing shows no change (within the noise across repeated runs). Unsurprising
as division is entirely different on 32-bit arm.
Raspbian uses armv6 for compatibility
Change by Gregory P. Smith :
--
nosy: +mark.dickinson
___
Python tracker
<https://bugs.python.org/issue46406>
___
___
Python-bugs-list mailing list
Unsubscribe:
Gregory P. Smith added the comment:
The PR was directly inspired by Mark Dickinson's code in the email thread
directly using __asm__ to get the instruction he wanted. There is usually a
way to make the compiler actually do what you intend. This appears to be it.
Interestingly
Change by Gregory P. Smith :
--
keywords: +patch
pull_requests: +28829
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/30626
___
Python tracker
<https://bugs.python.org/issu
New submission from Gregory P. Smith :
Based on a python-dev thread, we've come up with faster int division code for
CPython's bignums.
https://mail.python.org/archives/list/python-...@python.org/thread/ZICIMX5VFCX4IOFH5NUPVHCUJCQ4Q7QM/#NEUNFZU3TQU4CPTYZNF3WCN7DOJBBTK5
filing this issue
Gregory P. Smith added the comment:
[data] I finally dug up the old YouTube doc at work with their findings. Mostly
just posting this here for future public reference if anyone wants. Nothing
surprising.
When youtube experimented with a modified 2.7 adding "eternal refcounts"
Gregory P. Smith added the comment:
agreed.
--
resolution: -> rejected
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
when was the regression introduced?
I wouldn't necessarily rush an urgent release out unless this was just
introduced in the 3.10.1 patch release. Could it wait for 3.10.2 already
scheduled for four weeks from now?
--
nosy: +gregory.p.smith
Gregory P. Smith added the comment:
>From the context of help(typical_subclass_of_abstract_thing.method), the user
>isn't told what the base classes are or which one might have documentation.
>So it could become an exercise in frustration.
and LOL yes the parameter name is silly b
New submission from Gregory P. Smith :
Today `inspect.getdoc()` is quite simple. If a method has any docstring, it
returns it.
There is one idiom where this is not very useful to our users. A """See base
class.""" docstring on a method.
Rather than having to r
New submission from Gregory P. Smith :
cpython/b$ ./python
Python 3.11.0a3+ (heads/main-dirty:081a214008, Jan 11 2022, 02:48:22) [GCC
11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import ast
&g
Change by Gregory P. Smith :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
New changeset 1b30660c3b40da557050948ac8635f2dc50c4ee2 by Nikita Sobolev in
branch 'main':
bpo-46120: State that `|` is preferred over `Union` (GH-30222)
https://github.com/python/cpython/commit/1b30660c3b40da557050948ac8635f2dc50c4ee2
--
nosy
Change by Gregory P. Smith :
--
stage: test needed -> needs patch
___
Python tracker
<https://bugs.python.org/issue10202>
___
___
Python-bugs-list mai
Change by Gregory P. Smith :
--
versions: +Python 3.10, Python 3.11, Python 3.9 -Python 2.7, Python 3.1, Python
3.2
___
Python tracker
<https://bugs.python.org/issue10
Gregory P. Smith added the comment:
[pasting in my investigation that I replied with on a mailing list:]
The possible problem does appear real but it likely not frequently observed and
is something most people reading the code wouldn't see as it's rather esoteric:
https://blog.netherlabs.nl
Gregory P. Smith added the comment:
New changeset b8de8b7039cee47465b2af9950b0b9ed2d3f2903 by Nikita Sobolev in
branch 'main':
bpo-46150: ensure `fakeuser` does not exist in `PosixPathTest.test_expanduser`
(GH-30240)
https://github.com/python/cpython/commit
Change by Gregory P. Smith :
--
pull_requests: +28466
pull_request: https://github.com/python/cpython/pull/30244
___
Python tracker
<https://bugs.python.org/issue46
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue46150>
___
___
Python-
Gregory P. Smith added the comment:
It's not about an advantage, it's about removing the problem of what edit to
make when working on
assert
thing_that_has_a_meaningful_name.methods_have_good_names(value_from_somewhere)
== other_thing_that_is_meaningful, "Description of what the
New submission from Gregory P. Smith :
Now that we have a shiny new parser, can we finally get rid of this language
wart:
assert thing, description # works as intended
assert (thing, description) # always True as non-empty tuples are Truthy
This most often happens when extending thing
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue41222>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Gregory P. Smith :
The announcement email list
(https://mail.python.org/archives/list/python-announce-l...@python.org/) and
other places we make announcements beyond just the release pages like
https://www.python.org/downloads/release/python-3101/ should include a list
Gregory P. Smith added the comment:
While the discussion is interesting for some who are trying to work around
security policy mechanisms, we can't control what policies people enforce upon
themselves that make their machines useless for software development. Marking
as not a bug
Gregory P. Smith added the comment:
Confirmed.
Repro: Do an ubuntu 20.04 install and choose "experimental zfs" support during
install - https://ubuntu.com/blog/zfs-focus-on-ubuntu-20-04-lts-whats-new). On
such a zfs filesystem, the following tests from a ./python -m test.regrtest
Gregory P. Smith added the comment:
https://web.dev/webassembly-threads/ suggests otherwise.
--
___
Python tracker
<https://bugs.python.org/issue31370>
___
___
Gregory P. Smith added the comment:
related: https://www.python.org/dev/peps/pep-0490/ (rejected pep to auto-chain
from C API calls)
I don't come across it often, but I did just review some code at work today
where the chain might have been nice (though not a big deal).
--
nosy
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue38884>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Gregory P. Smith :
--
resolution: -> out of date
stage: needs patch -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
Nice trick with that RO bind mount. tests... yes we do have some writable
source tree messes in there to be dealt with.
--
___
Python tracker
<https://bugs.python.org/issue45
Gregory P. Smith added the comment:
General consensus: There isn't a common need for this.
--
nosy: +gregory.p.smith
resolution: -> rejected
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
Thanks malin!
--
resolution: -> fixed
stage: patch review -> commit review
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
New changeset 86c1265cdc64030c8921e0da5fcae2ac64299c26 by Ma Lin in branch
'3.9':
[3.9] bpo-41735: Fix thread lock in zlib.Decompress.flush() may go wrong
(GH-29588)
https://github.com/python/cpython/commit/86c1265cdc64030c8921e0da5fcae2ac64299c26
Gregory P. Smith added the comment:
New changeset 7edb6270a78c695e4c2ae2432797dc18105374fc by Ma Lin in branch
'main':
bpo-41735: Fix thread lock in zlib.Decompress.flush() may go wrong (GH-29587)
https://github.com/python/cpython/commit/7edb6270a78c695e4c2ae2432797dc18105374fc
Gregory P. Smith added the comment:
A new `intermediate_mode=` kwarg can't _default_ to `mode=` as that would flip
flop the API's default behavior again and no doubt disrupt someone elses
3.7-3.10 authored code depending on it. :(
Regardless I do somewhat like `intermediate_mode=` more than
Gregory P. Smith added the comment:
We do need to document this in ctypes.rst. Quite a bit of code depends on
ctypes.CDLL(None) and similar to get at symbols linked into the interpreter or
already dlopened into the process these days. It works on Linux; I'm assuming
it likely does
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
resolution: later ->
stage: resolved -> patch review
___
Python tracker
<https://bugs.python.or
101 - 200 of 3243 matches
Mail list logo
