summaryrefslogtreecommitdiffstats
path: root/PCbuild
diff options
context:
space:
mode:
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>2021-09-20 14:59:15 (GMT)
committerGitHub <noreply@github.com>2021-09-20 14:59:15 (GMT)
commit50c0551f97f86bbf9f659e1bbe78a14df7754cbe (patch)
tree42a502b8b8bf751e0aa4312cb74ab172fdaee178 /PCbuild
parent5822ab672a1d26ff1837103c1ed8e4c3c2a42b87 (diff)
downloadcpython-50c0551f97f86bbf9f659e1bbe78a14df7754cbe.zip
cpython-50c0551f97f86bbf9f659e1bbe78a14df7754cbe.tar.gz
cpython-50c0551f97f86bbf9f659e1bbe78a14df7754cbe.tar.bz2
bpo-45055: Add retry when downloading externals on Windows (GH-28399)
Co-authored-by: Ɓukasz Langa <lukasz@langa.pl> (cherry picked from commit ef9e22b253253615098d22cb49141a2a1024ee3c) Co-authored-by: Steve Dower <steve.dower@python.org>
Diffstat (limited to 'PCbuild')
-rwxr-xr-xPCbuild/get_external.py19
1 files changed, 18 insertions, 1 deletions
diff --git a/PCbuild/get_external.py b/PCbuild/get_external.py
index a682d38..4ecc892 100755
--- a/PCbuild/get_external.py
+++ b/PCbuild/get_external.py
@@ -3,6 +3,8 @@
import argparse
import os
import pathlib
+import sys
+import time
import zipfile
from urllib.request import urlretrieve
@@ -53,7 +55,22 @@ def main():
verbose=args.verbose,
)
final_name = args.externals_dir / args.tag
- extract_zip(args.externals_dir, zip_path).replace(final_name)
+ extracted = extract_zip(args.externals_dir, zip_path)
+ for wait in [1, 2, 3, 5, 8, 0]:
+ try:
+ extracted.replace(final_name)
+ break
+ except PermissionError as ex:
+ retry = f" Retrying in {wait}s..." if wait else ""
+ print(f"Encountered permission error '{ex}'.{retry}", file=sys.stderr)
+ time.sleep(wait)
+ else:
+ print(
+ f"ERROR: Failed to extract {final_name}.",
+ "You may need to restart your build",
+ file=sys.stderr,
+ )
+ sys.exit(1)
if __name__ == '__main__':