[PATCH] python: Correct the upstream status in NODEV patch

[PATCH] python: Correct the upstream status in NODEV patch

[Khem Raj]

Ensure it is reflecting correct information and status of patch

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../python/python3/test_posix_nodev_disable.patch           | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
index 85767e1e898..df60302520e 100644
--- a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
+++ b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
@@ -1,7 +1,11 @@
 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]
+See https://github.com/python/cpython/pull/31794 where NODEV
+handling is introduced.
+
+Upstream-Status: Inappropriate [Musl-Specific]
+
 Signed-off-by: Khem Raj <raj.khem@gmail.com>
 --- a/Lib/test/test_posix.py
 +++ b/Lib/test/test_posix.py

Re: [OE-core] [PATCH] python: Correct the upstream status in NODEV patch

[Ross Burton]

On 2 Sep 2025, at 03:21, Khem Raj via lists.openembedded.org <raj.khem=gmail.com@lists.openembedded.org> wrote:
> 
> Ensure it is reflecting correct information and status of patch
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
> .../python/python3/test_posix_nodev_disable.patch           | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> index 85767e1e898..df60302520e 100644
> --- a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> +++ b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> @@ -1,7 +1,11 @@
> 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]
> +See https://github.com/python/cpython/pull/31794 where NODEV
> +handling is introduced.
> +
> +Upstream-Status: Inappropriate [Musl-Specific]

Can you file at least a ticket for the issue that the tests don’t work on musl?  (I see that alpine also has this patch)

Ross

Re: [OE-core] [PATCH] python: Correct the upstream status in NODEV patch

[Khem Raj]

On Thu, Sep 4, 2025 at 5:22 AM Ross Burton <Ross.Burton@arm.com> wrote:
>
> On 2 Sep 2025, at 03:21, Khem Raj via lists.openembedded.org <raj.khem=gmail.com@lists.openembedded.org> wrote:
> >
> > Ensure it is reflecting correct information and status of patch
> >
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > ---
> > .../python/python3/test_posix_nodev_disable.patch           | 6 +++++-
> > 1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> > index 85767e1e898..df60302520e 100644
> > --- a/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> > +++ b/meta/recipes-devtools/python/python3/test_posix_nodev_disable.patch
> > @@ -1,7 +1,11 @@
> > 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]
> > +See https://github.com/python/cpython/pull/31794 where NODEV
> > +handling is introduced.
> > +
> > +Upstream-Status: Inappropriate [Musl-Specific]
>
> Can you file at least a ticket for the issue that the tests don’t work on musl?  (I see that alpine also has this patch)
>

right, I have already added to the issue which caused the problem,
hopefully that is a better place to call it out.

https://github.com/python/cpython/pull/31794#discussion_r2314773987

> Ross