From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peter =?utf-8?q?M=C3=BCller?= <peter.mueller@ipfire.org>
To: development@lists.ipfire.org
Subject:
 Re: [PATCH 1/2] python-six: Removal of python2 & 3 addon versions of six
Date: Fri, 14 May 2021 09:12:41 +0200
Message-ID: <8aae8eba-3a49-2a0d-bafb-7b5c6b467b9b@ipfire.org>
In-Reply-To: <20210513164708.2889937-1-adolf.belka@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0758952980070688721=="
List-Id: <development.lists.ipfire.org>

--===============0758952980070688721==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Reviewed-by: Peter M=C3=BCller <peter.mueller(a)ipfire.org>

> - six is a python compatibility module to enable modules to run on
>    both python2 and python3. The code from six has to be copied into
>    any other module/project that is intending to use it.
> - With the planned removal of python2 then neither version of this
>    compatibility module is needed.
> - Removal of the lfs and rootfiles. Although python-six is an addon
>    its rootfile was installed into the common folder rather than the
>    packages folder.
> - Removal of the python-six and python3-six entries in make.sh
>=20
> Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
> ---
>  config/rootfiles/common/python-six    |  9 ---
>  config/rootfiles/packages/python3-six |  6 --
>  lfs/python-six                        | 82 --------------------------
>  lfs/python3-six                       | 83 ---------------------------
>  make.sh                               |  2 -
>  5 files changed, 182 deletions(-)
>  delete mode 100644 config/rootfiles/common/python-six
>  delete mode 100644 config/rootfiles/packages/python3-six
>  delete mode 100644 lfs/python-six
>  delete mode 100644 lfs/python3-six
>=20
> diff --git a/config/rootfiles/common/python-six b/config/rootfiles/common/p=
ython-six
> deleted file mode 100644
> index 9520c38ae..000000000
> --- a/config/rootfiles/common/python-six
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/EGG-INFO
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/EGG-INFO/PKG-INFO
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/EGG-INFO/SOURCES.txt
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/EGG-INFO/dependency_=
links.txt
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/EGG-INFO/not-zip-safe
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/EGG-INFO/top_level.t=
xt
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/six.py
> -#usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg/six.pyc
> diff --git a/config/rootfiles/packages/python3-six b/config/rootfiles/packa=
ges/python3-six
> deleted file mode 100644
> index 0801d36da..000000000
> --- a/config/rootfiles/packages/python3-six
> +++ /dev/null
> @@ -1,6 +0,0 @@
> -#usr/lib/python3.8/site-packages/six-1.14.0-py3.8.egg-info
> -#usr/lib/python3.8/site-packages/six-1.14.0-py3.8.egg-info/PKG-INFO
> -#usr/lib/python3.8/site-packages/six-1.14.0-py3.8.egg-info/SOURCES.txt
> -#usr/lib/python3.8/site-packages/six-1.14.0-py3.8.egg-info/dependency_link=
s.txt
> -#usr/lib/python3.8/site-packages/six-1.14.0-py3.8.egg-info/top_level.txt
> -usr/lib/python3.8/site-packages/six.py
> diff --git a/lfs/python-six b/lfs/python-six
> deleted file mode 100644
> index 3383e6d0d..000000000
> --- a/lfs/python-six
> +++ /dev/null
> @@ -1,82 +0,0 @@
> -##########################################################################=
#####
> -#                                                                         =
    #
> -# IPFire.org - A linux based firewall                                     =
    #
> -# Copyright (C) 2007-2018  IPFire Team  <info(a)ipfire.org>               =
      #
> -#                                                                         =
    #
> -# This program is free software: you can redistribute it and/or modify    =
    #
> -# it under the terms of the GNU General Public License as published by    =
    #
> -# the Free Software Foundation, either version 3 of the License, or       =
    #
> -# (at your option) any later version.                                     =
    #
> -#                                                                         =
    #
> -# This program is distributed in the hope that it will be useful,         =
    #
> -# but WITHOUT ANY WARRANTY; without even the implied warranty of          =
    #
> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the           =
    #
> -# GNU General Public License for more details.                            =
    #
> -#                                                                         =
    #
> -# You should have received a copy of the GNU General Public License       =
    #
> -# along with this program.  If not, see <http://www.gnu.org/licenses/>.   =
    #
> -#                                                                         =
    #
> -##########################################################################=
#####
> -
> -##########################################################################=
#####
> -# Definitions
> -##########################################################################=
#####
> -
> -include Config
> -
> -VER        =3D 1.10.0
> -
> -THISAPP    =3D six-$(VER)
> -DL_FILE    =3D $(THISAPP).tar.gz
> -DL_FROM    =3D $(URL_IPFIRE)
> -DIR_APP    =3D $(DIR_SRC)/$(THISAPP)
> -TARGET     =3D $(DIR_INFO)/$(THISAPP)
> -PROG       =3D python-six
> -PAK_VER    =3D 1
> -
> -DEPS       =3D
> -
> -##########################################################################=
#####
> -# Top-level Rules
> -##########################################################################=
#####
> -
> -objects =3D $(DL_FILE)
> -
> -$(DL_FILE) =3D $(DL_FROM)/$(DL_FILE)
> -
> -$(DL_FILE)_MD5 =3D 34eed507548117b2ab523ab14b2f8b55
> -
> -install : $(TARGET)
> -
> -check : $(patsubst %,$(DIR_CHK)/%,$(objects))
> -
> -download :$(patsubst %,$(DIR_DL)/%,$(objects))
> -
> -md5 : $(subst %,%_MD5,$(objects))
> -
> -dist:=20
> -	@$(PAK)
> -
> -##########################################################################=
#####
> -# Downloading, checking, md5sum
> -##########################################################################=
#####
> -
> -$(patsubst %,$(DIR_CHK)/%,$(objects)) :
> -	@$(CHECK)
> -
> -$(patsubst %,$(DIR_DL)/%,$(objects)) :
> -	@$(LOAD)
> -
> -$(subst %,%_MD5,$(objects)) :
> -	@$(MD5)
> -
> -##########################################################################=
#####
> -# Installation Details
> -##########################################################################=
#####
> -
> -$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> -	@$(PREBUILD)
> -	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
> -	cd $(DIR_APP) && python setup.py install
> -	@rm -rf $(DIR_APP)
> -	@$(POSTBUILD)
> diff --git a/lfs/python3-six b/lfs/python3-six
> deleted file mode 100644
> index 9d467f273..000000000
> --- a/lfs/python3-six
> +++ /dev/null
> @@ -1,83 +0,0 @@
> -##########################################################################=
#####
> -#                                                                         =
    #
> -# IPFire.org - A linux based firewall                                     =
    #
> -# Copyright (C) 2007-2020  IPFire Team  <info(a)ipfire.org>               =
      #
> -#                                                                         =
    #
> -# This program is free software: you can redistribute it and/or modify    =
    #
> -# it under the terms of the GNU General Public License as published by    =
    #
> -# the Free Software Foundation, either version 3 of the License, or       =
    #
> -# (at your option) any later version.                                     =
    #
> -#                                                                         =
    #
> -# This program is distributed in the hope that it will be useful,         =
    #
> -# but WITHOUT ANY WARRANTY; without even the implied warranty of          =
    #
> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the           =
    #
> -# GNU General Public License for more details.                            =
    #
> -#                                                                         =
    #
> -# You should have received a copy of the GNU General Public License       =
    #
> -# along with this program.  If not, see <http://www.gnu.org/licenses/>.   =
    #
> -#                                                                         =
    #
> -##########################################################################=
#####
> -
> -##########################################################################=
#####
> -# Definitions
> -##########################################################################=
#####
> -
> -include Config
> -
> -VER        =3D 1.14.0
> -
> -THISAPP    =3D six-$(VER)
> -DL_FILE    =3D $(THISAPP).tar.gz
> -DL_FROM    =3D $(URL_IPFIRE)
> -DIR_APP    =3D $(DIR_SRC)/$(THISAPP)
> -TARGET     =3D $(DIR_INFO)/$(THISAPP)
> -PROG       =3D python3-six
> -PAK_VER    =3D 3
> -
> -DEPS       =3D
> -
> -##########################################################################=
#####
> -# Top-level Rules
> -##########################################################################=
#####
> -
> -objects =3D $(DL_FILE)
> -
> -$(DL_FILE) =3D $(DL_FROM)/$(DL_FILE)
> -
> -$(DL_FILE)_MD5 =3D 21674588a57e649d1a6d977ec3122140
> -
> -install : $(TARGET)
> -
> -check : $(patsubst %,$(DIR_CHK)/%,$(objects))
> -
> -download :$(patsubst %,$(DIR_DL)/%,$(objects))
> -
> -md5 : $(subst %,%_MD5,$(objects))
> -
> -dist:=20
> -	@$(PAK)
> -
> -##########################################################################=
#####
> -# Downloading, checking, md5sum
> -##########################################################################=
#####
> -
> -$(patsubst %,$(DIR_CHK)/%,$(objects)) :
> -	@$(CHECK)
> -
> -$(patsubst %,$(DIR_DL)/%,$(objects)) :
> -	@$(LOAD)
> -
> -$(subst %,%_MD5,$(objects)) :
> -	@$(MD5)
> -
> -##########################################################################=
#####
> -# Installation Details
> -##########################################################################=
#####
> -
> -$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> -	@$(PREBUILD)
> -	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
> -	cd $(DIR_APP) && python3 setup.py build
> -	cd $(DIR_APP) && python3 setup.py install --root=3D/
> -	@rm -rf $(DIR_APP)
> -	@$(POSTBUILD)
> diff --git a/make.sh b/make.sh
> index 25616cc4e..d3a96c6cd 100755
> --- a/make.sh
> +++ b/make.sh
> @@ -1458,7 +1458,6 @@ buildipfire() {
>    lfsmake2 igmpproxy
>    lfsmake2 fbset
>    lfsmake2 opus
> -  lfsmake2 python-six
>    lfsmake2 python-pyparsing
>    lfsmake2 spice-protocol
>    lfsmake2 spice
> @@ -1547,7 +1546,6 @@ buildipfire() {
>    lfsmake2 python-distutils-extra
>    lfsmake2 ddns
>    lfsmake2 python3-setuptools-scm
> -  lfsmake2 python3-six
>    lfsmake2 python3-dateutil
>    lfsmake2 python3-jmespath
>    lfsmake2 python3-colorama
>=20

--===============0758952980070688721==--