Skip to content

Checking and fixing rights while deleting the temporary directory on Windows #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
0xfoudy wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Checking and fixing rights while deleting the temporary directory on Windows #7

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
844bdb7
fix rights to file access in case an access error occurs while removi…
0xfoudy Mar 28, 2023
9f730b1
onerror deprecated, replaced by onexc
0xfoudy Mar 28, 2023
fb00d51
reverting last commit, onexc only available starting 3.12
0xfoudy Mar 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion eth_vertigo/interfaces/common/tester.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@
from tempfile import mkdtemp
from distutils.dir_util import copy_tree
import shutil
import os
import stat

from eth_vertigo.core import Mutation
from eth_vertigo.test_runner.file_editor import FileEditor
Expand Down Expand Up @@ -59,7 +61,14 @@ def clean_build_directory(project_path: str, build_directory: str = "build"):


def rm_temp_directory(temp_dir: str):
shutil.rmtree(temp_dir)
shutil.rmtree(temp_dir, onerror = redo_with_write)

def redo_with_write(redo_func, path, err):
if not os.access(path, os.W_OK):
os.chmod(path, stat.S_IWRITE)
redo_func(path)
else:
raise


def apply_mutation(mutation: Mutation, working_directory):
Expand Down