mirror of
https://git.yoctoproject.org/poky
synced 2026-01-29 21:08:42 +01:00
python3-setuptools-scm: respect GIT_CEILING_DIRECTORIES
Fixes https://bugzilla.yoctoproject.org/show_bug.cgi?id=15740 python3-setuptools-scm was ignoring GIT_CEILING_DIRECTORIES which is set by poky, and it was thus finding a wrong value of "toplevel" in ./src/setuptools_scm/_file_finders/git.py The code is supposed to generate the list of files contained in python3-setuptools-scm, but it was instead running "git archive" on whatever git repository was above the build directory, because the tarball containing the sources of python3-setuptools-scm does not contain a .git directory. This is barely noticeable when building as a subdirectory of poky which is only 48MB, but this was causing serious slowdowns of python3-setuptools-scm:do_compile when building inside a big git repository with files tracked using git-lfs (50 minutes in my use-case). Reported upstream as https://github.com/pypa/setuptools-scm/issues/1103 (From OE-Core rev: 4ebe72477484cf68165b6f736ce10373e97d0e6d) (From OE-Core rev: 369eebad4f38c3641be73dbc0490c87636e0912d) Signed-off-by: Etienne Cordonnier <ecordonnier@snap.com> Signed-off-by: Ross Burton <ross.burton@arm.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> Signed-off-by: Steve Sakoman <steve@sakoman.com>
This commit is contained in:
committed by
Steve Sakoman
parent
4b22f235b6
commit
d38a096fde
@@ -0,0 +1,36 @@
|
||||
From a1cc419a118560d63e1ab8838c256a3622185750 Mon Sep 17 00:00:00 2001
|
||||
From: Etienne Cordonnier <ecordonnier@snap.com>
|
||||
Date: Thu, 13 Feb 2025 15:44:40 +0100
|
||||
Subject: [PATCH] respect GIT_CEILING_DIRECTORIES
|
||||
|
||||
Fix for https://github.com/pypa/setuptools-scm/issues/1103
|
||||
|
||||
When searching for the root-directory of the git repository e.g. with git rev-parse --show-toplevel,
|
||||
git stops the search when reaching $GIT_CEILING_DIRECTORIES. By ignoring this variable, the function
|
||||
_git_toplevel can go above the real git repository (e.g. when packaging a tarball without .git repository),
|
||||
and then runs "git archive" on an unrelated git repository.
|
||||
|
||||
Upstream-Status: Pending
|
||||
|
||||
Signed-off-by: Ross Burton <ross.burton@arm.com>
|
||||
Signed-off-by: Etienne Cordonnier <ecordonnier@snap.com>
|
||||
---
|
||||
src/setuptools_scm/_run_cmd.py | 2 +-
|
||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||
|
||||
diff --git a/src/setuptools_scm/_run_cmd.py b/src/setuptools_scm/_run_cmd.py
|
||||
index f2a8285..7e13d9f 100644
|
||||
--- a/src/setuptools_scm/_run_cmd.py
|
||||
+++ b/src/setuptools_scm/_run_cmd.py
|
||||
@@ -98,7 +98,7 @@ def no_git_env(env: Mapping[str, str]) -> dict[str, str]:
|
||||
k: v
|
||||
for k, v in env.items()
|
||||
if not k.startswith("GIT_")
|
||||
- or k in ("GIT_EXEC_PATH", "GIT_SSH", "GIT_SSH_COMMAND")
|
||||
+ or k in ("GIT_CEILING_DIRECTORIES", "GIT_EXEC_PATH", "GIT_SSH", "GIT_SSH_COMMAND")
|
||||
}
|
||||
|
||||
|
||||
--
|
||||
2.43.0
|
||||
|
||||
@@ -6,6 +6,7 @@ argument or in a SCM managed file."
|
||||
LICENSE = "MIT"
|
||||
LIC_FILES_CHKSUM = "file://LICENSE;md5=838c366f69b72c5df05c96dff79b35f2"
|
||||
|
||||
SRC_URI += "file://0001-respect-GIT_CEILING_DIRECTORIES.patch"
|
||||
SRC_URI[sha256sum] = "b5f43ff6800669595193fd09891564ee9d1d7dcb196cab4b2506d53a2e1c95c7"
|
||||
|
||||
inherit pypi python_setuptools_build_meta
|
||||
|
||||
Reference in New Issue
Block a user