fixup the default_python_version setting

Author: aignas

python/private/pypi/generate_whl_library_build_bazel.bzl

@@ -45,13 +45,13 @@ whl_library_targets_from_requires(
 def generate_whl_library_build_bazel(
         *,
         annotation = None,
-        python_version = None,
+        default_python_version = None,
         **kwargs):
     """Generate a BUILD file for an unzipped Wheel
 
     Args:
         annotation: The annotation for the build file.
-        python_version: The python version to use to parse the METADATA.
+        default_python_version: The python version to use to parse the METADATA.
         **kwargs: Extra args serialized to be passed to the
             {obj}`whl_library_targets`.
 
@@ -80,8 +80,8 @@ def generate_whl_library_build_bazel(
         kwargs["srcs_exclude"] = annotation.srcs_exclude_glob
         if annotation.additive_build_content:
             additional_content.append(annotation.additive_build_content)
-    if python_version:
-        kwargs["python_version"] = repr(python_version)
+    if default_python_version:
+        kwargs["default_python_version"] = default_python_version
 
     contents = "\n".join(
         [

python/private/pypi/pep508_deps.bzl

@@ -22,7 +22,7 @@ load(":pep508_evaluate.bzl", "evaluate")
 load(":pep508_platform.bzl", "platform", "platform_from_str")
 load(":pep508_requirement.bzl", "requirement")
 
-def deps(name, *, requires_dist, platforms = [], extras = [], excludes = [], host_python_version = None):
+def deps(name, *, requires_dist, platforms = [], extras = [], excludes = [], default_python_version = None):
     """Parse the RequiresDist from wheel METADATA
 
     Args:
@@ -32,7 +32,7 @@ def deps(name, *, requires_dist, platforms = [], extras = [], excludes = [], hos
         excludes: {type}`list[str]` what packages should we exclude.
         extras: {type}`list[str]` the requested extras to generate targets for.
         platforms: {type}`list[str]` the list of target platform strings.
-        host_python_version: {type}`str` the host python version.
+        default_python_version: {type}`str` the host python version.
 
     Returns:
         A struct with attributes:
@@ -52,15 +52,15 @@ def deps(name, *, requires_dist, platforms = [], extras = [], excludes = [], hos
     # drop self edges
     excludes = [name] + [normalize_name(x) for x in excludes]
 
-    host_python_version = host_python_version or DEFAULT_PYTHON_VERSION
+    default_python_version = default_python_version or DEFAULT_PYTHON_VERSION
     platforms = [
-        platform_from_str(p, python_version = host_python_version)
+        platform_from_str(p, python_version = default_python_version)
         for p in platforms
     ]
 
     abis = sorted({p.abi: True for p in platforms if p.abi})
-    if host_python_version and len(abis) > 1:
-        _, _, minor_version = host_python_version.partition(".")
+    if default_python_version and len(abis) > 1:
+        _, _, minor_version = default_python_version.partition(".")
         minor_version, _, _ = minor_version.partition(".")
         default_abi = "cp3" + minor_version
     elif len(abis) > 1:

python/private/pypi/whl_library.bzl

@@ -387,7 +387,7 @@ def _whl_library_impl(rctx):
     if BZLMOD_ENABLED:
         # The following attributes are unset on bzlmod and we pass data through
         # the hub via load statements.
-        python_version = None
+        default_python_version = None
         target_platforms = []
     else:
         # NOTE @aignas 2025-04-16: if BZLMOD_ENABLED, we should use
@@ -397,9 +397,9 @@ def _whl_library_impl(rctx):
         # `pip_parse` invocation, so we will have the host target platform
         # only. Even if somebody would change the code to support
         # `experimental_target_platforms`, they would be for a single python
-        # version. Hence, using the `python_version` that we get from the
+        # version. Hence, using the `default_python_version` that we get from the
         # interpreter is correct. Hence, we unset the argument if we are on bzlmod.
-        python_version = metadata["python_version"]
+        default_python_version = metadata["python_version"]
         target_platforms = rctx.attr.experimental_target_platforms or [host_platform(rctx)]
 
     metadata = whl_metadata(
@@ -416,7 +416,7 @@ def _whl_library_impl(rctx):
         dep_template = rctx.attr.dep_template or "@{}{{name}}//:{{target}}".format(rctx.attr.repo_prefix),
         entry_points = entry_points,
         target_platforms = target_platforms,
-        python_version = python_version,
+        default_python_version = default_python_version,
         # TODO @aignas 2025-04-14: load through the hub:
         annotation = None if not rctx.attr.annotation else struct(**json.decode(rctx.read(rctx.attr.annotation))),
         data_exclude = rctx.attr.pip_data_exclude,

python/private/pypi/whl_library_targets.bzl

@@ -41,7 +41,7 @@ def whl_library_targets_from_requires(
         requires_dist = [],
         extras = [],
         target_platforms = [],
-        python_version = None,
+        default_python_version = None,
         group_deps = [],
         **kwargs):
     """The macro to create whl targets from the METADATA.
@@ -59,14 +59,14 @@ def whl_library_targets_from_requires(
         extras: {type}`list[str]` The list of requested extras. This essentially includes extra transitive dependencies in the final targets depending on the wheel `METADATA`.
         target_platforms: {type}`list[str]` The list of target platforms to create
             dependency closures for.
-        python_version: {type}`str` The python version to assume when parsing
+        default_python_version: {type}`str` The python version to assume when parsing
             the `METADATA`. This is only used when the `target_platforms` do not
             include the version information.
         **kwargs: Extra args passed to the {obj}`whl_library_targets`
     """
     package_deps = _parse_requires_dist(
         name = name,
-        python_version = python_version,
+        default_python_version = default_python_version,
         requires_dist = requires_dist,
         excludes = group_deps,
         extras = extras,
@@ -86,7 +86,7 @@ def whl_library_targets_from_requires(
 def _parse_requires_dist(
         *,
         name,
-        python_version,
+        default_python_version,
         requires_dist,
         excludes,
         extras,
@@ -110,7 +110,7 @@ def _parse_requires_dist(
         platforms = target_platforms,
         excludes = excludes,
         extras = extras,
-        host_python_version = python_version,
+        default_python_version = default_python_version,
     )
 
 def whl_library_targets(