matthias/mayan-edms
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Do a copy,delete instead of a rename

Browse Source 
os.rename doesn't work across multiple filesystems.

Signed-off-by: Roberto Rosario <roberto.rosario.gonzalez@gmail.com>
This commit is contained in:
Roberto Rosario
2019-05-05 18:32:52 -04:00
parent ce70701b7e
commit 251b4bac40
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
mayan/apps/dependencies/classes.py
View File

@@ -433,9 +433,14 @@ class JavaScriptDependency(Dependency):
# Copy the content under the dependency's extracted content folder
# 'package' to the final location.
Path(temporary_directory, 'package').rename(
target=force_text(path_install)
# We do a copy and delete instead of move because os.rename doesn't
# support renames across filesystems.
path_uncompressed_package = Path(temporary_directory, 'package')
shutil.rmtree(force_text(path_install))
shutil.copytree(
force_text(path_uncompressed_package), force_text(path_install)
)
shutil.rmtree(force_text(path_uncompressed_package))
# Clean up temporary directory used for download
shutil.rmtree(path=temporary_directory, ignore_errors=True)