diff --git a/kiwixbuild/dependencies/base.py b/kiwixbuild/dependencies/base.py index dbe5ef2..a527478 100644 --- a/kiwixbuild/dependencies/base.py +++ b/kiwixbuild/dependencies/base.py @@ -131,8 +131,17 @@ class ReleaseDownload(Source): def _download(self, context): context.try_skip(neutralEnv("archive_dir"), self.full_name) - for archive in self.archives: - neutralEnv("download")(archive) + archive_iter = iter(self.archives) + archive = next(archive_iter, None) + while archive: + try: + neutralEnv("download")(archive) + except SkipCommand as e: + archive = next(archive_iter, None) + if not archive: + raise e + continue + archive = next(archive_iter, None) def _extract(self, context): context.try_skip(self.extract_path) diff --git a/kiwixbuild/utils.py b/kiwixbuild/utils.py index 8cce351..1c1eb14 100644 --- a/kiwixbuild/utils.py +++ b/kiwixbuild/utils.py @@ -275,7 +275,12 @@ def extract_archive(archive_path, dest_dir, topdir=None, name=None): perm = (member.external_attr >> 16) & 0x1FF os.chmod(pj(tmpdir, getname(member)), perm) name = name or topdir - os.rename(pj(tmpdir, topdir), pj(dest_dir, name)) + shutil.copytree(pj(tmpdir, topdir), pj(dest_dir, name), symlinks=True, dirs_exist_ok=True) + # Be sure that all directory in tmpdir are writable to allow correct suppersion of it + for root, dirs, _files in os.walk(tmpdir): + for d in dirs: + os.chmod(pj(root, d), stat.S_IWRITE|stat.S_IREAD|stat.S_IEXEC) + else: if name: dest_dir = pj(dest_dir, name)