diff mbox series

[10/22] python3-jinja2: upgrade 3.1.4 -> 3.1.5

Message ID 1734945065-4373-10-git-send-email-wangmy@fujitsu.com
State New
Headers show
Series [01/22] diffoscope: upgrade 283 -> 284 | expand

Commit Message

Mingyu Wang (Fujitsu) Dec. 23, 2024, 9:10 a.m. UTC
From: Wang Mingyu <wangmy@fujitsu.com>

0001-python-module-do-not-manipulate-the-environment-when.patch
removed since it's included in 3.1.5

Changelog:
 https://jinja.palletsprojects.com/en/stable/changes/

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
---
 .../python/python3-jinja2/fix-3.13.patch      | 87 -------------------
 ...inja2_3.1.4.bb => python3-jinja2_3.1.5.bb} |  6 +-
 2 files changed, 1 insertion(+), 92 deletions(-)
 delete mode 100644 meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch
 rename meta/recipes-devtools/python/{python3-jinja2_3.1.4.bb => python3-jinja2_3.1.5.bb} (85%)

Comments

Mathieu Dubois-Briand Dec. 24, 2024, 9:12 a.m. UTC | #1
On Mon Dec 23, 2024 at 10:10 AM CET, wangmy via lists.openembedded.org wrote:
> From: Wang Mingyu <wangmy@fujitsu.com>
>
> 0001-python-module-do-not-manipulate-the-environment-when.patch
> removed since it's included in 3.1.5
>
> Changelog:
>  https://jinja.palletsprojects.com/en/stable/changes/
>
> Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
> ---
>  .../python/python3-jinja2/fix-3.13.patch      | 87 -------------------
>  ...inja2_3.1.4.bb => python3-jinja2_3.1.5.bb} |  6 +-
>  2 files changed, 1 insertion(+), 92 deletions(-)
>  delete mode 100644 meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch
>  rename meta/recipes-devtools/python/{python3-jinja2_3.1.4.bb => python3-jinja2_3.1.5.bb} (85%)
>
> diff --git a/meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch b/meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch
> deleted file mode 100644
> index 34ecd15176..0000000000
> --- a/meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch
> +++ /dev/null
> @@ -1,87 +0,0 @@
> -From cf6ba7732b49ab4637aa747186cf1d1572688584 Mon Sep 17 00:00:00 2001
> -From: Thomas Grainger <tagrain@gmail.com>
> -Date: Mon, 13 May 2024 18:02:35 +0100
> -Subject: [PATCH] fix test_package_zip_list on 3.13
> -
> -Upstream-Status: Backport [https://github.com/pallets/jinja/pull/1979]
> -Signed-off-by: Alexander Kanavin <alex@linutronix.de>
> ----

Hi Wang,

We have some failing ptests on this one:

AssertionError:
Failed ptests:
{'python3-jinja2': 'START: ptest-runner\n'

https://valkyrie.yoctoproject.org/#/builders/61/builds/633/steps/12/logs/stdio
https://valkyrie.yoctoproject.org/#/builders/73/builds/635/steps/13/logs/stdio

Ptest logs can be found here:

https://valkyrie.yocto.io/pub/non-release/20241223-59/testresults/qemuarm64-ptest/core-image-ptest-python3-jinja2/
https://valkyrie.yocto.io/pub/non-release/20241223-59/testresults/qemux86-64-ptest/core-image-ptest-python3-jinja2/
diff mbox series

Patch

diff --git a/meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch b/meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch
deleted file mode 100644
index 34ecd15176..0000000000
--- a/meta/recipes-devtools/python/python3-jinja2/fix-3.13.patch
+++ /dev/null
@@ -1,87 +0,0 @@ 
-From cf6ba7732b49ab4637aa747186cf1d1572688584 Mon Sep 17 00:00:00 2001
-From: Thomas Grainger <tagrain@gmail.com>
-Date: Mon, 13 May 2024 18:02:35 +0100
-Subject: [PATCH] fix test_package_zip_list on 3.13
-
-Upstream-Status: Backport [https://github.com/pallets/jinja/pull/1979]
-Signed-off-by: Alexander Kanavin <alex@linutronix.de>
----
- src/jinja2/loaders.py | 32 ++++++++++++++++++++++++++------
- tests/test_loader.py  |  4 ++--
- 2 files changed, 28 insertions(+), 8 deletions(-)
-
-diff --git a/src/jinja2/loaders.py b/src/jinja2/loaders.py
-index 9eaf647..8c2c86c 100644
---- a/src/jinja2/loaders.py
-+++ b/src/jinja2/loaders.py
-@@ -238,6 +238,30 @@ class FileSystemLoader(BaseLoader):
-         return sorted(found)
- 
- 
-+if sys.version_info >= (3, 13):
-+
-+    def _get_zipimporter_files(z: t.Any) -> t.Dict[str, object]:
-+        try:
-+            get_files = z._get_files
-+        except AttributeError as e:
-+            raise TypeError(
-+                "This zip import does not have the required"
-+                " metadata to list templates."
-+            ) from e
-+        return get_files()
-+else:
-+
-+    def _get_zipimporter_files(z: t.Any) -> t.Dict[str, object]:
-+        try:
-+            files = z._files
-+        except AttributeError as e:
-+            raise TypeError(
-+                "This zip import does not have the required"
-+                " metadata to list templates."
-+            ) from e
-+        return files  # type: ignore[no-any-return]
-+
-+
- class PackageLoader(BaseLoader):
-     """Load templates from a directory in a Python package.
- 
-@@ -382,11 +406,7 @@ class PackageLoader(BaseLoader):
-                     for name in filenames
-                 )
-         else:
--            if not hasattr(self._loader, "_files"):
--                raise TypeError(
--                    "This zip import does not have the required"
--                    " metadata to list templates."
--                )
-+            files = _get_zipimporter_files(self._loader)
- 
-             # Package is a zip file.
-             prefix = (
-@@ -395,7 +415,7 @@ class PackageLoader(BaseLoader):
-             )
-             offset = len(prefix)
- 
--            for name in self._loader._files.keys():
-+            for name in files:
-                 # Find names under the templates directory that aren't directories.
-                 if name.startswith(prefix) and name[-1] != os.path.sep:
-                     results.append(name[offset:].replace(os.path.sep, "/"))
-diff --git a/tests/test_loader.py b/tests/test_loader.py
-index 77d686e..e0683e4 100644
---- a/tests/test_loader.py
-+++ b/tests/test_loader.py
-@@ -364,8 +364,8 @@ def test_package_zip_source(package_zip_loader, template, expect):
- 
- 
- @pytest.mark.xfail(
--    platform.python_implementation() == "PyPy",
--    reason="PyPy's zipimporter doesn't have a '_files' attribute.",
-+    sys.implementation.name == "pypy",
-+    reason="zipimporter doesn't have a '_files' attribute",
-     raises=TypeError,
- )
- def test_package_zip_list(package_zip_loader):
--- 
-2.39.5
-
diff --git a/meta/recipes-devtools/python/python3-jinja2_3.1.4.bb b/meta/recipes-devtools/python/python3-jinja2_3.1.5.bb
similarity index 85%
rename from meta/recipes-devtools/python/python3-jinja2_3.1.4.bb
rename to meta/recipes-devtools/python/python3-jinja2_3.1.5.bb
index 84c40796c6..2a9e361b6e 100644
--- a/meta/recipes-devtools/python/python3-jinja2_3.1.4.bb
+++ b/meta/recipes-devtools/python/python3-jinja2_3.1.5.bb
@@ -4,7 +4,7 @@  HOMEPAGE = "https://pypi.org/project/Jinja2/"
 LICENSE = "BSD-3-Clause"
 LIC_FILES_CHKSUM = "file://LICENSE.txt;md5=5dc88300786f1c214c1e9827a5229462"
 
-SRC_URI[sha256sum] = "4a3aee7acbbe7303aede8e9648d13b8bf88a429282aa6122a993f0ac800cb369"
+SRC_URI[sha256sum] = "8fefff8dc3034e27bb80d67c671eb8a9bc424c0ef4c0826edbff304cceff43bb"
 
 PYPI_PACKAGE = "jinja2"
 
@@ -14,10 +14,6 @@  CLEANBROKEN = "1"
 
 inherit pypi python_flit_core ptest-python-pytest
 
-SRC_URI += " \
-	file://fix-3.13.patch \
-"
-
 RDEPENDS:${PN}-ptest += " \
     python3-unixadmin \
 "