On Fri, Oct 13, 2023 at 10:08 AM Trevor Gamblin wrote:
>
> There are a handful of CLI-related ptests that consistently fail with an
> apparent permissions issue, but making the
> .../site-packages/cpuinfo/cpuinfo.py script executable does not seem to
> fix the problem. Skip these tests for now, and make sure that the right
> /usr/bin/env line is present in py-cpuinfo's scripts on the image.
>
Good. Can you also move it out of PTESTS_PROBLEMS_META_PYTHON into
either PTESTS_FAST_META_PYTHON or PTESTS_SLOW_META_PYTHON
> Signed-off-by: Trevor Gamblin
> ---
> .../0001-test_cli.py-disable.patch| 32 +++
> .../python/python3-py-cpuinfo_9.0.0.bb| 8 +
> 2 files changed, 40 insertions(+)
> create mode 100644
> meta-python/recipes-devtools/python/python3-py-cpuinfo/0001-test_cli.py-disable.patch
>
> diff --git
> a/meta-python/recipes-devtools/python/python3-py-cpuinfo/0001-test_cli.py-disable.patch
>
> b/meta-python/recipes-devtools/python/python3-py-cpuinfo/0001-test_cli.py-disable.patch
> new file mode 100644
> index 00..e4165d2d66
> --- /dev/null
> +++
> b/meta-python/recipes-devtools/python/python3-py-cpuinfo/0001-test_cli.py-disable.patch
> @@ -0,0 +1,32 @@
> +From 620cb80b0bb5ec58fe15592f033487a4e1cd3859 Mon Sep 17 00:00:00 2001
> +From: Trevor Gamblin
> +Date: Thu, 12 Oct 2023 16:46:53 -0400
> +Subject: [PATCH] test_cli.py: disable
> +
> +These tests fail if .../site-packages/cpuinfo/cpuinfo.py isn't
> +executable, but setting it so during do_install doesn't seem to work.
> +Disable them for now so that the ptests work with the remaining ~150
> +successes.
> +
> +Upstream-Status: Inappropriate [OE-Specific]
> +
> +Signed-off-by: Trevor Gamblin
> +---
> + tests/test_cli.py | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/tests/test_cli.py b/tests/test_cli.py
> +index a6b1985..43c4334 100644
> +--- a/tests/test_cli.py
> b/tests/test_cli.py
> +@@ -6,6 +6,7 @@ import helpers
> +
> +
> +
> ++@unittest.skip("Disable while permissions issues are figured out")
> + class TestCLI(unittest.TestCase):
> + def setUp(self):
> + helpers.backup_data_source(cpuinfo)
> +--
> +2.41.0
> +
> diff --git a/meta-python/recipes-devtools/python/python3-py-cpuinfo_9.0.0.bb
> b/meta-python/recipes-devtools/python/python3-py-cpuinfo_9.0.0.bb
> index 5fbac03ac0..84172c058e 100644
> --- a/meta-python/recipes-devtools/python/python3-py-cpuinfo_9.0.0.bb
> +++ b/meta-python/recipes-devtools/python/python3-py-cpuinfo_9.0.0.bb
> @@ -11,8 +11,16 @@ SRC_URI[sha256sum] =
> "3cdbbf3fac90dc6f118bfd64384f309edeadd902d7c8fb17f02ffa1fc3
> inherit ptest pypi setuptools3
>
> SRC_URI += "file://run-ptest \
> +file://0001-test_cli.py-disable.patch \
> "
>
> +do_install:append() {
> +# Make sure we use /usr/bin/env python3
> +for PYTHSCRIPT in `grep -rIl '^#!.*python' ${D}`; do
> +sed -i -e '1s|^#!.*|#!/usr/bin/env ${PYTHON_PN}|' $PYTHSCRIPT
> +done
> +}
> +
> do_install_ptest() {
> install -d ${D}${PTEST_PATH}/tests
> cp -rf ${S}/tests/* ${D}${PTEST_PATH}/tests/
> --
> 2.41.0
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105484):
https://lists.openembedded.org/g/openembedded-devel/message/105484
Mute This Topic: https://lists.openembedded.org/mt/101944756/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-