BRNSystems/minetest
1
0
Fork 0
forked from Mirrorlandia_minetest/minetest
Code Pull Requests 1 Activity

Fix crash when trying to overwrite a package

Browse Source 
Before #11646, core.copy_dir would overwrite the target if it exists. Adding core.delete_dir restores the exact same behaviour

Fixes #12303
This commit is contained in:
rubenwardy 2022-08-15 08:30:46 +01:00 committed by sfan5
parent b6db2c7262
commit 50df5e2f59
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
builtin/mainmenu/pkgmgr.lua
View File

@ -608,11 +608,10 @@ function pkgmgr.install_dir(type, path, basename, targetpath)
end
local from = basefolder and basefolder.path or path
if targetpath then
core.delete_dir(targetpath)
else
if not targetpath then
targetpath = core.get_texturepath() .. DIR_DELIM .. basename
end
core.delete_dir(targetpath)
if not core.copy_dir(from, targetpath, false) then
return nil,
fgettext("Failed to install $1 to $2", basename, targetpath)
@ -690,6 +689,7 @@ function pkgmgr.install_dir(type, path, basename, targetpath)
end
-- Copy it
core.delete_dir(targetpath)
if not core.copy_dir(basefolder.path, targetpath, false) then
return nil,
fgettext("Failed to install $1 to $2", basename, targetpath)