]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph.spec.in: clarify two important comments 5077/head
authorNathan Cutler <ncutler@suse.com>
Thu, 25 Jun 2015 16:55:39 +0000 (18:55 +0200)
committerNathan Cutler <ncutler@suse.com>
Thu, 25 Jun 2015 17:55:12 +0000 (19:55 +0200)
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>
ceph.spec.in

index 8a1d987ba21154e33a5e3b4b24e879903c06ebad..638b0b9b34077cb1643b33b686ac7d6cc970ae05 100644 (file)
@@ -18,9 +18,6 @@
   %global _with_systemd 1
 %endif
 
-#################################################################################
-# common
-#################################################################################
 Name:          ceph
 Version:       @VERSION@
 Release:       @RPM_RELEASE@%{?dist}
@@ -33,6 +30,9 @@ Source0:      http://ceph.com/download/%{name}-%{version}.tar.bz2
 %if 0%{?fedora} || 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}
@@ -98,7 +98,7 @@ BuildRequires:        snappy-devel
 %endif
 
 #################################################################################
-# specific
+# distro-conditional dependencies
 #################################################################################
 %if 0%{defined suse_version}
 BuildRequires: net-tools