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

[Tomas Radej]

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