Hi,

May I ask you again to apply the patch, please? It is very important that pacemaker not require Python 2 in F23 due to the Python 3 initiative.

Thank you very much.

On 30/07/15 23:35, Andrew Beekhof wrote:
The main package used to contain python code, it was split off at some point in 
the past.

On 30 Jul 2015, at 9:14 pm, Tomas Radej <tra...@redhat.com> wrote:

Hi,

I was wondering why there's an explicit "Python >= 2.4" requirement in
the pacemaker SPEC file in Fedora (line 68 in F23 and Rawhide). I
analysed the RPMs and found no *.py files, binary extensions, or Python
executables in the main package, which suggests that the requirement is
no longer necessary.

If that's indeed the case, I'd like to ask you to include the patch
below in the F23 and Rawhide branches in Fedora. It now appears to be
the only change in pacemaker that is necessary for the "Python 3 as
Default" change in Fedora to proceed. The CTS package can remain based
on Python 2 for the time being as it is not included in the Fedora base
images.

Patch:

 From a54a24503a7c57f57d52c20e280c182e966f870b Mon Sep 17 00:00:00 2001
From: Tomas Radej <tra...@redhat.com>
Date: Thu, 30 Jul 2015 10:26:46 +0200
Subject: [PATCH] Dropped unnecessary Requires on Python in the main package

---
pacemaker.spec | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/pacemaker.spec b/pacemaker.spec
index 666a1bc..779f6e0 100644
--- a/pacemaker.spec
+++ b/pacemaker.spec
@@ -2,7 +2,7 @@
%global uname hacluster
%global pcmk_docdir %{_docdir}/%{name}

-%global specversion 2
+%global specversion 3
%global commit a9c81774b89f21f990be255f9862446d1a38afee
%global shortcommit %(c=%{commit}; echo ${c:0:7})
%global github_owner ClusterLabs
@@ -65,7 +65,6 @@ Requires:      resource-agents
Requires:      %{name}-libs = %{version}-%{release}
Requires:      %{name}-cluster-libs = %{version}-%{release}
Requires:      %{name}-cli = %{version}-%{release}
-Requires:      python >= 2.4

%if %{defined systemd_requires}
%systemd_requires
@@ -512,6 +511,9 @@ exit 0
%doc AUTHORS

%changelog
+* Thu Jul 30 2015 Tomas Radej <tra...@redhat.com> - 1.1.12-3.1
+- Dropped unnecessary Requires on Python in the main package
+
* Thu Jun 18 2015 Fedora Release Engineering
<rel-...@lists.fedoraproject.org> - 1.1.12-2.1
- Rebuilt for https://fedoraproject.org/wiki/Fedora_23_Mass_Rebuild

--
2.4.3
--
Tomas Radej



_______________________________________________
Developers mailing list
Developers@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/developers


_______________________________________________
Developers mailing list
Developers@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/developers


--
Tomas Radej

_______________________________________________
Developers mailing list
Developers@clusterlabs.org
http://clusterlabs.org/mailman/listinfo/developers

Reply via email to