[ClusterLabs Developers] Python dependency in Fedora's pacemaker SPEC unnecessary?

[Andrew Beekhof]

I did the builds but fedpkg update craps itself.  Shrug.

> On 19 Aug 2015, at 12:20 am, Tomas Radej <tradej at redhat.com> wrote:
> 
> 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 <tradej at 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 <tradej at 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 <tradej at redhat.com> - 1.1.12-3.1
>>> +- Dropped unnecessary Requires on Python in the main package
>>> +
>>> * Thu Jun 18 2015 Fedora Release Engineering
>>> <rel-eng at 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 at clusterlabs.org
>>> http://clusterlabs.org/mailman/listinfo/developers
>> 
>> 
>> _______________________________________________
>> Developers mailing list
>> Developers at clusterlabs.org
>> http://clusterlabs.org/mailman/listinfo/developers
>> 
> 
> -- 
> Tomas Radej
> 
> _______________________________________________
> Developers mailing list
> Developers at clusterlabs.org
> http://clusterlabs.org/mailman/listinfo/developers