From eb5b78c4bfb0d12a29ec55635898017fa1b91256 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Thu, 2 Nov 2023 15:51:21 +0100 Subject: [PATCH] all: drop support for building with Python2 Now that we no longer test on CentOS7, we also have no more tests that build using Python2. Note that build with Python2 is currently broken already (which would be fixable). Drop Python2 too. Existing Python scripts still use a common subset of Python2 and Python3. They can be improved to use Python3 features in the future. --- .gitlab-ci/run-test.sh | 3 --- NEWS | 1 + contrib/fedora/rpm/NetworkManager.spec | 11 ----------- 3 files changed, 1 insertion(+), 14 deletions(-) diff --git a/.gitlab-ci/run-test.sh b/.gitlab-ci/run-test.sh index 50d0903619..758b8ada05 100755 --- a/.gitlab-ci/run-test.sh +++ b/.gitlab-ci/run-test.sh @@ -61,7 +61,6 @@ check_run_assert() { meson+gcc+docs+valgrind autotools+clang meson+clang - autotools+gcc+docs+el7+py2 rpm+autotools rpm+meson tarball @@ -118,8 +117,6 @@ check_run_clean meson+gcc+docs+valgrind && BUILD_TYPE=meson CC=gcc WITH_DO check_run_clean autotools+clang && BUILD_TYPE=autotools CC=clang WITH_DOCS=0 contrib/scripts/nm-ci-run.sh check_run_clean meson+clang && BUILD_TYPE=meson CC=clang WITH_DOCS=0 contrib/scripts/nm-ci-run.sh -check_run_clean autotools+gcc+docs+el7+py2 && test $IS_CENTOS_7 = 1 && PYTHON=python2 BUILD_TYPE=autotools CC=gcc WITH_DOCS=1 contrib/scripts/nm-ci-run.sh - check_run_clean rpm+autotools && test $IS_FEDORA = 1 -o $IS_CENTOS = 1 && ./contrib/fedora/rpm/build_clean.sh -g -w crypto_gnutls -w debug -w iwd -w test -W meson check_run_clean rpm+meson && test $IS_FEDORA = 1 && ./contrib/fedora/rpm/build_clean.sh -g -w crypto_gnutls -w debug -w iwd -w test -w meson diff --git a/NEWS b/NEWS index e7838bd35f..cd46906b50 100644 --- a/NEWS +++ b/NEWS @@ -10,6 +10,7 @@ Overview of changes since NetworkManager-1.44 link local addresses on default wired connection. * Honor udev property ID_NET_MANAGED_BY to only manage an interface when set to "org.freedesktop.NetworkManager". +* Drop build support with Python2. Python3 is now required. ============================================= NetworkManager-1.44 diff --git a/contrib/fedora/rpm/NetworkManager.spec b/contrib/fedora/rpm/NetworkManager.spec index afc7e8ee14..db07a30ac7 100644 --- a/contrib/fedora/rpm/NetworkManager.spec +++ b/contrib/fedora/rpm/NetworkManager.spec @@ -303,21 +303,10 @@ BuildRequires: mobile-broadband-provider-info-devel BuildRequires: newt-devel %endif BuildRequires: /usr/bin/dbus-launch -%if 0%{?fedora} >= 28 || 0%{?rhel} >= 8 BuildRequires: python3 BuildRequires: python3-gobject-base BuildRequires: python3-dbus BuildRequires: python3-pexpect -%else -BuildRequires: python2 -BuildRequires: pygobject3-base -BuildRequires: dbus-python -BuildRequires: pexpect -%if 0%{?rhel} >= 7 && %{with meson} -BuildRequires: python36-dbus -BuildRequires: python36-gobject -%endif -%endif BuildRequires: libselinux-devel BuildRequires: polkit-devel BuildRequires: jansson-devel