]> granicus.if.org Git - graphviz/commit
gvtest.py: fix: use os.remove instead of pathlib.unlink to remove a file
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 5 Sep 2021 22:56:27 +0000 (15:56 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 13 Sep 2021 14:45:59 +0000 (07:45 -0700)
commitfb175c2113900d7b63794a33020ceb046d96ab64
treef24405a31279adccc52ed9476a5287d32a78ca5d
parentb94bd8c04d629bbbab858dbf5421f0d65e57565e
gvtest.py: fix: use os.remove instead of pathlib.unlink to remove a file

There were two problems with using pathlib.unlink here:

  1. If the caller did not pass in a value for `dst`, it was assigned to the
     second return value of a `tempfile.mkstemp` call. That is, `dst` would be a
     string, not a `pathlib.Path`, and calling `.unlink()` would fail
     cryptically.

  2. The `missing_ok` parameter which this code was using was only added to
     `pathlib.unlink` in Python 3.8. The stated Python baseline for Graphviz
     currently is Python 3.6. If this code was reached with Python < 3.8 it
     would result in an exception, obscuring the original problem.
rtest/gvtest.py