Browse Source

Spec: drop unneeded dependency

corosynclib-devel doesn't need to have a dependency on corosync package.

It's expected that libraries are still working properly (e.g. indicating
errors to their users) when there's no corosync process around in that
moment, and from this perspective it doesn't matter whether it is
installed at all for some purposes, especially having linkage with them
in mind.

Note that the inverse dependency, main corosync package on corosynclib,
is already there (not strictly needed, likely just to enforce package
version match -- otherwise RPM's dependency generator will handle this
on its own using SONAMEs -- hence the comments to that effect are also
added), so breaking this symmetry:
- is supposed to be harmless modulo cases that should be fixed to
  express explicit dependency on corosync's runtime anyway
  (but only for runtime, i.e., Requires as opposed to BuildRequires)
- will effectively enable more lightweight get-build-deps-and-build
  process for programs linking with corosynclibs (e.g. pacemaker),
  as corosync package won't need to be installed needlessly

Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
Reviewed-by: Jan Friesse <jfriesse@redhat.com>
(cherry picked from commit c91c4825ca7f3a3d2cc12358e74ccca3edae0437)
Jan Pokorný 9 years ago
parent
commit
ff57bf9ea7
1 changed files with 2 additions and 1 deletions
  1. 2 1
      corosync.spec.in

+ 2 - 1
corosync.spec.in

@@ -31,6 +31,7 @@ URL: http://corosync.github.io/corosync/
 Source0: http://build.clusterlabs.org/corosync/releases/%{name}-%{version}%{?gittarver}.tar.gz
 Source0: http://build.clusterlabs.org/corosync/releases/%{name}-%{version}%{?gittarver}.tar.gz
 
 
 # Runtime bits
 # Runtime bits
+# The automatic dependency overridden in favor of explicit version lock
 Requires: corosynclib = %{version}-%{release}
 Requires: corosynclib = %{version}-%{release}
 Requires(pre): /usr/sbin/useradd
 Requires(pre): /usr/sbin/useradd
 Requires(post): /sbin/chkconfig
 Requires(post): /sbin/chkconfig
@@ -297,7 +298,6 @@ This package contains corosync test agents.
 %package -n corosynclib
 %package -n corosynclib
 Summary: The Corosync Cluster Engine Libraries
 Summary: The Corosync Cluster Engine Libraries
 Group: System Environment/Libraries
 Group: System Environment/Libraries
-Requires: %{name} = %{version}-%{release}
 
 
 %description -n corosynclib
 %description -n corosynclib
 This package contains corosync libraries.
 This package contains corosync libraries.
@@ -375,6 +375,7 @@ The Corosync Cluster Engine APIs.
 Summary: The Corosync Cluster Engine Qdevice
 Summary: The Corosync Cluster Engine Qdevice
 Group: System Environment/Base
 Group: System Environment/Base
 Requires: %{name} = %{version}-%{release}
 Requires: %{name} = %{version}-%{release}
+# The automatic dependency overridden in favor of explicit version lock
 Requires: corosynclib = %{version}-%{release}
 Requires: corosynclib = %{version}-%{release}
 Requires: nss-tools
 Requires: nss-tools