make-release: Remove unused optional version argument 49/14149/4
authorMichael Meffie <mmeffie@sinenomine.net>
Thu, 16 Apr 2020 11:21:51 +0000 (07:21 -0400)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 12 Jun 2020 06:07:14 +0000 (02:07 -0400)
The make-release help shows an optional version argument, but in fact
the version info is always generated from the git tag name argument,
which makes sense when creating releases.

Continue to throw away the second positional argument just in case
someone is still passing a second argument, but issue a warning if they
do.

Change-Id: Ie4c6e6efb7693e53a02fd009eecd64b47250c848
Reviewed-on: https://gerrit.openafs.org/14149
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: Benjamin Kaduk <kaduk@mit.edu>

build-tools/make-release

index 12e818b..34a23ea 100755 (executable)
@@ -25,9 +25,12 @@ pod2usage(1) if $help;
 pod2usage(-exitstatus => 0, -verbose => 2) if $man;
 
 my $tagName = shift;
-my $version = shift;
+my $notused = shift;  # Used to be "version".
 
 pod2usage(2) if !defined($tagName);
+if (defined $notused) {
+    warn "Warning: argument '$notused' is ignored.\n";
+}
 
 # Tag the repository
 
@@ -40,7 +43,7 @@ if ($tagPoint) {
         or die "git push failed with : $!";
 }
 
-$version = `git describe --abbrev=4 $tagName`;
+my $version = `git describe --abbrev=4 $tagName`;
 chomp $version;
 $version=~s/openafs-[^-]*-//;
 $version=~s/_/./g;
@@ -132,7 +135,7 @@ make-release - Make an OpenAFS release from git
 
 =head1 SYNOPSIS
 
-make-release [options] <tag> [<version>]
+make-release [options] <tag>
 
  Options:
     --help               brief help message