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

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
This commit is contained in:
Steve Dower 2021-09-20 15:33:00 +01:00 committed by GitHub
parent a856364cc9
commit ef9e22b253
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 18 additions and 1 deletions

View File

@ -3,6 +3,8 @@
import argparse import argparse
import os import os
import pathlib import pathlib
import sys
import time
import zipfile import zipfile
from urllib.request import urlretrieve from urllib.request import urlretrieve
@ -53,7 +55,22 @@ def main():
verbose=args.verbose, verbose=args.verbose,
) )
final_name = args.externals_dir / args.tag 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__': if __name__ == '__main__':