From 10d9c401828a4699a764b6a87060f4a07708fa01 Mon Sep 17 00:00:00 2001
From: Ralf Zerres <ralf.zerres@networkx.de>
Date: Thu, 22 Nov 2018 16:42:00 +0100
Subject: [PATCH] tape-admin: rework ltfs-reformat

* force reformatting for given tape
* wipe() does not make sence, since the used space will not be freed
  without a format

Signed-off-by: Ralf Zerres <ralf.zerres@networkx.de>
---
 bin/tape-admin | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/bin/tape-admin b/bin/tape-admin
index 79f30ec..55a6dd1 100755
--- a/bin/tape-admin
+++ b/bin/tape-admin
@@ -560,7 +560,7 @@ ltfs_format () {
     fi
 
     make_err_file
-    ${MKLTFS} --device=$ltfs_devname --volume-name=${volume_name} --tape-serial=${volume_serial} 2>${ERRFILE}
+    ${MKLTFS} --device=$ltfs_devname --volume-name=${volume_name} --tape-serial=${volume_serial} --force 2>${ERRFILE}
     RET=$?
     rm -f ${ERRFILE}
     return $RET
@@ -688,9 +688,11 @@ ltfs_reformat () {
 	if [ ${#need_format} -ge 1 ]; then
 	    ltfs_format ${volume_name_active} ${volume_serial}
 	else
-	    ltfs_wipe
+	    #ltfs_wipe
 	    ltfs_umount
-	    ltfs_format ${volume_name_active} ${volume_serial}
+	    if [ $? -eq 0 ]; then
+		ltfs_format ${volume_name_active} ${volume_serial}
+	    fi
 	fi
         ltfs_mount
     fi
@@ -1393,8 +1395,10 @@ parse_params () {
 		shift 2
 		;;
 	    --ltfs-reformat)
-		volume_name="$2"
-		shift 2
+		shift 1
+		count=$#
+		test $count -ge 1 && volume_name="$1"
+		shift $count
 		cmd=ltfs-reformat
 		;;
 	    --ltfs-umount)