Remove fixes for Windows release artifacts

Author: savannahostrowski

PCbuild/get_external.py

@@ -3,8 +3,8 @@
 import argparse
 import os
 import pathlib
+import shutil
 import sys
-import tarfile
 import time
 import urllib.error
 import urllib.request
@@ -56,8 +56,7 @@ def fetch_release(tag, tarball_dir, *, org='python', verbose=False):
 
 def extract_tarball(externals_dir, tarball_path, tag):
     output_path = externals_dir / tag
-    with tarfile.open(tarball_path) as tf:
-        tf.extractall(os.fspath(externals_dir))
+    shutil.unpack_archive(os.fspath(tarball_path), os.fspath(output_path))
     return output_path
 
 
@@ -116,23 +115,21 @@ def main():
             verbose=args.verbose,
         )
         extracted = extract_zip(args.externals_dir, zip_path)
-
-    if extracted != final_name:
-        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 rename {extracted} to {final_name}.",
-                "You may need to restart your build",
-                file=sys.stderr,
-            )
-            sys.exit(1)
+    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__':

Tools/jit/_llvm.py

@@ -83,11 +83,6 @@ async def _find_tool(tool: str, llvm_version: str, *, echo: bool = False) -> str
     # PCbuild externals:
     externals = os.environ.get("EXTERNALS_DIR", _targets.EXTERNALS)
     path = os.path.join(externals, _EXTERNALS_LLVM_TAG, "bin", tool)
-    # On Windows, executables need .exe extension
-    if os.name == "nt" and not path.endswith(".exe"):
-        path_with_exe = path + ".exe"
-        if os.path.exists(path_with_exe):
-            path = path_with_exe
     if await _check_tool_version(path, llvm_version, echo=echo):
         return path
     # Homebrew-installed executables: