redhat: fix rpmbuild warnings 18/14118/3
authorMichael Meffie <mmeffie@sinenomine.net>
Fri, 20 Mar 2020 21:53:22 +0000 (17:53 -0400)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 10 Apr 2020 15:41:53 +0000 (11:41 -0400)
Fix warnings issued by recent versions of rpmbuild:

    warning: Macro expanded in comment on line 110: %{afsvers}/...
    warning: extra tokens at the end of %endif directive in line 1469:
             %endif  # build_userspace
    warning: line 331: It's not recommended to have unversioned Obsoletes:
             Obsoletes: openafs-client-compat

The first two warnings are just issues with comments, which apparently
are not completely ignored by rpmbuild.  The third issue is a warning
about an unversioned "Obsoletes" directive. Remove the old Obsoletes for
openafs-client-compat, which was obsoleted no later than the 1.4.x
series (more than 10 years ago).

While here clean up the spec by removing the old cvs $Revsion$ keyword
from the comments at the top of the file, and removing an old commented
out setup directive.

Change-Id: I8d7a050ea6a0cc7a2d9a6af9a91d25ce545586e7
Reviewed-on: https://gerrit.openafs.org/14118
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/packaging/RedHat/openafs.spec.in

index 758fcdb..01fa3cf 100644 (file)
@@ -1,4 +1,4 @@
-# Openafs Spec $Revision$
+# Openafs Spec
 
 %define afsvers @PACKAGE_VERSION@
 %define pkgvers @LINUX_PKGVER@
@@ -107,7 +107,6 @@ BuildRequires: elfutils-devel
 
 ExclusiveArch: %{ix86} x86_64 ia64 s390 s390x sparc64 ppc ppc64 ppc64le aarch64
 
-#    http://dl.openafs.org/dl/openafs/candidate/%{afsvers}/...
 Source0: http://www.openafs.org/dl/openafs/%{afsvers}/openafs-%{afsvers}-src.tar.bz2
 Source1: http://www.openafs.org/dl/openafs/%{afsvers}/openafs-%{afsvers}-doc.tar.bz2
 %define srcdir openafs-%{afsvers}
@@ -328,7 +327,6 @@ module.
 Summary: OpenAFS client compatibility symlinks
 Requires: openafs = %{version}, openafs-client = %{version}
 Group: Networking/Filesystems
-Obsoletes: openafs-client-compat
 
 %description compat
 The AFS distributed filesystem.  AFS is a distributed filesystem
@@ -426,7 +424,6 @@ krb4 lookalike services.
 : @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
 
 # Install OpenAFS src and doc
-#%setup -q -n %{srcdir}
 %setup -q -b 1 -n %{srcdir}
 
 ##############################################################################
@@ -1466,7 +1463,7 @@ dkms remove -m %{name} -v %{dkms_version} --rpm_safe_upgrade --all ||:
 %{_mandir}/man8/akeyconvert.8.gz
 %{_mandir}/man8/asetkey.8.gz
 %endif
-%endif  # build_userspace
+%endif
 
 ##############################################################################
 ###