]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph.spec.in: clarify two important comments
authorNathan Cutler <ncutler@suse.com>
Thu, 25 Jun 2015 16:55:39 +0000 (18:55 +0200)
committerNathan Cutler <ncutler@suse.com>
Mon, 26 Oct 2015 11:40:54 +0000 (12:40 +0100)
First, the terms "common" and "specific" are vague. Second,
"common" can easily be confused with the ceph-common subpackage.

Fix this by rephrasing to "distro-unconditional dependencies" and
"distro-conditional dependencies", respectively.

Third, move the "distro-unconditional dependencies" header so it
is above the part where these dependencies are actually defined.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit dd212fd1d367f70c61f65a97599959668b44aa53)

ceph.spec.in

index cf2d901f585d08470940be87a6405745167f4549..6497e9893b302619f67425cbdebccbf9a7d245dd 100644 (file)
@@ -8,9 +8,6 @@
 
 %{!?_udevrulesdir: %global _udevrulesdir /lib/udev/rules.d}
 
-#################################################################################
-# common
-#################################################################################
 Name:          ceph
 Version:       @VERSION@
 Release:       @RPM_RELEASE@%{?dist}
@@ -23,6 +20,9 @@ Source0:      http://ceph.com/download/%{name}-%{version}.tar.bz2
 %if 0%{?fedora} || 0%{?centos} || 0%{?rhel}
 Patch0:                init-ceph.in-fedora.patch
 %endif
+#################################################################################
+# dependencies that apply across all distro families
+#################################################################################
 Requires:      librbd1 = %{epoch}:%{version}-%{release}
 Requires:      librados2 = %{epoch}:%{version}-%{release}
 Requires:      libcephfs1 = %{epoch}:%{version}-%{release}
@@ -80,7 +80,7 @@ BuildRequires:        net-tools
 %endif
 
 #################################################################################
-# specific
+# distro-conditional dependencies
 #################################################################################
 %if 0%{defined suse_version}
 %if 0%{?suse_version} > 1210