]> granicus.if.org Git - graphviz/commitdiff
use shutil.rmtree in preference to pathlib unlink
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 24 Oct 2020 19:15:10 +0000 (12:15 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 31 Oct 2020 02:15:43 +0000 (19:15 -0700)
The missing_ok argument was only added to pathlib.Path.unlink in Python 3.8.
This made the cleanup function fail (sometimes silently) on versions of Python
prior to 3.8.

rtest/rtest.py

index f11ee78d207345d3e7fd8656f97807f1b39dfb6c..9bb2e5b57310708e032d5eea97dacd929edc484a 100755 (executable)
@@ -10,6 +10,7 @@
 import os
 import shutil
 import tempfile
+import shutil
 import subprocess
 import sys
 import platform
@@ -300,9 +301,9 @@ def doTest(TEST):
   TESTTYPES = {}
 
 def cleanup():
-  pathlib.Path(TMPFILE1).unlink(missing_ok=True)
-  pathlib.Path(TMPFILE2).unlink(missing_ok=True)
-  pathlib.Path(TMPINFILE).unlink(missing_ok=True)
+  shutil.rmtree(TMPFILE1, ignore_errors=True)
+  shutil.rmtree(TMPFILE2, ignore_errors=True)
+  shutil.rmtree(TMPINFILE, ignore_errors=True)
 atexit.register(cleanup)
 
 # Set REFDIR