On Mon, Sep 1, 2025 at 2:40 AM Quentin Schulz <[email protected]> wrote:
>
> Hi Khem,
>
> On 8/22/25 8:33 PM, Khem Raj via lists.openembedded.org wrote:
> > Take a partial patch to disable portion of test_makedev, its also applied
> > in alpine. NODEV does not exist on musl
> >
> > Add test_null_dlsym to ignore list on musl, it needs GNU ifunc support
> > and musl does not implement GNU ifuncs
> >
> > fixes
> > Failed ptests:
> > {'python3': ['test_null_dlsym', 'test_makedev', 'python3']}
> >
> > Signed-off-by: Khem Raj <[email protected]>
> > ---
> >   .../python3/test_posix_nodev_disable.patch       | 16 ++++++++++++++++
> >   meta/recipes-devtools/python/python3_3.13.7.bb   |  6 ++++--
> >   2 files changed, 20 insertions(+), 2 deletions(-)
> >   create mode 100644 
> > meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> >
> > diff --git 
> > a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch 
> > b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> > new file mode 100644
> > index 00000000000..85767e1e898
> > --- /dev/null
> > +++ b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> > @@ -0,0 +1,16 @@
> > +Disable part of test_posix.test_makedev that depends on the NODEV
> > +macro being defined (missing on musl).
> > +
> > +Upstream-Status: Submitted [https://github.com/python/cpython/pull/31794]
>
> Uh? This is merged already (for a year) so should be Backport instead
> with the commit log.
>

I think I should have said this as a reference to where the issue got
introduced.

> Additionally...
>
> > +Signed-off-by: Khem Raj <[email protected]>
> > +--- a/Lib/test/test_posix.py
> > ++++ b/Lib/test/test_posix.py
> > +@@ -757,7 +757,7 @@ class PosixTester(unittest.TestCase):
> > +             self.assertRaises((ValueError, OverflowError), posix.makedev, 
> > x, minor)
> > +             self.assertRaises((ValueError, OverflowError), posix.makedev, 
> > major, x)
> > +
> > +-        if sys.platform == 'linux':
> > ++        if False:
>
> This change isn't in the linked Pull Request, so this is all very
> misleading.

Right.

>
> Cheers,
> Quentin
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#222691): 
https://lists.openembedded.org/g/openembedded-core/message/222691
Mute This Topic: https://lists.openembedded.org/mt/114840579/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to