comment: use a constant

Author: aignas

python/private/python_repository.bzl

@@ -15,7 +15,7 @@
 """This file contains repository rules and macros to support toolchain registration.
 """
 
-load("//python:versions.bzl", "PLATFORMS")
+load("//python:versions.bzl", "FREETHREADED", "PLATFORMS")
 load(":auth.bzl", "get_auth")
 load(":repo_utils.bzl", "REPO_DEBUG_ENV_VAR", "repo_utils")
 load(":text_util.bzl", "render")
@@ -64,7 +64,7 @@ def _python_repository_impl(rctx):
     python_version = rctx.attr.python_version
     python_version_info = python_version.split(".")
     release_filename = rctx.attr.release_filename
-    version_suffix = "t" if "freethreaded" in release_filename else ""
+    version_suffix = "t" if FREETHREADED in release_filename else ""
     python_short_version = "{0}.{1}{suffix}".format(
         suffix = version_suffix,
         *python_version_info

python/versions.bzl

@@ -724,11 +724,11 @@ def _generate_platforms():
             os_name = v.os_name,
             arch = v.arch,
         )
+        for p, v in platforms.items()
         for suffix, freethreaded_value in {
             "": "no",
-            "-freethreaded": "yes",
+            "-" + FREETHREADED: "yes",
         }.items()
-        for p, v in platforms.items()
     }
 
 PLATFORMS = _generate_platforms()
@@ -761,18 +761,21 @@ def get_release_info(platform, python_version, base_url = DEFAULT_RELEASE_BASE_U
     release_filename = None
     rendered_urls = []
     for u in url:
-        p, _, _ = platform.partition("-freethreaded")
-
-        if "freethreaded" in platform:
-            build = "freethreaded+{}-full".format({
-                "aarch64-apple-darwin": "pgo+lto",
-                "aarch64-unknown-linux-gnu": "lto",
-                "ppc64le-unknown-linux-gnu": "lto",
-                "s390x-unknown-linux-gnu": "lto",
-                "x86_64-apple-darwin": "pgo+lto",
-                "x86_64-pc-windows-msvc": "pgo",
-                "x86_64-unknown-linux-gnu": "pgo+lto",
-            }[p])
+        p, _, _ = platform.partition("-" + FREETHREADED)
+
+        if FREETHREADED in platform:
+            build = "{}+{}-full".format(
+                FREETHREADED,
+                {
+                    "aarch64-apple-darwin": "pgo+lto",
+                    "aarch64-unknown-linux-gnu": "lto",
+                    "ppc64le-unknown-linux-gnu": "lto",
+                    "s390x-unknown-linux-gnu": "lto",
+                    "x86_64-apple-darwin": "pgo+lto",
+                    "x86_64-pc-windows-msvc": "pgo",
+                    "x86_64-unknown-linux-gnu": "pgo+lto",
+                }[p],
+            )
         else:
             build = "install_only"