Commitf65aa0d

miss-islington

and

hukkin

authored

[3.12]gh-118761: Improve import time ofsubprocess (GH-129427) (#129448)

gh-118761: Improve import time of `subprocess` (GH-129427)* subprocess: lazy import signal and locale to improve module import time(cherry picked from commit49f2465)Co-authored-by: Taneli Hukkinen <3275109+hukkin@users.noreply.github.com>

1 parent0e54315 commitf65aa0dCopy full SHA for f65aa0d

File tree

2 files changed

+16

-2

lines changed

Lib
- subprocess.py
Misc/NEWS.d/next/Library
- 2025-01-29-10-53-32.gh-issue-118761.i8wjpV.rst

2 files changed

+16

-2

lines changed

`‎Lib/subprocess.py‎`

Lines changed: 14 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -43,10 +43,8 @@`
`43`	`43`	`importbuiltins`
`44`	`44`	`importerrno`
`45`	`45`	`importio`
`46`		`-importlocale`
`47`	`46`	`importos`
`48`	`47`	`importtime`
`49`		`-importsignal`
`50`	`48`	`importsys`
`51`	`49`	`importthreading`
`52`	`50`	`importwarnings`
`@@ -138,6 +136,8 @@ def __init__(self, returncode, cmd, output=None, stderr=None):`
`138`	`136`
`139`	`137`	`def__str__(self):`
`140`	`138`	`ifself.returncodeandself.returncode<0:`
	`139`	`+# Lazy import to improve module import time`
	`140`	`+importsignal`
`141`	`141`	`try:`
`142`	`142`	`return"Command '%s' died with %r."% (`
`143`	`143`	`self.cmd,signal.Signals(-self.returncode))`
`@@ -375,6 +375,8 @@ def _text_encoding():`
`375`	`375`	`ifsys.flags.utf8_mode:`
`376`	`376`	`return"utf-8"`
`377`	`377`	`else:`
	`378`	`+# Lazy import to improve module import time`
	`379`	`+importlocale`
`378`	`380`	`returnlocale.getencoding()`
`379`	`381`
`380`	`382`
`@@ -1655,6 +1657,9 @@ def send_signal(self, sig):`
`1655`	`1657`	`# Don't signal a process that we know has already died.`
`1656`	`1658`	`ifself.returncodeisnotNone:`
`1657`	`1659`	`return`
	`1660`	`+`
	`1661`	`+# Lazy import to improve module import time`
	`1662`	`+importsignal`
`1658`	`1663`	`ifsig==signal.SIGTERM:`
`1659`	`1664`	`self.terminate()`
`1660`	`1665`	`elifsig==signal.CTRL_C_EVENT:`
`@@ -1759,6 +1764,9 @@ def _posix_spawn(self, args, executable, env, restore_signals,`
`1759`	`1764`
`1760`	`1765`	`kwargs= {}`
`1761`	`1766`	`ifrestore_signals:`
	`1767`	`+# Lazy import to improve module import time`
	`1768`	`+importsignal`
	`1769`	`+`
`1762`	`1770`	`# See _Py_RestoreSignals() in Python/pylifecycle.c`
`1763`	`1771`	`sigset= []`
`1764`	`1772`	`forsignamein ('SIGPIPE','SIGXFZ','SIGXFSZ'):`
`@@ -2208,9 +2216,13 @@ def send_signal(self, sig):`
`2208`	`2216`	`defterminate(self):`
`2209`	`2217`	`"""Terminate the process with SIGTERM`
`2210`	`2218`	`"""`
	`2219`	`+# Lazy import to improve module import time`
	`2220`	`+importsignal`
`2211`	`2221`	`self.send_signal(signal.SIGTERM)`
`2212`	`2222`
`2213`	`2223`	`defkill(self):`
`2214`	`2224`	`"""Kill the process with SIGKILL`
`2215`	`2225`	`"""`
	`2226`	`+# Lazy import to improve module import time`
	`2227`	`+importsignal`
`2216`	`2228`	`self.send_signal(signal.SIGKILL)`

`‎Misc/NEWS.d/next/Library/2025-01-29-10-53-32.gh-issue-118761.i8wjpV.rst‎`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	+Improve import time of:mod:`subprocess` by lazy importing ``locale`` and
	`2`	+``signal``. Patch by Taneli Hukkinen.

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commitf65aa0d

File tree

2 files changed

2 files changed

`‎Lib/subprocess.py‎`

`‎Misc/NEWS.d/next/Library/2025-01-29-10-53-32.gh-issue-118761.i8wjpV.rst‎`

0 commit comments