-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ceph.spec.in: clarify two important comments #5077
Conversation
@@ -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 | |||
################################################################################# | |||
# distro-unconditional dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe reword unconditional -> "independent" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"distro-independent dependencies" . . . dunno, sounds rather confusing.
Originally I had written "dependencies that apply across all distros" and "dependencies that are specific only to a particular distro" but that seemed too verbose. Another option would be "non-distro-specific dependencies" and "distro-specific dependencies".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"dependencies that apply across all distro families" sounds fine to me (not too verbose)
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>
6b0e3cd
to
dd212fd
Compare
@ktdreyer updated |
ceph.spec.in: clarify two important comments
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