ref: 9808986ecdffae601440c8278ba31c027985ab35
parent: 8bc4aa2b74a8fd2caaa27f14e09dc7bb45296ac7
author: Roberto E. Vargas Caballero <k0ga@shike2.com>
date: Wed Aug 29 03:00:09 EDT 2018
[tests/ar] Fix cleanup of temporary files Trap was executed before setting the variables with the names of the temporary files, and it meant that these variables were empty and the files were not removed.
--- a/tests/ar/execute/0001-append.sh
+++ b/tests/ar/execute/0001-append.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
###########################################################################
#Append generated files at once to an empty archive
--- a/tests/ar/execute/0002-append.sh
+++ b/tests/ar/execute/0002-append.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
###########################################################################
#Append generated files one by one to an empty file
--- a/tests/ar/execute/0003-append.sh
+++ b/tests/ar/execute/0003-append.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* f1 f2 f3 $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* f1 f2 f3 $tmp1 $tmp2" 0 2 3
###########################################################################
#Append generated files at once to an existing archive
--- a/tests/ar/execute/0004-append.sh
+++ b/tests/ar/execute/0004-append.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
###########################################################################
#empty file list
--- a/tests/ar/execute/0005-append.sh
+++ b/tests/ar/execute/0005-append.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
###########################################################################
#empty file list
--- a/tests/ar/execute/0006-append.sh
+++ b/tests/ar/execute/0006-append.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file.a f1 f2 f3 $tmp1 $tmp2" 0 2 3
###########################################################################
#empty file list
--- a/tests/ar/execute/0013-print.sh
+++ b/tests/ar/execute/0013-print.sh
@@ -1,11 +1,10 @@
#!/bin/sh
-
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0014-print.sh
+++ b/tests/ar/execute/0014-print.sh
@@ -1,11 +1,10 @@
#!/bin/sh
-
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0015-print.sh
+++ b/tests/ar/execute/0015-print.sh
@@ -1,13 +1,11 @@
#!/bin/sh
-
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
-trap "rm -f file.a $tmp1 $tmp2" 0 2 3
+trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
#print 2nd member with verbose
--- a/tests/ar/execute/0016-print.sh
+++ b/tests/ar/execute/0016-print.sh
@@ -1,11 +1,10 @@
#!/bin/sh
-
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0017-print.sh
+++ b/tests/ar/execute/0017-print.sh
@@ -1,10 +1,10 @@
#!/bin/sh
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file.a $tmp1 $tmp2" 0 2 3
#and now with no members in command line
--- a/tests/ar/execute/0018-list.sh
+++ b/tests/ar/execute/0018-list.sh
@@ -4,8 +4,8 @@
tmp1=`mktemp`
tmp2=`mktemp`
-trap "rm -f file.a $tmp1 $tmp2" 0 2 3
+trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
#list 1st member
--- a/tests/ar/execute/0019-list.sh
+++ b/tests/ar/execute/0019-list.sh
@@ -4,8 +4,8 @@
tmp1=`mktemp`
tmp2=`mktemp`
-trap "rm -f file.a $tmp1 $tmp2" 0 2 3
+trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
#print 3rd member
--- a/tests/ar/execute/0020-list.sh
+++ b/tests/ar/execute/0020-list.sh
@@ -4,6 +4,7 @@
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0021-list.sh
+++ b/tests/ar/execute/0021-list.sh
@@ -4,6 +4,7 @@
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0022-list.sh
+++ b/tests/ar/execute/0022-list.sh
@@ -4,8 +4,8 @@
tmp1=`mktemp`
tmp2=`mktemp`
-trap "rm -f file.a $tmp1 $tmp2" 0 2 3
+trap "rm -f file.a $tmp1 $tmp2" 0 2 3
############################################################################
#list 1st member
--- a/tests/ar/execute/0023-extract.sh
+++ b/tests/ar/execute/0023-extract.sh
@@ -1,10 +1,10 @@
#!/bin/sh
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0024-extract.sh
+++ b/tests/ar/execute/0024-extract.sh
@@ -1,12 +1,11 @@
#!/bin/sh
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#extract 3rd member
--- a/tests/ar/execute/0025-extract.sh
+++ b/tests/ar/execute/0025-extract.sh
@@ -1,10 +1,10 @@
#!/bin/sh
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0026-extract.sh
+++ b/tests/ar/execute/0026-extract.sh
@@ -1,10 +1,10 @@
#!/bin/sh
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
+
trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
--- a/tests/ar/execute/0027-extract.sh
+++ b/tests/ar/execute/0027-extract.sh
@@ -1,12 +1,11 @@
#!/bin/sh
-
set -e
tmp1=`mktemp`
tmp2=`mktemp`
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#extract without parameters
--- a/tests/ar/execute/0029-move.sh
+++ b/tests/ar/execute/0029-move.sh
@@ -1,13 +1,11 @@
#!/bin/sh
-
-
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#move 1st without specifier
--- a/tests/ar/execute/0030-move.sh
+++ b/tests/ar/execute/0030-move.sh
@@ -1,13 +1,11 @@
#!/bin/sh
-
-
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#move 1st at the end
--- a/tests/ar/execute/0031-move.sh
+++ b/tests/ar/execute/0031-move.sh
@@ -1,13 +1,11 @@
#!/bin/sh
-
-
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#and now, test without parameters
--- a/tests/ar/execute/0032-update.sh
+++ b/tests/ar/execute/0032-update.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update one member that already exist
--- a/tests/ar/execute/0033-update.sh
+++ b/tests/ar/execute/0033-update.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update one member that doesn't exist
--- a/tests/ar/execute/0034-update.sh
+++ b/tests/ar/execute/0034-update.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update one member that exists and using -a
--- a/tests/ar/execute/0035-update.sh
+++ b/tests/ar/execute/0035-update.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update one member that exists and using -b
--- a/tests/ar/execute/0036-update.sh
+++ b/tests/ar/execute/0036-update.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update one member that doesn't exist and using -b
--- a/tests/ar/execute/0037-update.sh
+++ b/tests/ar/execute/0037-update.sh
@@ -2,10 +2,10 @@
set -e
-trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update one member that doesn't exist and using -a
--- a/tests/ar/execute/0038-update.sh
+++ b/tests/ar/execute/0038-update.sh
@@ -2,10 +2,10 @@
set -e
-#trap "rm -f file* $tmp1 $tmp2" 0 2 3
-
tmp1=`mktemp`
tmp2=`mktemp`
+
+trap "rm -f file* $tmp1 $tmp2" 0 2 3
############################################################################
#Update without parameters