remove platform_version and platform_release flags for now

Author: rickeylev

python/config_settings/BUILD.bazel

@@ -220,15 +220,3 @@ string_flag(
 define_pypi_internal_flags(
     name = "define_pypi_internal_flags",
 )
-
-label_flag(
-    name = "pip_platform_release_config",
-    build_setting_default = ":_pip_platform_release_default_config",
-    visibility = ["//visibility:public"],
-)
-
-label_flag(
-    name = "pip_platform_version_config",
-    build_setting_default = ":_pip_platform_version_default_config",
-    visibility = ["//visibility:public"],
-)

python/private/pypi/env_marker_setting.bzl

@@ -43,6 +43,10 @@ def env_marker_setting(*, name, expression, **kwargs):
         sys_platform = select(sys_platform_select_map),
         platform_machine = select(platform_machine_select_map),
         platform_system = select(platform_system_select_map),
+        platform_release = select({
+            "@platforms//os:osx": "USE_OSX_VERSION_FLAG",
+            "//conditions:default": "",
+        }),
         **kwargs
     )
 
@@ -90,9 +94,11 @@ def _env_marker_setting_impl(ctx):
     # https://peps.python.org/pep-0738/#platform
     # Similar for iOS:
     # https://peps.python.org/pep-0730/#platform
-    env["platform_release"] = _get_flag(ctx.attr._platform_release_config_flag)
+    env["platform_release"] = ctx.attr.platform_release
     env["platform_system"] = ctx.attr.platform_system
-    env["platform_version"] = _get_flag(ctx.attr._platform_version_config_flag)
+
+    # For lack of a better option, just use an empty string for now.
+    env["platform_version"] = ""
 
     env.update(env_aliases())
 
@@ -118,16 +124,9 @@ for the specification of behavior.
         ),
         "os_name": attr.string(),
         "platform_machine": attr.string(),
+        "platform_release": attr.string(),
         "platform_system": attr.string(),
         "sys_platform": attr.string(),
-        "_platform_release_config_flag": attr.label(
-            default = "//python/config_settings:pip_platform_release_config",
-            providers = [[config_common.FeatureFlagInfo], [BuildSettingInfo]],
-        ),
-        "_platform_version_config_flag": attr.label(
-            default = "//python/config_settings:pip_platform_version_config",
-            providers = [[config_common.FeatureFlagInfo], [BuildSettingInfo]],
-        ),
         "_python_full_version_flag": attr.label(
             default = "//python/config_settings:python_version",
             providers = [config_common.FeatureFlagInfo],

python/private/pypi/flags.bzl

@@ -81,18 +81,6 @@ def define_pypi_internal_flags(name):
         name = "_internal_pip_whl",
         visibility = ["//visibility:public"],
     )
-    _platform_release_config(
-        name = "_pip_platform_release_default_config",
-        value = select({
-            "@platforms//os:osx": "USE_OSX_VERSION_FLAG",
-            "//conditions:default": "",
-        }),
-        visibility = ["//visibility:public"],
-    )
-    _platform_version_config(
-        name = "_pip_platform_version_default_config",
-        visibility = ["//visibility:public"],
-    )
 
 def _allow_wheels_flag_impl(ctx):
     input = ctx.attr._setting[BuildSettingInfo].value
@@ -109,32 +97,3 @@ This rule allows us to greatly reduce the number of config setting targets at no
 if we are duplicating some of the functionality of the `native.config_setting`.
 """,
 )
-
-def _platform_release_config_impl(ctx):
-    value = ctx.attr.value
-    if value == "USE_OSX_VERSION_FLAG":
-        value = ctx.attr._osx_version[BuildSettingInfo].value
-
-    return [BuildSettingInfo(value = ctx.attr.value)]
-
-# This value is loosely some version-value looking thing, but the format
-# varies depending on the OS.
-_platform_release_config = rule(
-    implementation = _platform_release_config_impl,
-    attrs = {
-        "value": attr.string(),
-        "_osx_version": attr.label(
-            default = "//python/config_settings:pip_whl_osx_version",
-        ),
-    },
-)
-
-def _platform_version_config_impl(ctx):
-    _ = ctx  # @unused
-    return [BuildSettingInfo(value = "")]
-
-# Despite its name, this "version" value is not a simple version value.
-# It's a more detailed, arbitrary, description the OS gives about itself.
-_platform_version_config = rule(
-    implementation = _platform_version_config_impl,
-)