[Buildroot] [PATCH] python-pyxml: remove obsolete package

[Buildroot] [PATCH] python-pyxml: remove obsolete package

[yegorslists at googlemail.com]

From: Yegor Yefremov <yegorslists@googlemail.com>

PyXML is bitrotten and obsolete. You'll find the functionality it
previously provided is now included in the Python standard library.

So no need to keep this package in Buildroot anymore.

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/Config.in                      |  1 -
 package/python-pyxml/Config.in         | 12 ------------
 package/python-pyxml/python-pyxml.hash |  2 --
 package/python-pyxml/python-pyxml.mk   | 17 -----------------
 4 files changed, 32 deletions(-)
 delete mode 100644 package/python-pyxml/Config.in
 delete mode 100644 package/python-pyxml/python-pyxml.hash
 delete mode 100644 package/python-pyxml/python-pyxml.mk

diff --git a/package/Config.in b/package/Config.in
index d31b503..d2476f7 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -666,7 +666,6 @@ menu "External python modules"
 	source "package/python-pyudev/Config.in"
 	source "package/python-pyusb/Config.in"
 	source "package/python-pyxb/Config.in"
-	source "package/python-pyxml/Config.in"
 	source "package/python-pyyaml/Config.in"
 	source "package/python-pyzmq/Config.in"
 	source "package/python-requests/Config.in"
diff --git a/package/python-pyxml/Config.in b/package/python-pyxml/Config.in
deleted file mode 100644
index 82316ba..0000000
--- a/package/python-pyxml/Config.in
+++ /dev/null
@@ -1,12 +0,0 @@
-config BR2_PACKAGE_PYTHON_PYXML
-	bool "python-pyxml"
-	depends on BR2_PACKAGE_PYTHON
-	select BR2_PACKAGE_EXPAT
-	help
-	  This is the Python XML package. The distribution contains a
-	  validating XML parser, an implementation of the SAX and DOM
-	  programming interfaces, an interface to the Expat parser (and the
-	  Expat parser itself), and a C helper module that can speed up
-	  xmllib.py by a factor of 5. There's even documentation!
-
-	  http://pyxml.sourceforge.net/topics/index.html
diff --git a/package/python-pyxml/python-pyxml.hash b/package/python-pyxml/python-pyxml.hash
deleted file mode 100644
index df98c46..0000000
--- a/package/python-pyxml/python-pyxml.hash
+++ /dev/null
@@ -1,2 +0,0 @@
-# Locally computed
-sha256 9fab66f9584fb8e67aebd8745a5c97bf1c5a2e2e461adf68862bcec64e448c13  PyXML-0.8.4.tar.gz
diff --git a/package/python-pyxml/python-pyxml.mk b/package/python-pyxml/python-pyxml.mk
deleted file mode 100644
index 7ccce8e..0000000
--- a/package/python-pyxml/python-pyxml.mk
+++ /dev/null
@@ -1,17 +0,0 @@
-################################################################################
-#
-# python-pyxml
-#
-################################################################################
-
-PYTHON_PYXML_VERSION = 0.8.4
-PYTHON_PYXML_SOURCE = PyXML-$(PYTHON_PYXML_VERSION).tar.gz
-PYTHON_PYXML_SITE = http://downloads.sourceforge.net/project/pyxml/pyxml/$(PYTHON_PYXML_VERSION)
-PYTHON_PYXML_LICENSE = BSD-3c
-PYTHON_PYXML_LICENSE_FILES = LICENCE
-PYTHON_PYXML_SETUP_TYPE = distutils
-PYTHON_PYXML_DEPENDENCIES = expat
-PYTHON_PYXML_BUILD_OPTS = --with-libexpat=$(STAGING_DIR)/usr
-PYTHON_PYXML_INSTALL_TARGET_OPTS = --with-libexpat=$(STAGING_DIR)/usr
-
-$(eval $(python-package))
-- 
2.1.4

[Buildroot] [PATCH] python-pyxml: remove obsolete package

[Yann E. MORIN]

Yegor, All,

On 2015-12-16 08:44 +0100, yegorslists at googlemail.com spake thusly:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> PyXML is bitrotten and obsolete. You'll find the functionality it
> previously provided is now included in the Python standard library.
> 
> So no need to keep this package in Buildroot anymore.

You forgot to add it to Config.in.legacy .

Otherwise OK for me.

Regards,
Yann E. MORIN.

> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---
>  package/Config.in                      |  1 -
>  package/python-pyxml/Config.in         | 12 ------------
>  package/python-pyxml/python-pyxml.hash |  2 --
>  package/python-pyxml/python-pyxml.mk   | 17 -----------------
>  4 files changed, 32 deletions(-)
>  delete mode 100644 package/python-pyxml/Config.in
>  delete mode 100644 package/python-pyxml/python-pyxml.hash
>  delete mode 100644 package/python-pyxml/python-pyxml.mk
> 
> diff --git a/package/Config.in b/package/Config.in
> index d31b503..d2476f7 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -666,7 +666,6 @@ menu "External python modules"
>  	source "package/python-pyudev/Config.in"
>  	source "package/python-pyusb/Config.in"
>  	source "package/python-pyxb/Config.in"
> -	source "package/python-pyxml/Config.in"
>  	source "package/python-pyyaml/Config.in"
>  	source "package/python-pyzmq/Config.in"
>  	source "package/python-requests/Config.in"
> diff --git a/package/python-pyxml/Config.in b/package/python-pyxml/Config.in
> deleted file mode 100644
> index 82316ba..0000000
> --- a/package/python-pyxml/Config.in
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -config BR2_PACKAGE_PYTHON_PYXML
> -	bool "python-pyxml"
> -	depends on BR2_PACKAGE_PYTHON
> -	select BR2_PACKAGE_EXPAT
> -	help
> -	  This is the Python XML package. The distribution contains a
> -	  validating XML parser, an implementation of the SAX and DOM
> -	  programming interfaces, an interface to the Expat parser (and the
> -	  Expat parser itself), and a C helper module that can speed up
> -	  xmllib.py by a factor of 5. There's even documentation!
> -
> -	  http://pyxml.sourceforge.net/topics/index.html
> diff --git a/package/python-pyxml/python-pyxml.hash b/package/python-pyxml/python-pyxml.hash
> deleted file mode 100644
> index df98c46..0000000
> --- a/package/python-pyxml/python-pyxml.hash
> +++ /dev/null
> @@ -1,2 +0,0 @@
> -# Locally computed
> -sha256 9fab66f9584fb8e67aebd8745a5c97bf1c5a2e2e461adf68862bcec64e448c13  PyXML-0.8.4.tar.gz
> diff --git a/package/python-pyxml/python-pyxml.mk b/package/python-pyxml/python-pyxml.mk
> deleted file mode 100644
> index 7ccce8e..0000000
> --- a/package/python-pyxml/python-pyxml.mk
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -################################################################################
> -#
> -# python-pyxml
> -#
> -################################################################################
> -
> -PYTHON_PYXML_VERSION = 0.8.4
> -PYTHON_PYXML_SOURCE = PyXML-$(PYTHON_PYXML_VERSION).tar.gz
> -PYTHON_PYXML_SITE = http://downloads.sourceforge.net/project/pyxml/pyxml/$(PYTHON_PYXML_VERSION)
> -PYTHON_PYXML_LICENSE = BSD-3c
> -PYTHON_PYXML_LICENSE_FILES = LICENCE
> -PYTHON_PYXML_SETUP_TYPE = distutils
> -PYTHON_PYXML_DEPENDENCIES = expat
> -PYTHON_PYXML_BUILD_OPTS = --with-libexpat=$(STAGING_DIR)/usr
> -PYTHON_PYXML_INSTALL_TARGET_OPTS = --with-libexpat=$(STAGING_DIR)/usr
> -
> -$(eval $(python-package))
> -- 
> 2.1.4
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH] python-pyxml: remove obsolete package

[Thomas Petazzoni]

Hello,

On Wed, 16 Dec 2015 08:44:28 +0100, yegorslists at googlemail.com wrote:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> PyXML is bitrotten and obsolete. You'll find the functionality it
> previously provided is now included in the Python standard library.
> 
> So no need to keep this package in Buildroot anymore.
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---
>  package/Config.in                      |  1 -
>  package/python-pyxml/Config.in         | 12 ------------
>  package/python-pyxml/python-pyxml.hash |  2 --
>  package/python-pyxml/python-pyxml.mk   | 17 -----------------
>  4 files changed, 32 deletions(-)
>  delete mode 100644 package/python-pyxml/Config.in
>  delete mode 100644 package/python-pyxml/python-pyxml.hash
>  delete mode 100644 package/python-pyxml/python-pyxml.mk

As Yann said, can you respin this patch after taking care of
Config.in.legacy handling ?

Or maybe instead of removing the package we should mark it as
deprecated ?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] python-pyxml: remove obsolete package

[Yann E. MORIN]

Thomas, All,

On 2015-12-18 22:28 +0100, Thomas Petazzoni spake thusly:
> On Wed, 16 Dec 2015 08:44:28 +0100, yegorslists at googlemail.com wrote:
> > From: Yegor Yefremov <yegorslists@googlemail.com>
> > 
> > PyXML is bitrotten and obsolete. You'll find the functionality it
> > previously provided is now included in the Python standard library.
> > 
> > So no need to keep this package in Buildroot anymore.
> > 
> > Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
> > ---
> >  package/Config.in                      |  1 -
> >  package/python-pyxml/Config.in         | 12 ------------
> >  package/python-pyxml/python-pyxml.hash |  2 --
> >  package/python-pyxml/python-pyxml.mk   | 17 -----------------
> >  4 files changed, 32 deletions(-)
> >  delete mode 100644 package/python-pyxml/Config.in
> >  delete mode 100644 package/python-pyxml/python-pyxml.hash
> >  delete mode 100644 package/python-pyxml/python-pyxml.mk
> 
> As Yann said, can you respin this patch after taking care of
> Config.in.legacy handling ?
> 
> Or maybe instead of removing the package we should mark it as
> deprecated ?

I usually agree that we first mark package as being deprecated before we
get rid of them.

However, python-pyxml is so old, its features have long been incoporated
into python stdlib. Probably not many people are using it, if any at
all.

So, I'd vote in favour of just removing it altogether right now.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH] python-pyxml: remove obsolete package

[Thomas Petazzoni]

Hello,

On Fri, 18 Dec 2015 22:40:21 +0100, Yann E. MORIN wrote:

> I usually agree that we first mark package as being deprecated before we
> get rid of them.
> 
> However, python-pyxml is so old, its features have long been incoporated
> into python stdlib. Probably not many people are using it, if any at
> all.
> 
> So, I'd vote in favour of just removing it altogether right now.

Fine with me. Yegor, can you respin your patch then with
Config.in.legacy handling?

It would have been good to hear from Clayton to know why he added this
package in May 2015. I think it might have been at some point a
dependency of some SELinux stuff, but even the pending SELinux patches
don't make any reference to it. The only reference I could find was in
the changelog of one of the patches, which says:

"""
  - Remove dependancy on host-python-pyxml (Ryan B.)
"""

But the current python-pyxml package only implements the target
variant. So, /me confused.

Clayton, Ryan?



-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] python-pyxml: remove obsolete package

[Yegor Yefremov]

On Fri, Dec 18, 2015 at 10:45 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Fri, 18 Dec 2015 22:40:21 +0100, Yann E. MORIN wrote:
>
>> I usually agree that we first mark package as being deprecated before we
>> get rid of them.
>>
>> However, python-pyxml is so old, its features have long been incoporated
>> into python stdlib. Probably not many people are using it, if any at
>> all.
>>
>> So, I'd vote in favour of just removing it altogether right now.
>
> Fine with me. Yegor, can you respin your patch then with
> Config.in.legacy handling?

I'll respin the patch.

> It would have been good to hear from Clayton to know why he added this
> package in May 2015. I think it might have been at some point a
> dependency of some SELinux stuff, but even the pending SELinux patches
> don't make any reference to it. The only reference I could find was in
> the changelog of one of the patches, which says:
>
> """
>   - Remove dependancy on host-python-pyxml (Ryan B.)
> """
>
> But the current python-pyxml package only implements the target
> variant. So, /me confused.
>
> Clayton, Ryan?
>
>
>
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com