On Fri, Oct 13, 2023 at 10:08 AM Trevor Gamblin <[email protected]> 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 <[email protected]> > --- > .../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 0000000000..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 <[email protected]> > +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 <[email protected]> > +--- > + 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 > + > + > + > [email protected]("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: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
