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

[Andrew Beekhof]

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