All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-python][PATCH] python3-termcolor: Use setuptools instead of distutils
@ 2022-02-28 22:48 Khem Raj
  2022-02-28 22:56 ` Justin Bronder
  0 siblings, 1 reply; 3+ messages in thread
From: Khem Raj @ 2022-02-28 22:48 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Khem Raj, Justin Bronder

Signed-off-by: Khem Raj <raj.khem@gmail.com>
Cc: Justin Bronder <jsbronder@cold-front.org>
---
 ...-Use-setuptools-instead-of-distutils.patch | 27 +++++++++++++++++++
 .../python/python3-termcolor_1.1.0.bb         |  3 ++-
 2 files changed, 29 insertions(+), 1 deletion(-)
 create mode 100644 meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch

diff --git a/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch b/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
new file mode 100644
index 0000000000..a55a6a914b
--- /dev/null
+++ b/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
@@ -0,0 +1,27 @@
+From e30308284c721210e2ba50d8b3d159cedf5eada8 Mon Sep 17 00:00:00 2001
+From: Khem Raj <raj.khem@gmail.com>
+Date: Mon, 28 Feb 2022 14:44:42 -0800
+Subject: [PATCH] setup.py: Use setuptools instead of distutils
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+---
+ setup.py | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/setup.py b/setup.py
+index c4fe4ed..dad1d29 100755
+--- a/setup.py
++++ b/setup.py
+@@ -23,7 +23,7 @@
+ # Author: Konstantin Lepa <konstantin.lepa@gmail.com>
+ 
+ import os
+-from distutils.core import setup
++from setuptools import setup
+ 
+ prjdir = os.path.dirname(__file__)
+ 
+-- 
+2.35.1
+
diff --git a/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb b/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
index 17df7f983d..15bab622a6 100644
--- a/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
+++ b/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
@@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://COPYING.txt;md5=809e8749b63567978acfbd81d9f6a27d"
 
 inherit pypi setuptools3
 
-SRC_URI[md5sum] = "043e89644f8909d462fbbfa511c768df"
+SRC_URI += "file://0001-setup.py-Use-setuptools-instead-of-distutils.patch"
+
 SRC_URI[sha256sum] = "1d6d69ce66211143803fbc56652b41d73b4a400a2891d7bf7a1cdf4c02de613b"
 
 BBCLASSEXTEND = "native"
-- 
2.35.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: python3-termcolor: Use setuptools instead of distutils
  2022-02-28 22:48 [meta-python][PATCH] python3-termcolor: Use setuptools instead of distutils Khem Raj
@ 2022-02-28 22:56 ` Justin Bronder
  2022-02-28 23:14   ` [oe] " Khem Raj
  0 siblings, 1 reply; 3+ messages in thread
From: Justin Bronder @ 2022-02-28 22:56 UTC (permalink / raw)
  To: openembedded-devel

So, am I right to stand corrected that patching stagnant projects to use
distutils over setuptools is preferable to dropping the recipe?  I had assumed
from other distribution work that cleaning things out when the opportunity arose
was the norm.  If so, I'll stop trying to figure out if upstream is active and
will fix the package along with sending patch upstream.

Regardless,

Signed-off-by: Justin Bronder <jsbronder@cold-front.org>

On 28/02/22 14:48 -0800, Khem Raj wrote:
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> Cc: Justin Bronder <jsbronder@cold-front.org>
> ---
>  ...-Use-setuptools-instead-of-distutils.patch | 27 +++++++++++++++++++
>  .../python/python3-termcolor_1.1.0.bb         |  3 ++-
>  2 files changed, 29 insertions(+), 1 deletion(-)
>  create mode 100644 meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
> 
> diff --git a/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch b/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
> new file mode 100644
> index 0000000000..a55a6a914b
> --- /dev/null
> +++ b/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
> @@ -0,0 +1,27 @@
> +From e30308284c721210e2ba50d8b3d159cedf5eada8 Mon Sep 17 00:00:00 2001
> +From: Khem Raj <raj.khem@gmail.com>
> +Date: Mon, 28 Feb 2022 14:44:42 -0800
> +Subject: [PATCH] setup.py: Use setuptools instead of distutils
> +
> +Upstream-Status: Pending
> +Signed-off-by: Khem Raj <raj.khem@gmail.com>
> +---
> + setup.py | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/setup.py b/setup.py
> +index c4fe4ed..dad1d29 100755
> +--- a/setup.py
> ++++ b/setup.py
> +@@ -23,7 +23,7 @@
> + # Author: Konstantin Lepa <konstantin.lepa@gmail.com>
> + 
> + import os
> +-from distutils.core import setup
> ++from setuptools import setup
> + 
> + prjdir = os.path.dirname(__file__)
> + 
> +-- 
> +2.35.1
> +
> diff --git a/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb b/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
> index 17df7f983d..15bab622a6 100644
> --- a/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
> +++ b/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
> @@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://COPYING.txt;md5=809e8749b63567978acfbd81d9f6a27d"
>  
>  inherit pypi setuptools3
>  
> -SRC_URI[md5sum] = "043e89644f8909d462fbbfa511c768df"
> +SRC_URI += "file://0001-setup.py-Use-setuptools-instead-of-distutils.patch"
> +
>  SRC_URI[sha256sum] = "1d6d69ce66211143803fbc56652b41d73b4a400a2891d7bf7a1cdf4c02de613b"
>  
>  BBCLASSEXTEND = "native"
> -- 
> 2.35.1
> 

-- 
Justin Bronder


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [oe] python3-termcolor: Use setuptools instead of distutils
  2022-02-28 22:56 ` Justin Bronder
@ 2022-02-28 23:14   ` Khem Raj
  0 siblings, 0 replies; 3+ messages in thread
From: Khem Raj @ 2022-02-28 23:14 UTC (permalink / raw)
  To: Justin Bronder, openembeded-devel

On Mon, Feb 28, 2022 at 2:56 PM Justin Bronder <jsbronder@cold-front.org> wrote:
>
> So, am I right to stand corrected that patching stagnant projects to use
> distutils over setuptools is preferable to dropping the recipe?  I had assumed
> from other distribution work that cleaning things out when the opportunity arose
> was the norm.  If so, I'll stop trying to figure out if upstream is active and
> will fix the package along with sending patch upstream.

you are doing the right thing and removal is on cards. These recipes
were however added in 2019 and 2020 into OE
so I thought of giving an opportunity to original submitters to
comment before we can remove them.

>
> Regardless,
>
> Signed-off-by: Justin Bronder <jsbronder@cold-front.org>
>
> On 28/02/22 14:48 -0800, Khem Raj wrote:
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > Cc: Justin Bronder <jsbronder@cold-front.org>
> > ---
> >  ...-Use-setuptools-instead-of-distutils.patch | 27 +++++++++++++++++++
> >  .../python/python3-termcolor_1.1.0.bb         |  3 ++-
> >  2 files changed, 29 insertions(+), 1 deletion(-)
> >  create mode 100644 meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
> >
> > diff --git a/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch b/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
> > new file mode 100644
> > index 0000000000..a55a6a914b
> > --- /dev/null
> > +++ b/meta-python/recipes-devtools/python/python3-termcolor/0001-setup.py-Use-setuptools-instead-of-distutils.patch
> > @@ -0,0 +1,27 @@
> > +From e30308284c721210e2ba50d8b3d159cedf5eada8 Mon Sep 17 00:00:00 2001
> > +From: Khem Raj <raj.khem@gmail.com>
> > +Date: Mon, 28 Feb 2022 14:44:42 -0800
> > +Subject: [PATCH] setup.py: Use setuptools instead of distutils
> > +
> > +Upstream-Status: Pending
> > +Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > +---
> > + setup.py | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/setup.py b/setup.py
> > +index c4fe4ed..dad1d29 100755
> > +--- a/setup.py
> > ++++ b/setup.py
> > +@@ -23,7 +23,7 @@
> > + # Author: Konstantin Lepa <konstantin.lepa@gmail.com>
> > +
> > + import os
> > +-from distutils.core import setup
> > ++from setuptools import setup
> > +
> > + prjdir = os.path.dirname(__file__)
> > +
> > +--
> > +2.35.1
> > +
> > diff --git a/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb b/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
> > index 17df7f983d..15bab622a6 100644
> > --- a/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
> > +++ b/meta-python/recipes-devtools/python/python3-termcolor_1.1.0.bb
> > @@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://COPYING.txt;md5=809e8749b63567978acfbd81d9f6a27d"
> >
> >  inherit pypi setuptools3
> >
> > -SRC_URI[md5sum] = "043e89644f8909d462fbbfa511c768df"
> > +SRC_URI += "file://0001-setup.py-Use-setuptools-instead-of-distutils.patch"
> > +
> >  SRC_URI[sha256sum] = "1d6d69ce66211143803fbc56652b41d73b4a400a2891d7bf7a1cdf4c02de613b"
> >
> >  BBCLASSEXTEND = "native"
> > --
> > 2.35.1
> >
>
> --
> Justin Bronder
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#95607): https://lists.openembedded.org/g/openembedded-devel/message/95607
> Mute This Topic: https://lists.openembedded.org/mt/89463744/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-02-28 23:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-28 22:48 [meta-python][PATCH] python3-termcolor: Use setuptools instead of distutils Khem Raj
2022-02-28 22:56 ` Justin Bronder
2022-02-28 23:14   ` [oe] " Khem Raj

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.