[Python-checkins] bpo-45055: Add retry when downloading externals on Windows (GH-28399)

zooba webhook-mailer at python.org
Mon Sep 20 10:33:23 EDT 2021


https://github.com/python/cpython/commit/ef9e22b253253615098d22cb49141a2a1024ee3c
commit: ef9e22b253253615098d22cb49141a2a1024ee3c
branch: main
author: Steve Dower <steve.dower at python.org>
committer: zooba <steve.dower at microsoft.com>
date: 2021-09-20T15:33:00+01:00
summary:

bpo-45055: Add retry when downloading externals on Windows (GH-28399)

Co-authored-by: Łukasz Langa <lukasz at langa.pl>

files:
M PCbuild/get_external.py

diff --git a/PCbuild/get_external.py b/PCbuild/get_external.py
index a682d3849f14c..4ecc8925349c9 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__':



More information about the Python-checkins mailing list