#1 Use pycryptodomex instead of pycrypto
Merged 4 years ago by aviso. Opened 4 years ago by pghmcfc.
rpms/ pghmcfc/python-dns pycryptodomex  into  master

file modified
+17 -8
@@ -25,7 +25,7 @@ 

  

  Name:           python-%{py_package_name}

  Version:        1.16.0

- Release:        4%{?dist}

+ Release:        5%{?dist}

  Summary:        %{summary}

  

  License:        MIT
@@ -40,25 +40,28 @@ 

  BuildRequires:  python2-devel

  BuildRequires:  python2-setuptools

  BuildRequires:  python2-typing

- BuildRequires:  python2-crypto

+ BuildRequires:  python2-pycryptodomex

  %endif

  

  %if %{with python3}

  BuildRequires:  python%{python3_pkgversion}-devel

  BuildRequires:  python%{python3_pkgversion}-setuptools

- BuildRequires:  python%{python3_pkgversion}-crypto

+ BuildRequires:  python%{python3_pkgversion}-ecdsa

+ BuildRequires:  python%{python3_pkgversion}-pycryptodomex

  %endif

  

  %if 0%{?with_python3}

  BuildRequires:  python%{python3_pkgversion}-devel

  BuildRequires:  python%{python3_pkgversion}-setuptools

- BuildRequires:  python%{python3_pkgversion}-crypto

+ BuildRequires:  python%{python3_pkgversion}-ecdsa

+ BuildRequires:  python%{python3_pkgversion}-pycryptodomex

  %endif

  

  %if 0%{?with_python3_other}

  BuildRequires:  python%{python3_other_pkgversion}-devel

  BuildRequires:  python%{python3_other_pkgversion}-setuptools

- BuildRequires:  python%{python3_other_pkgversion}-crypto

+ BuildRequires:  python%{python3_other_pkgversion}-ecdsa

+ BuildRequires:  python%{python3_other_pkgversion}-pycryptodomex

  %endif

  

  %description %{_description}
@@ -70,7 +73,7 @@ 

  Summary:        %{summary}

  %{?python_provide:%python_provide python2-%{py_package_name}}

  

- Requires:       python2-crypto

+ Requires:       python2-pycryptodomex

  

  %description -n python2-%{py_package_name} %{_description}

  %endif
@@ -81,7 +84,8 @@ 

  Summary:        %{summary}

  %{?python_provide:%python_provide python%{python3_pkgversion}-%{py_package_name}}

  

- Requires:       python%{python3_pkgversion}-crypto

+ Requires:       python%{python3_pkgversion}-ecdsa

+ Requires:       python%{python3_pkgversion}-pycryptodomex

  

  %description -n python%{python3_pkgversion}-%{py_package_name} %{_description}

  %endif
@@ -92,7 +96,8 @@ 

  Summary:        %{summary}

  %{?python_provide:%python_provide python%{python3_other_pkgversion}-%{py_package_name}}

  

- Requires:       python%{python3_other_pkgversion}-crypto

+ Requires:       python%{python3_other_pkgversion}-ecdsa

+ Requires:       python%{python3_other_pkgversion}-pycryptodomex

  

  %description -n python%{python3_other_pkgversion}-%{py_package_name} %{_description}

  %endif
@@ -173,6 +178,10 @@ 

  

  

  %changelog

+ * Tue Nov  5 2019 Paul Howarth <paul@city-fan.org> - 1.16.0-5

+ - Use pycryptodomex instead of pycrypto

+ - Also use python-ecdsa (except with Python 2)

+ 

  * Thu Oct 03 2019 Miro Hrončok <mhroncok@redhat.com> - 1.16.0-4

  - Rebuilt for Python 3.8.0rc1 (#1748018)

  

Upstream prefers pycryptodome/pycryptodomex to pycrypto because they are actively maintained, unlike pycrypto (I am the pycrypto maintainer in Fedora, and would like to retire it before too long).

I also added python-ecdsa as a build and runtime dependency for ECDSA support, except for the python2 package as python2-ecdsa has already been dropped as part of the great python2 package purge,

If you look at the test suite output from the package build, you'll see fewer skipped tests after making this change.

@pwouters I plan to merge this during the week if there is no objection.

Pull-Request has been merged by aviso

4 years ago

Thanks. I forgot to do this.