New submission from Gregory P. Smith :
os.makedirs used to pass its mode argument on down recursively so that every
level of directory it created would have the specified permissions.
That was removed in Python 3.7 as https://bugs.python.org/issue19930 as if it
were a mis-feature. Maybe
Gregory P. Smith added the comment:
My point is that re.match is a common bug when people really want re.search.
re.prefixmatch makes it explicit and non-confusing and thus unlikely to be used
wrong or misunderstood when read or reviewed.
The term "match" when talking abo
New submission from Gregory P. Smith :
A well known anti-pattern in Python is use of re.match when you meant to use
re.search.
re.fullmatch was added in 3.4 via https://bugs.python.org/issue16203 for
similar reasons.
re.prefixmatch would be similar: we want the re.match behavior, but want
Gregory P. Smith added the comment:
working as intended, they aren't classes.
--
resolution: -> rejected
stage: needs patch -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
Not a big deal if we don't, I just found it odd so I figured I'd pose the
question. That it's been in three releases and only just now come up is
pretty telling that isn't critical.
The code in question was trying to identify public functions in a module
New submission from Gregory P. Smith :
Python 3.7-3.10a1:
```
>>> List.__name__
Traceback (most recent call last):
File "", line 1, in
File "/usr/lib/python3.8/typing.py", line 760, in __getattr__
raise AttributeError(attr)
AttributeError: __name__
>
Gregory P. Smith added the comment:
This bug was introduced into the Python 3.6.8 "bugfix" release.
https://github.com/python/cpython/pull/23231 will fix it if anyone needs that
on modern 3.6.
--
nosy: +gregory.p.smith
versions: +
Gregory P. Smith added the comment:
my bad :)
--
resolution: -> not a bug
stage: needs patch -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by pmp-p :
--
status: closed -> pending
___
Python tracker
<https://bugs.python.org/issue32682>
___
___
Python-bugs-list mailing list
Unsubscrib
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue8041>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Gregory P. Smith added the comment:
I suspect so. If any modern supported python 3.x version runs into an issue
like this I think opening a fresh bugreport is good.
Closing as not reproducable / obsolete.
--
resolution: -> out of date
stage: test needed -> resolved
status
Gregory P. Smith added the comment:
Thanks for the patch! I'm leaving this open as dealing with the other aspect:
* Wrong attribute names around asserts: autospect/auto_spec -> autospec,
set_spec -> spec_set
is still a possibility. (given you also found a number of those in our
co
Gregory P. Smith added the comment:
New changeset 4662fa9bfe4a849fe87bfb321d8ef0956c89a772 by vabr-g in branch
'master':
bpo-41877 Check for asert, aseert, assrt in mocks (GH-23165)
https://github.com/python/cpython/commit/4662fa9bfe4a849fe87bfb321d8ef0956c89a772
Change by Gregory P. Smith :
--
versions: +Python 3.10 -Python 3.6
___
Python tracker
<https://bugs.python.org/issue24651>
___
___
Python-bugs-list mailin
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue24651>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue41877>
___
___
Python-bugs-list mailing list
Un
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue41877>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue42266>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Gregory P. Smith added the comment:
at a glance, it looks like the PR needs updating.
--
___
Python tracker
<https://bugs.python.org/issue13501>
___
___
Pytho
Change by Gregory P. Smith :
--
resolution: -> fixed
stage: patch review -> commit review
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
New changeset d3b4e068077dd26927ae7485bd0303e09d962c02 by Alexey Izbyshev in
branch 'master':
bpo-42146: Unify cleanup in subprocess_fork_exec() (GH-22970)
https://github.com/python/cpython/commit/d3b4e068077dd26927ae7485bd0303e09d962c02
New submission from Gregory P. Smith :
The opcodes generated for the closure defining a class body looks like they
might be suboptimal. It seems stuck using the generic LOAD_NAME and STORE_NAME
opcodes rather than the LOAD_GLOBAL and STORE_FAST and friends as one would
expect and as happens
Gregory P. Smith added the comment:
The first four bytes of the file do not identify a zip file. A zip file is
identified by the end of file central directory. Which you then must read
entries of to determine where the start of the archive may be... often not at
position zero
Gregory P. Smith added the comment:
ZipFile.open() is not the code for opening a zip file. :)
That's the code for opening a file embedded within an already constructed
mode='r' archive as done the ZipFile.__init__() constructor. By the time
you've gotten to the open() method, you've loaded
Gregory P. Smith added the comment:
New changeset c0590c0033e86f98cdf5f2ca6898656f98ab4053 by Alexey Izbyshev in
branch 'master':
bpo-42146: Fix memory leak in subprocess.Popen() in case of uid/gid overflow
(GH-22966)
https://github.com/python/cpython/commit
Gregory P. Smith added the comment:
Performance improvement measured on a 1.4Ghz quad aarch64 A57 (nvidia jetson
nano):
#define VFORK_USABLE 1
test_subprocess: 36.5 seconds
#undef VFORK_USABLE
test_subprocess: 45 seconds
Nice. I really didn't expect a 20% speedup on its testsuite alone
Gregory P. Smith added the comment:
Nice links. LOL, yes, musl source was going to be my next stop if the larger
libc sources proved impossible for a mere mortal to reason about. :)
regarding macOS, agreed. If someone needs vfork() to work there, I believe it
could be made to.
Options
Gregory P. Smith added the comment:
New changeset be3c3a0e468237430ad7d19a33c60d306199a7f2 by Gregory P. Smith in
branch 'master':
bpo-35823: Allow setsid() after vfork() on Linux. (GH-22945)
https://github.com/python/cpython/commit/be3c3a0e468237430ad7d19a33c60d306199a7f2
Gregory P. Smith added the comment:
for what it's worth: false positives are always going to be possible in any
such "magic" check as is_zipfile is.
we don't check the start of the file because zip files are defined by their end
of file central directory which contains length i
Change by Gregory P. Smith :
--
pull_requests: +21863
pull_request: https://github.com/python/cpython/pull/22945
___
Python tracker
<https://bugs.python.org/issue35
Gregory P. Smith added the comment:
regarding excluding the setsid() case: I was being conservative as I couldn't
find a reference of what was and wasn't allowed after vfork.
I found one thing suggesting that on macOS setsid() was not safe after vfork().
But that appeared to be a Darwin
Gregory P. Smith added the comment:
New changeset 473db47747bb8bc986d88ad81799bcbd88153ac5 by Alexey Izbyshev in
branch 'master':
bpo-35823: subprocess: Fix handling of pthread_sigmask() errors (GH-22944)
https://github.com/python/cpython/commit/473db47747bb8bc986d88ad81799bcbd88153ac5
Gregory P. Smith added the comment:
Thank you for taking this on! I'm calling it fixed for now as the buildbots
are looking happy with it. If issues with it arise we can address them.
--
resolution: -> fixed
stage: commit review -> resolved
status: open -&g
Gregory P. Smith added the comment:
> * To avoid repeating long parameter lists in several functions, we can move
> them to a struct. The downside is that we'd need to convert child_exec() to
> use that struct all over the place. I don't have a strong preference here.
Agreed that
Gregory P. Smith added the comment:
now waiting to see how happy all of the buildbots are...
We currently have a `__linux__` check in the code deciding VFORK_USABLE.
>From what I can tell, vfork probably also works on macOS (darwin).
Lets let this run for a bit on Linux and it
Gregory P. Smith added the comment:
New changeset 976da903a746a5455998e9ca45fbc4d3ad3479d8 by Alexey Izbyshev in
branch 'master':
bpo-35823: subprocess: Use vfork() instead of fork() on Linux when safe
(GH-11671)
https://github.com/python/cpython/commit
Gregory P. Smith added the comment:
If you use os.fork() or any of the multiprocessing start methods that call
os.fork() with a process involving threads, this is expected behavior.
os.fork() cannot be used in processes that have threads without potential for
deadlock.
Specifically, make
Change by Gregory P. Smith :
--
resolution: -> fixed
stage: patch review -> commit review
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
New changeset 786addd9d07b6c712b8ea9ee06e1f9f41c1b67a1 by Gregory P. Smith in
branch 'master':
bpo-41586: Attempt to make the pipesize tests more robust. (GH-22839)
https://github.com/python/cpython/commit/786addd9d07b6c712b8ea9ee06e1f9f41c1b67a1
Change by Gregory P. Smith :
--
pull_requests: +21793
stage: commit review -> patch review
pull_request: https://github.com/python/cpython/pull/22839
___
Python tracker
<https://bugs.python.org/issu
Gregory P. Smith added the comment:
this caused a variety of buildbot failures. investigating.
--
resolution: fixed ->
status: closed -> open
___
Python tracker
<https://bugs.python.org/i
Gregory P. Smith added the comment:
fyi - handy command to get that
python -m test.regrtest -v test_subprocess | ts '.%s'
then process that using whatever you want to compute deltas and sort. i fed
the output of that into:
```
#!/usr/bin/python3
"""Parse `python -m test.r
Gregory P. Smith added the comment:
the slowest tests on Linux today (with number of seconds prepended on my system
that runs the suite serially in 34 seconds on a debug build):
3.01 test_check_output_stdout_arg (test.test_subprocess.ProcessTestCaseNoPoll)
... ok
3.01
Change by Gregory P. Smith :
--
resolution: -> not a bug
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
since the time this was filed, subprocess has evolved a lot and third party
options for child process have appeared as well as modern things like:
https://docs.python.org/3/library/asyncio-subprocess.html (stdlib)
https://trio.readthedocs.io/en/stable
Gregory P. Smith added the comment:
2.7 is end of life.
if you have regular expression performance issues with something in 3, please
open a new issue.
--
nosy: +gregory.p.smith
resolution: -> wont fix
stage: -> resolved
status: pending -&g
Change by Gregory P. Smith :
--
versions: +Python 3.10 -Python 3.8
___
Python tracker
<https://bugs.python.org/issue13501>
___
___
Python-bugs-list mailin
Gregory P. Smith added the comment:
based on the reaction from those who own mock at this point, i'm going to close
this. shipping more handy mock matchers with mock makes sense to me as they
belong together. but i'm not willing to argue it if anyone thinks it'll be a
maintenance burden
Gregory P. Smith added the comment:
Parsing support for `f(**mapping)` support is indeed still missing.
as lib2to3 is pending deprecation at this point, i'm not going to work on this.
anyone is welcome to pick it up. modifying the lib2to3 grammar, and any
related code, and adding a test
Gregory P. Smith added the comment:
Thanks Ruben!
--
resolution: -> fixed
stage: patch review -> commit review
status: open -> closed
type: -> enhancement
___
Python tracker
<https://bugs.python
Gregory P. Smith added the comment:
New changeset 23c0fb8edd16fe6d796df2853a5369fd783e05b7 by Ruben Vorderman in
branch 'master':
bpo-41586: Add pipesize parameter to subprocess & F_GETPIPE_SZ and F_SETPIPE_SZ
to fcntl. (GH-21921)
https://github.com/python/cpython/co
Gregory P. Smith added the comment:
status: lib2to3 PendingDeprecationWarning shipped in 3.9. Since we don't have
a specific release planned for the final deprecation, I'll leave this issue
open while we figure that out. Once we do, we should promote this to a regular
DeprecationWarning
Change by Gregory P. Smith :
--
assignee: gregory.p.smith ->
stage: patch review ->
___
Python tracker
<https://bugs.python.org/issue40360>
___
___
Pyth
Gregory P. Smith added the comment:
thanks Kyle!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Gregory P. Smith added the comment:
Status: Waiting until after the importlib.metadata and importlib.resources
backports drop support for both Python 2.7 and 3.5 as keeping them in sync
would be a pain for maintainers (per jaraco on our sprint importlib-any chat
today
Change by Gregory P. Smith :
--
versions: +Python 3.10 -Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python.org/issue38693>
___
___
Python-bugs-list m
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:
I assume a rolling hash is linear at best, even if you do add some skip ahead
checks.
--
___
Python tracker
<https://bugs.python.org/issue41
Gregory P. Smith added the comment:
Another potential algorithm to consider in large needle situations is a
Rabin-Karp rolling hash string search.
If used, it's the kind of algorithm that I'd probably bail out to an alternate
method on if a run of Rabin-Karp started having a high percentage
Gregory P. Smith added the comment:
FWIW I think your numbers look good, a small needle cut-off is likely a good
idea.
--
___
Python tracker
<https://bugs.python.org/issue41
Change by Gregory P. Smith :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue41972>
___
___
Python-bugs-list mailing list
Unsubscribe:
I think the np.cov is from the numpy module (imported/aliased as np?).
If so, the numpy repository should have what you are looking for:
https://github.com/numpy/numpy/blob/156cd054e007b05d4ac4829e10a369d19dd2b0b1/numpy/lib/function_base.py#L2276
Hope that helps
Bruno
On Tuesday, 13
Gregory P. Smith added the comment:
New changeset 64eb259cc1e42a5f74b5911a518d2c50daa8d50b by Kyle Evans in branch
'master':
bpo-40422: Move _Py_*_SUPPRESS_IPH bits into _Py_closerange (GH-22672)
https://github.com/python/cpython/commit/64eb259cc1e42a5f74b5911a518d2c50daa8d50b
Gregory P. Smith added the comment:
Just reuse this bpo issue. I'll mark the PRs as "skip news"; don't worry about
a new news blurb entry as it's all tied to the original one.
--
status: closed -> open
___
Python tracker
<https:/
Change by Gregory P. Smith :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue40423>
___
___
Python-bugs-list mailing list
Un
Gregory P. Smith added the comment:
Ah, yeah ENOSYS is it. I had to do this trick in older subprocess versions for
something else. Still visible here in the old 2.7 backport:
https://github.com/google/python-subprocess32/blob/main/_posixsubprocess.c#L801
Change by Gregory P. Smith :
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue40422>
___
___
Pyth
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue40422>
___
___
Python-
Change by Gregory P. Smith :
--
versions: +Python 3.10 -Python 3.9
___
Python tracker
<https://bugs.python.org/issue40423>
___
___
Python-bugs-list mailin
Gregory P. Smith added the comment:
for reference, very recent Linux kernels appear to have gained a close_range
syscall. http://lkml.iu.edu/hypermail/linux/kernel/2008.0/02649.html
Your diff isn't quite sufficient as is. When depending on a syscall that has a
function provided by libc
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:
looks like it, thanks!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue41972>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Gregory P. Smith added the comment:
It's a known issue and has been outlined very well and still comes up from time
to time in real world applications, which tend to see this issue and Dave's
presentation and just work around it in any way possible for their system and
move on with life
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue36125>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue14527>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Change by pmp-p :
--
nosy: -pmpp
___
Python tracker
<https://bugs.python.org/issue14527>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Change by pmp-p :
--
pull_requests: +21399
pull_request: https://github.com/python/cpython/pull/22361
___
Python tracker
<https://bugs.python.org/issue32
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue19915>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Change by Gregory P. Smith :
--
stage: backport needed -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue9146>
___
___
Change by Gregory P. Smith :
--
resolution: -> fixed
stage: backport needed -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by pmp-p :
--
versions: +Python 3.10, Python 3.9
___
Python tracker
<https://bugs.python.org/issue35243>
___
___
Python-bugs-list mailing list
Unsub
Change by pmp-p :
--
nosy: +pmpp
___
Python tracker
<https://bugs.python.org/issue14527>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
Gregory P. Smith added the comment:
On POSIX systems, keeping the file open means you will keep a handle to the
original file in the case where something moves a new file into it's place (as
is normal during software package updates) or otherwise unlinks the original.
That is the situation
Change by pmp-p :
--
nosy: -pmpp
___
Python tracker
<https://bugs.python.org/issue41770>
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.pyth
pmp-p added the comment:
Hi, you just need to call del sys.modules['xxx'] and have a look at
https://docs.python.org/3.7/library/importlib.html#importlib.invalidate_caches
before importing again
--
nosy: +pmpp
___
Python tracker
<ht
Change by pmp-p :
--
pull_requests: +21251
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/22190
___
Python tracker
<https://bugs.python.org/issu
Change by pmp-p :
--
versions: +Python 3.6, Python 3.7, Python 3.8, Python 3.9 -Python 2.6, Python
2.7, Python 3.1, Python 3.2, Python 3.3, Python 3.4, Python 3.5
___
Python tracker
<https://bugs.python.org/issue14
Change by Gregory P. Smith :
--
versions: +Python 3.10 -Python 3.8
___
Python tracker
<https://bugs.python.org/issue29988>
___
___
Python-bugs-list mailin
pmp-p added the comment:
all PyRun_InteractiveOne* functions are also affected
it is really annoying when implementing repl behaviour when embedding( use
cases : pyodide, android, wasi )
But I think the correct patch is :
-char *newtok = PyOS_Readline(stdin, stdout, tok->pro
Gregory P. Smith added the comment:
The gps-* bots have been updated.
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue41642>
___
___
Change by Gregory P. Smith :
--
pull_requests: -21110
___
Python tracker
<https://bugs.python.org/issue6721>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Gregory P. Smith :
--
pull_requests: -2
___
Python tracker
<https://bugs.python.org/issue36533>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Gregory P. Smith :
--
versions: +Python 3.10 -Python 3.9
___
Python tracker
<https://bugs.python.org/issue4356>
___
___
Python-bugs-list mailin
Change by Gregory P. Smith :
--
assignee: -> gregory.p.smith
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue41586>
___
___
Python-
Gregory P. Smith added the comment:
fwiw, no PEP is needed for things like this. it'd just be an alternative
library implementing the core of the zlib and/or gzip modules behind the scenes.
normally this kind of thing would be done using a check for the availability of
the library
Change by Gregory P. Smith :
--
pull_requests: -21037
___
Python tracker
<https://bugs.python.org/issue41566>
___
___
Python-bugs-list mailing list
Unsubscribe:
Gregory P. Smith added the comment:
While this is present in 3.7 (and earlier?), 3.7 is EOL - security fix only
stage. the 3.8 and 3.9 PRs should automerge after CI finishes.
please reopen the issue or ping me on those PRs if they somehow fail to do so.
--
resolution: -> fi
Gregory P. Smith added the comment:
Thanks!
fyi for confirmation incase anyone doubted:
>>> issubclass(ssl.SSLEOFError, OSError)
True
So from a code point of view, anything already catching the error still catches
the error. 100% bugfix.
--
assignee: christi
601 - 700 of 5107 matches
Mail list logo
