import shutil in sage-ptest

[jdemeyer]

The following can happen when running sage -tp [...]:

Traceback (most recent call last):
  File "/SSD1/sage-5.1.rc0/local/bin/sage-ptest", line 396, in <module>
    shutil.rmtree(TMP)
NameError: name 'shutil' is not defined

apply attachment: 13195_ptest_shutil.patch to the scripts (local/bin) repository.

Component: scripts

Author: Jeroen Demeyer

Reviewer: Volker Braun

Merged: sage-5.1.rc1

Issue created by migration from https://trac.sagemath.org/ticket/13195

[jdemeyer]

Attachment: 13195_ptest_shutil.patch.gz

[jdemeyer]

Description changed:

--- 
+++ 
@@ -6,3 +6,5 @@
     shutil.rmtree(TMP)
 NameError: name 'shutil' is not defined
 ```
+
+**apply** [attachment: 13195_ptest_shutil.patch](https://github.com/sagemath/sage-prod/files/10655888/13195_ptest_shutil.patch.gz) to the scripts (`local/bin`) repository.

[vbraun]

Reviewer: Volker Braun

[vbraun]

comment:2

I've never seen that error but its better to import it if we use it, of course ;-)

[jhpalmieri]

comment:3

For what it's worth, I tested it by adding a line:

diff --git a/sage-ptest b/sage-ptest
--- a/sage-ptest
+++ b/sage-ptest
@@ -388,6 +388,7 @@ for gr in range(0,numglobaliteration):
     # Should TMP be a subdirectory of tempfile.gettempdir() rather than SAGE_TE
     TMP = os.path.join(SAGE_TESTDIR, '%s-%s' % (HOSTNAME, os.getpid()))
     TMP = os.path.abspath(TMP)
+    os.makedirs(TMP)
     try:
         os.makedirs(TMP)
     except OSError:

Naturally, this resulted in an error before the patch, success after the patch.