diff mbox series

[V2] setuptools3-base.bbclass: override default subprocess timeout

Message ID 20250217055455.3351270-1-hongxu.jia@windriver.com
State Accepted, archived
Commit a3a2edbf7139b7f8c665c2b0b13e094a334e4441
Headers show
Series [V2] setuptools3-base.bbclass: override default subprocess timeout | expand

Commit Message

Hongxu Jia Feb. 17, 2025, 5:54 a.m. UTC
The environment variable SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT allows to override
the subprocess timeout. The default is 40 seconds and should work for most
needs.[1] However, it was not enough while using git shallow tarball and starting
multiple Yocto world builds in one host.

|   File "tmp/work/x86_64-linux/python3-scancode-native/32.1.0/recipe-sysroot-
native/usr/lib/python3.13/subprocess.py", line 1263, in _check_timeout
|     raise TimeoutExpired(
|     ...<2 lines>...
|             stderr=b''.join(stderr_seq) if stderr_seq else None)
| subprocess.TimeoutExpired: Command '['git', '--git-dir', 'tmp/work/x86_64-
linux/python3-scancode-native/32.1.0/git/.git', 'status', '--porcelain',
'--untracked-files=no']' timed out after 40 seconds

Explicitly set variable SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT to 600s in bbclass,
and we could override it in local.conf

[1] https://github.com/pypa/setuptools-scm/blob/main/docs/overrides.md

Issue: LINCD-21838
(LOCAL REV: NOT UPSTREAM) -- Sent to oe-core on 20250217

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/classes-recipe/setuptools3-base.bbclass | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Yoann Congal Feb. 17, 2025, 6:28 a.m. UTC | #1
Hello,


Le lun. 17 févr. 2025 à 06:54, hongxu via lists.openembedded.org
<hongxu.jia=eng.windriver.com@lists.openembedded.org> a écrit :

> The environment variable SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT allows to
> override
> the subprocess timeout. The default is 40 seconds and should work for most
> needs.[1] However, it was not enough while using git shallow tarball and
> starting
> multiple Yocto world builds in one host.
>
> |   File
> "tmp/work/x86_64-linux/python3-scancode-native/32.1.0/recipe-sysroot-
> native/usr/lib/python3.13/subprocess.py", line 1263, in _check_timeout
> |     raise TimeoutExpired(
> |     ...<2 lines>...
> |             stderr=b''.join(stderr_seq) if stderr_seq else None)
> | subprocess.TimeoutExpired: Command '['git', '--git-dir',
> 'tmp/work/x86_64-
> linux/python3-scancode-native/32.1.0/git/.git', 'status', '--porcelain',
> '--untracked-files=no']' timed out after 40 seconds
>
> Explicitly set variable SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT to 600s in
> bbclass,
> and we could override it in local.conf
>
> [1] https://github.com/pypa/setuptools-scm/blob/main/docs/overrides.md
>
> Issue: LINCD-21838
> (LOCAL REV: NOT UPSTREAM) -- Sent to oe-core on 20250217

That looks like an internal tag that should not be part of the patch sent
to oe-core.

Regards,


> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>  meta/classes-recipe/setuptools3-base.bbclass | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/meta/classes-recipe/setuptools3-base.bbclass
> b/meta/classes-recipe/setuptools3-base.bbclass
> index 27af6abc58..190d9e6e3a 100644
> --- a/meta/classes-recipe/setuptools3-base.bbclass
> +++ b/meta/classes-recipe/setuptools3-base.bbclass
> @@ -23,6 +23,10 @@ export CCSHARED  = "-fPIC -DPIC"
>  # the python executable
>  export LINKFORSHARED = "${SECURITY_CFLAGS} -Xlinker -export-dynamic"
>
> +# The environment variable SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT allows
> +# to override the subprocess timeout.
> +export SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT ??= "600"
> +
>  FILES:${PN} += "${PYTHON_SITEPACKAGES_DIR}"
>  FILES:${PN}-staticdev += "${PYTHON_SITEPACKAGES_DIR}/*.a"
>  FILES:${PN}-dev += "${PYTHON_SITEPACKAGES_DIR}/*.la"
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#211513):
> https://lists.openembedded.org/g/openembedded-core/message/211513
> Mute This Topic: https://lists.openembedded.org/mt/111227231/4316185
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> yoann.congal@smile.fr]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
diff mbox series

Patch

diff --git a/meta/classes-recipe/setuptools3-base.bbclass b/meta/classes-recipe/setuptools3-base.bbclass
index 27af6abc58..190d9e6e3a 100644
--- a/meta/classes-recipe/setuptools3-base.bbclass
+++ b/meta/classes-recipe/setuptools3-base.bbclass
@@ -23,6 +23,10 @@  export CCSHARED  = "-fPIC -DPIC"
 # the python executable
 export LINKFORSHARED = "${SECURITY_CFLAGS} -Xlinker -export-dynamic"
 
+# The environment variable SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT allows
+# to override the subprocess timeout.
+export SETUPTOOLS_SCM_SUBPROCESS_TIMEOUT ??= "600"
+
 FILES:${PN} += "${PYTHON_SITEPACKAGES_DIR}"
 FILES:${PN}-staticdev += "${PYTHON_SITEPACKAGES_DIR}/*.a"
 FILES:${PN}-dev += "${PYTHON_SITEPACKAGES_DIR}/*.la"