Rename tempd to wheel_directory for clarity

Author: sbidoul

src/pip/_internal/operations/build/wheel.py

@@ -14,25 +14,25 @@ def build_wheel_pep517(
     name: str,
     backend: BuildBackendHookCaller,
     metadata_directory: str,
-    tempd: str,
+    wheel_directory: str,
 ) -> str | None:
     """Build one InstallRequirement using the PEP 517 build process.
 
     Returns path to wheel if successfully built. Otherwise, returns None.
     """
     assert metadata_directory is not None
     try:
-        logger.debug("Destination directory: %s", tempd)
+        logger.debug("Destination directory: %s", wheel_directory)
 
         runner = runner_with_spinner_message(
             f"Building wheel for {name} (pyproject.toml)"
         )
         with backend.subprocess_runner(runner):
             wheel_name = backend.build_wheel(
-                wheel_directory=tempd,
+                wheel_directory=wheel_directory,
                 metadata_directory=metadata_directory,
             )
     except Exception:
         logger.error("Failed building wheel for %s", name)
         return None
-    return os.path.join(tempd, wheel_name)
+    return os.path.join(wheel_directory, wheel_name)

src/pip/_internal/operations/build/wheel_editable.py

@@ -14,23 +14,23 @@ def build_wheel_editable(
     name: str,
     backend: BuildBackendHookCaller,
     metadata_directory: str,
-    tempd: str,
+    wheel_directory: str,
 ) -> str | None:
     """Build one InstallRequirement using the PEP 660 build process.
 
     Returns path to wheel if successfully built. Otherwise, returns None.
     """
     assert metadata_directory is not None
     try:
-        logger.debug("Destination directory: %s", tempd)
+        logger.debug("Destination directory: %s", wheel_directory)
 
         runner = runner_with_spinner_message(
             f"Building editable for {name} (pyproject.toml)"
         )
         with backend.subprocess_runner(runner):
             try:
                 wheel_name = backend.build_editable(
-                    wheel_directory=tempd,
+                    wheel_directory=wheel_directory,
                     metadata_directory=metadata_directory,
                 )
             except HookMissing as e:
@@ -44,4 +44,4 @@ def build_wheel_editable(
     except Exception:
         logger.error("Failed building editable for %s", name)
         return None
-    return os.path.join(tempd, wheel_name)
+    return os.path.join(wheel_directory, wheel_name)

src/pip/_internal/operations/build/wheel_legacy.py

@@ -33,7 +33,7 @@ def format_command_result(
 
 def get_legacy_build_wheel_path(
     names: list[str],
-    temp_dir: str,
+    wheel_directory: str,
     name: str,
     command_args: list[str],
     command_output: str,
@@ -55,7 +55,7 @@ def get_legacy_build_wheel_path(
         msg += format_command_result(command_args, command_output)
         logger.warning(msg)
 
-    return os.path.join(temp_dir, names[0])
+    return os.path.join(wheel_directory, names[0])
 
 
 def build_wheel_legacy(
@@ -64,7 +64,7 @@ def build_wheel_legacy(
     source_dir: str,
     global_options: list[str],
     build_options: list[str],
-    tempd: str,
+    wheel_directory: str,
 ) -> str | None:
     """Build one unpacked package using the "legacy" build process.
 
@@ -89,12 +89,12 @@ def build_wheel_legacy(
         setup_py_path,
         global_options=global_options,
         build_options=build_options,
-        destination_dir=tempd,
+        destination_dir=wheel_directory,
     )
 
     spin_message = f"Building wheel for {name} (setup.py)"
     with open_spinner(spin_message) as spinner:
-        logger.debug("Destination directory: %s", tempd)
+        logger.debug("Destination directory: %s", wheel_directory)
 
         try:
             output = call_subprocess(
@@ -108,10 +108,10 @@ def build_wheel_legacy(
             logger.error("Failed building wheel for %s", name)
             return None
 
-        names = os.listdir(tempd)
+        names = os.listdir(wheel_directory)
         wheel_path = get_legacy_build_wheel_path(
             names=names,
-            temp_dir=tempd,
+            wheel_directory=wheel_directory,
             name=name,
             command_args=wheel_args,
             command_output=output,

src/pip/_internal/wheel_builder.py

@@ -188,7 +188,7 @@ def _build_one_inside_env(
     global_options: list[str],
     editable: bool,
 ) -> str | None:
-    with TemporaryDirectory(dir=output_dir) as temp_dir:
+    with TemporaryDirectory(dir=output_dir) as wheel_directory:
         assert req.name
         if req.use_pep517:
             assert req.metadata_directory
@@ -206,14 +206,14 @@ def _build_one_inside_env(
                     name=req.name,
                     backend=req.pep517_backend,
                     metadata_directory=req.metadata_directory,
-                    tempd=temp_dir,
+                    wheel_directory=wheel_directory,
                 )
             else:
                 wheel_path = build_wheel_pep517(
                     name=req.name,
                     backend=req.pep517_backend,
                     metadata_directory=req.metadata_directory,
-                    tempd=temp_dir,
+                    wheel_directory=wheel_directory,
                 )
         else:
             wheel_path = build_wheel_legacy(
@@ -222,7 +222,7 @@ def _build_one_inside_env(
                 source_dir=req.unpacked_source_directory,
                 global_options=global_options,
                 build_options=build_options,
-                tempd=temp_dir,
+                wheel_directory=wheel_directory,
             )
 
         if wheel_path is not None:

tests/unit/test_wheel.py

@@ -45,7 +45,7 @@ def call_get_legacy_build_wheel_path(
 ) -> str | None:
     wheel_path = get_legacy_build_wheel_path(
         names=names,
-        temp_dir="/tmp/abcd",
+        wheel_directory="/tmp/abcd",
         name="pendulum",
         command_args=["arg1", "arg2"],
         command_output="output line 1\noutput line 2\n",