remove windows-only error, see #50

purarue · Sep 8, 2023 · d9e7fb2 · d9e7fb2
1 parent ece2c4b
commit d9e7fb2
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/browserexport/parse.py b/browserexport/parse.py
@@ -105,9 +105,6 @@ def _read_buf_as_sqlite_db(buf: BinaryIO) -> sqlite3.Connection:
         # once the tempfile is deleted, we can still access the data
         dbin.backup(dbout)
 
-        # needed for windows to be able to delete the folder
-        tfp.unlink(missing_ok=True)
-
     assert not tfp.exists(), f"tempfile {tfp} should be deleted, but still exists"
     return dbout
 

diff --git a/browserexport/save.py b/browserexport/save.py
@@ -68,8 +68,6 @@ def _path_backup(
             _sqlite_backup(srcp, tfp)
             _print_sqlite_db_to_stdout(tfp)
 
-            # needed for windows to be able to delete the folder
-            tfp.unlink(missing_ok=True)
         assert not tfp.exists(), f"expected {tfp} to be deleted, but it still exists!"
         return None
     else: