#2 Switch BuildRequires to python3
Merged 3 years ago by rathann. Opened 4 years ago by pviktori.
rpms/ pviktori/inn py-br  into  master

file modified
+5 -2
@@ -3,7 +3,7 @@ 

  Summary: The InterNetNews system, an Usenet news server

  Name: inn

  Version: 2.6.3

- Release: 8%{?dist}

+ Release: 9%{?dist}

  #see LICENSE file for details

  License: GPLv2+ and BSD and MIT and Public Domain

  URL: https://www.eyrie.org/~eagle/software/inn/
@@ -43,7 +43,7 @@ 

  BuildRequires: perl(ExtUtils::Embed)

  BuildRequires: perl(GD)

  BuildRequires: perl(MIME::Parser)

- BuildRequires: python2

+ BuildRequires: python3

  BuildRequires: systemd

  BuildRequires: wget

  BuildRequires: %{_bindir}/gpgv2
@@ -611,6 +611,9 @@ 

  %{_mandir}/man1/inews*

  

  %changelog

+ * Wed Sep 09 2020 Petr Viktorin <pviktori@redhat.com> - 2.6.3-9

+ - Switch BuildRequires to python3

+ 

  * Tue Jul 28 2020 Fedora Release Engineering <releng@fedoraproject.org> - 2.6.3-8

  - Rebuilt for https://fedoraproject.org/wiki/Fedora_33_Mass_Rebuild

  

inn seems to build fine with BuildRequiring python3 instead of python2.
Can the BuildRequires be switched to python3? (Or alternatively, removed altogether, or switched to /usr/bin/python – depending on how it's used in the build.)

Thanks for the PR.

python support is not actually enabled, so no wonder the change has no effect. Upstream documentation says python 3.3+ is supported, so yes, we can switch to python3. And see if it works with 3.8 and 3.9.

I have a couple of patches upcoming that reformat that section of the spec file, so I won't be merging your change as-is.

rebased onto cff99b0

3 years ago

I've rebased on top of the changes; could you please look again?

Pull-Request has been merged by rathann

3 years ago