Strip Xcode versions like "9.0gm" for the version flag.
Turns out, those exist.
RELNOTES: None.
"""Rules that allows select() to differentiate between Apple OS versions."""
+def _strip_version(version):
+ """Strip trailing characters from version names.
+
+ Some OS versions look like "9.0gm", which is not useful for select()
+ statements. Thus, we strip the trailing "gm" part.
+
+ Args:
+ version: the version string
+
+ Returns:
+ The version with trailing letters stripped.
+ """
+ result = ""
+ for ch in str(version):
+ if not ch.isdigit() and ch != ".":
+ break
+
+ result += ch
+
+ return result
+
+
def _xcode_version_flag_impl(ctx):
"""A rule that allows select() to differentiate between Xcode versions."""
xcode_config = ctx.attr._xcode_config[apple_common.XcodeVersionConfig]
-
return struct(providers = [
- config_common.FeatureFlagInfo(value = str(xcode_config.xcode_version()))])
+ config_common.FeatureFlagInfo(value = _strip_version(
+ xcode_config.xcode_version()))])
def _ios_sdk_version_flag_impl(ctx):
"""A rule that allows select() to select based on the iOS SDK version."""
xcode_config = ctx.attr._xcode_config[apple_common.XcodeVersionConfig]
- return struct(providers = [config_common.FeatureFlagInfo(value = str(
+ return struct(providers = [
+ config_common.FeatureFlagInfo(value = _strip_version(
xcode_config.sdk_version_for_platform(
apple_common.platform.ios_device)))])
@@ -35,7 +58,8 @@ def _tvos_sdk_version_flag_impl(ctx):
"""A rule that allows select() to select based on the tvOS SDK version."""
xcode_config = ctx.attr._xcode_config[apple_common.XcodeVersionConfig]
- return struct(providers = [config_common.FeatureFlagInfo(value = str(
+ return struct(providers = [
+ config_common.FeatureFlagInfo(value = _strip_version(
xcode_config.sdk_version_for_platform(
apple_common.platform.tvos_device)))])
@@ -44,7 +68,8 @@ def _watchos_sdk_version_flag_impl(ctx):
"""A rule that allows select() to select based on the watchOS SDK version."""
xcode_config = ctx.attr._xcode_config[apple_common.XcodeVersionConfig]
- return struct(providers = [config_common.FeatureFlagInfo(value = str(
+ return struct(providers = [
+ config_common.FeatureFlagInfo(value = _strip_version(
xcode_config.sdk_version_for_platform(
apple_common.platform.watchos_device)))])
@@ -53,7 +78,8 @@ def _macos_sdk_version_flag_impl(ctx):
"""A rule that allows select() to select based on the macOS SDK version."""
xcode_config = ctx.attr._xcode_config[apple_common.XcodeVersionConfig]
- return struct(providers = [config_common.FeatureFlagInfo(value = str(
+ return struct(providers = [
+ config_common.FeatureFlagInfo(value = _strip_version(
xcode_config.sdk_version_for_platform(
apple_common.platform.macos_device)))])
