From d302aeab8fd1ec79ceb1b31aedb69fdb6493f38e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Marczykowski-G=C3=B3recki?= Date: Wed, 15 Jan 2020 21:41:17 +0100 Subject: [PATCH] Drop Python2 support QubesOS/qubes-issues#5297 --- .travis.yml | 1 - Makefile | 2 +- README.md | 7 +------ debian/control | 14 +------------- rpm_spec/qubes-core-admin-client.spec.in | 23 +---------------------- run-tests | 2 +- 6 files changed, 5 insertions(+), 44 deletions(-) diff --git a/.travis.yml b/.travis.yml index 5469ce4..8dd50ce 100644 --- a/.travis.yml +++ b/.travis.yml @@ -2,7 +2,6 @@ sudo: required dist: bionic language: python python: - - '2.7' - '3.5' - '3.6' - '3.7' diff --git a/Makefile b/Makefile index 6bf4e05..376ee87 100644 --- a/Makefile +++ b/Makefile @@ -1,7 +1,7 @@ .PHONY: all all: build -PYTHON ?= python +PYTHON ?= python3 .PHONY: build build: diff --git a/README.md b/README.md index f837916..f521902 100644 --- a/README.md +++ b/README.md @@ -6,9 +6,4 @@ https://www.qubes-os.org/doc/admin-api/ for protocol specification. Compatibility ============= -Most of the API modules are compatible with Python >= 2.7. -Very few parts require Python >= 3.5: - - tools (`qvm-*`) - - qubesadmin.events module (for asyncio module) - -Parts not compatible with Python < 3.5, are not installed in such environment. +This package requires Python >= 3.5 diff --git a/debian/control b/debian/control index 7907434..6d9c537 100644 --- a/debian/control +++ b/debian/control @@ -5,8 +5,7 @@ Maintainer: Marek Marczykowski-Górecki Build-Depends: debhelper (>= 9), dh-python, - python-all, - python-setuptools, + python3-all, python3-all, python3-setuptools, python3-sphinx, @@ -30,17 +29,6 @@ Depends: Description: Qubes administrative tools Tools to manage Qubes system using Admin API -Package: python-qubesadmin -Architecture: any -Depends: - python-docutils, - python-lxml, - ${python:Depends}, -Provides: - ${python:Provides}, -Description: Python 2 bindings for Qubes Admin API - Python 2 client side module for Qubes Admin API - Package: python3-qubesadmin Architecture: any Depends: diff --git a/rpm_spec/qubes-core-admin-client.spec.in b/rpm_spec/qubes-core-admin-client.spec.in index bb66ddf..8aef2b7 100644 --- a/rpm_spec/qubes-core-admin-client.spec.in +++ b/rpm_spec/qubes-core-admin-client.spec.in @@ -7,8 +7,6 @@ Group: Qubes License: LGPLv2.1+ URL: https://www.qubes-os.org -BuildRequires: python2-setuptools -BuildRequires: python2-devel BuildRequires: python%{python3_pkgversion}-setuptools BuildRequires: python%{python3_pkgversion}-devel BuildRequires: python%{python3_pkgversion}-sphinx @@ -21,22 +19,9 @@ Source0: %{name}-%{version}.tar.gz %description This package include managemt tools, like qvm-*. -%package -n python2-qubesadmin -Summary: Python2 module qubesadmin -Requires: python-daemon -Requires: python-docutils -Requires: python2-lxml - -%description -n python2-qubesadmin -Python2 module qubesadmin. - %package -n python%{python3_pkgversion}-qubesadmin Summary: Python%{python3_pkgversion} module qubesadmin -%if 0%{?rhel} >= 7 -Requires: python-daemon -%else -Requires: python3-daemon -%endif +Requires: python%{python3_pkgversion}-daemon Requires: python%{python3_pkgversion}-docutils Requires: python%{python3_pkgversion}-lxml Requires: python%{python3_pkgversion}-xcffib @@ -55,8 +40,6 @@ make -C doc PYTHON=%{__python3} SPHINXBUILD=sphinx-build-%{python3_version} man %install rm -rf build -%make_install PYTHON=%{__python2} -rm -rf build %make_install PYTHON=%{__python3} make -C doc DESTDIR=$RPM_BUILD_ROOT \ @@ -73,10 +56,6 @@ make -C doc DESTDIR=$RPM_BUILD_ROOT \ %{_mandir}/man1/qvm-*.1* %{_mandir}/man1/qubes*.1* -%files -n python2-qubesadmin -%{python2_sitelib}/qubesadmin-*egg-info -%{python2_sitelib}/qubesadmin - %files -n python%{python3_pkgversion}-qubesadmin %{python3_sitelib}/qubesadmin-*egg-info %{python3_sitelib}/qubesadmin diff --git a/run-tests b/run-tests index 4489a04..2e0a101 100755 --- a/run-tests +++ b/run-tests @@ -1,6 +1,6 @@ #!/bin/sh -: "${PYTHON:=python}" +: "${PYTHON:=python3}" : "${ROOTDIR:=.}" : "${TESTPYTHONPATH:=$ROOTDIR/test-packages}"