Previously we only checked the previous tag, not the current one. Move the
check up a bit, make it more explicit.

Signed-off-by: Peter Hutterer <[email protected]>
---
 release.sh |    9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/release.sh b/release.sh
index c827fe1..2d6f1d4 100755
--- a/release.sh
+++ b/release.sh
@@ -111,6 +111,12 @@ while [ $# != 0 ]; do
     esac
 done
 
+if [ -z "$tag_previous" ] || [ -z "$tag_current" ]; then
+    echo "error: missing previous or current tag"
+    usage
+    exit 1
+fi
+
 # Check for uncommitted/queued changes.
 if [ "x$ignorechanges" != "x1" ]; then
     set +e
@@ -168,8 +174,7 @@ tarbz2="$modulever.tar.bz2"
 announce="$tarball_dir/$modulever.announce"
 
 echo "checking parameters"
-if ! [ -f "$tarball_dir/$tarbz2" ] ||
-     [ -z "$tag_previous" ]; then
+if ! [ -f "$tarball_dir/$tarbz2" ]; then
     echo "error: incorrect parameters!"
     usage
     exit 1
-- 
1.7.10.4


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to