Merge pull request #3698 from boegel/unset_TAPE

make sure that $TAPE is unset when using piped tar

Author: akesandgren

easybuild/tools/filetools.py

@@ -141,9 +141,11 @@
     '.tb2': "tar xjf %(filepath)s",
     '.tbz': "tar xjf %(filepath)s",
     '.tbz2': "tar xjf %(filepath)s",
-    # xzipped or xzipped tarball
-    '.tar.xz': "unxz %(filepath)s --stdout | tar x",
-    '.txz': "unxz %(filepath)s --stdout | tar x",
+    # xzipped or xzipped tarball;
+    # need to make sure that $TAPE is not set to avoid 'tar x' command failing,
+    # see https://github.com/easybuilders/easybuild-framework/issues/3652
+    '.tar.xz': "unset TAPE; unxz %(filepath)s --stdout | tar x",
+    '.txz': "unset TAPE; unxz %(filepath)s --stdout | tar x",
     '.xz': "unxz %(filepath)s",
     # tarball
     '.tar': "tar xf %(filepath)s",

test/framework/filetools.py

@@ -92,8 +92,8 @@ def test_extract_cmd(self):
             ('untar.gz', "gunzip -c untar.gz > untar"),
             ("/some/path/test.gz", "gunzip -c /some/path/test.gz > test"),
             ('test.xz', "unxz test.xz"),
-            ('test.tar.xz', "unxz test.tar.xz --stdout | tar x"),
-            ('test.txz', "unxz test.txz --stdout | tar x"),
+            ('test.tar.xz', "unset TAPE; unxz test.tar.xz --stdout | tar x"),
+            ('test.txz', "unset TAPE; unxz test.txz --stdout | tar x"),
             ('test.iso', "7z x test.iso"),
             ('test.tar.Z', "tar xzf test.tar.Z"),
             ('test.foo.bar.sh', "cp -a test.foo.bar.sh ."),