Skip to content

Support all Android platforms version syntax, not just int version,… #433

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
anlexN wants to merge 1 commit into
base: main
Choose a base branch
from
+44 −44
Open

Support all Android platforms version syntax, not just int version,… #433

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
72 changes: 36 additions & 36 deletions rules/android_sdk_repository/helper.bzl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,27 +26,27 @@ _bool_flag = rule(
build_setting = config.bool(flag = True),
)

def _int_flag_impl(ctx):
def _string_flag_impl(ctx):
allowed_values = ctx.attr.values
value = ctx.build_setting_value
if len(allowed_values) != 0 and value not in ctx.attr.values:
fail(
"Error setting %s: invalid value '%d'. Allowed values are %s" % (
"Error setting %s: invalid value '%s'. Allowed values are %s" % (
ctx.label,
value,
",".join([str(val) for val in allowed_values]),
),
)

int_flag = rule(
implementation = _int_flag_impl,
build_setting = config.int(flag = True),
string_flag = rule(
implementation = _string_flag_impl,
build_setting = config.string(flag = True),
attrs = {
"values": attr.int_list(
"values": attr.string_list(
doc = "The list of allowed values for this setting. An error is raised if any other value is given.",
),
},
doc = "An int-typed build setting that can be set on the command line",
doc = "An string-typed build setting that can be set on the command line",
)

def _create_config_setting_rule():
Expand Down Expand Up @@ -94,15 +94,15 @@ def create_android_sdk_rules(
build_tools_version: string, the version of Android's build tools to use.
build_tools_directory: string, the directory name of the build tools in
sdk's build-tools directory.
api_levels: list of ints, the API levels from which to get android.jar
api_levels: list of strings, the API levels from which to get android.jar
et al. and create android_sdk rules.
default_api_level: int, the API level to alias the default sdk to if
default_api_level: string, the API level to alias the default sdk to if
--android_sdk is not specified on the command line.
"""

_create_config_setting_rule()

int_flag(
string_flag(
name = "api_level",
build_setting_default = default_api_level,
values = api_levels,
Expand Down Expand Up @@ -154,7 +154,7 @@ def create_android_sdk_rules(
"build-tools/%s/lib/d8.jar" % build_tools_directory,
":build_tools_libs",
] + [
"platforms/android-%d/%s" % (api_level, filename)
"platforms/android-%s/%s" % (api_level, filename)
for api_level in api_levels
for filename in ["android.jar", "core-for-system-modules.jar", "framework.aidl"]
] + select({
Expand All @@ -164,44 +164,44 @@ def create_android_sdk_rules(
)

for api_level in api_levels:
if api_level >= 23:
if api_level >= "23":
# Android 23 removed most of org.apache.http from android.jar and moved it
# to a separate jar.
java_import(
name = "org_apache_http_legacy-%d" % api_level,
jars = ["platforms/android-%d/optional/org.apache.http.legacy.jar" % api_level],
name = "org_apache_http_legacy-%s" % api_level,
jars = ["platforms/android-%s/optional/org.apache.http.legacy.jar" % api_level],
)

if api_level >= 28:
if api_level >= "28":
# Android 28 removed most of android.test from android.jar and moved it
# to separate jars.
java_import(
name = "legacy_test-%d" % api_level,
name = "legacy_test-%s" % api_level,
jars = [
"platforms/android-%d/optional/android.test.base.jar" % api_level,
"platforms/android-%d/optional/android.test.mock.jar" % api_level,
"platforms/android-%d/optional/android.test.runner.jar" % api_level,
"platforms/android-%s/optional/android.test.base.jar" % api_level,
"platforms/android-%s/optional/android.test.mock.jar" % api_level,
"platforms/android-%s/optional/android.test.runner.jar" % api_level,
],
neverlink = 1,
)

if api_level >= 29:
if api_level >= "29":
# Android 29 is min api that compatible with Car App Library
java_import(
name = "android_car-%d" % api_level,
jars = ["platforms/android-%d/optional/android.car.jar" % api_level],
name = "android_car-%s" % api_level,
jars = ["platforms/android-%s/optional/android.car.jar" % api_level],
neverlink = 1,
)

native.config_setting(
name = "api_%d_enabled" % api_level,
name = "api_%s_enabled" % api_level,
flag_values = {
":api_level": str(api_level),
},
)

android_sdk(
name = "sdk-%d" % api_level,
name = "sdk-%s" % api_level,
aapt = select({
":windows": "build-tools/%s/aapt.exe" % build_tools_directory,
"//conditions:default": ":aapt_binary",
Expand All @@ -218,15 +218,15 @@ def create_android_sdk_rules(
":windows": "build-tools/%s/aidl.exe" % build_tools_directory,
"//conditions:default": ":aidl_binary",
}),
android_jar = "platforms/android-%d/android.jar" % api_level,
android_jar = "platforms/android-%s/android.jar" % api_level,
apksigner = ":apksigner",
build_tools_version = build_tools_version,
dx = select({
"d8_standalone_dexer": ":d8_compat_dx",
"dx_standalone_dexer": ":dx_binary",
"//conditions:default": ":d8_compat_dx",
}),
framework_aidl = "platforms/android-%d/framework.aidl" % api_level,
framework_aidl = "platforms/android-%s/framework.aidl" % api_level,
legacy_main_dex_list_generator = ":generate_main_dex_list",
main_dex_classes = "build-tools/%s/mainDexClasses.rules" % build_tools_directory,
main_dex_list_creator = ":main_dex_list_creator",
Expand All @@ -243,34 +243,34 @@ def create_android_sdk_rules(
)

native.toolchain(
name = "sdk-%d-toolchain" % api_level,
name = "sdk-%s-toolchain" % api_level,
exec_compatible_with = HOST_CONSTRAINTS,
toolchain = ":sdk-%d" % api_level,
toolchain = ":sdk-%s" % api_level,
toolchain_type = ":sdk_toolchain_type",
target_settings = [
":api_%d_enabled" % api_level,
":api_%s_enabled" % api_level,
],
)

create_dummy_sdk_toolchain()

if default_api_level >= 29:
if default_api_level >= "29":
native.alias(
name = "android_car",
actual = "android_car-%d" % default_api_level,
actual = "android_car-%s" % default_api_level,
)

native.alias(
name = "org_apache_http_legacy",
actual = ":org_apache_http_legacy-%d" % default_api_level,
actual = ":org_apache_http_legacy-%s" % default_api_level,
)

sdk_alias_dict = {
"//conditions:default": "sdk-%d" % default_api_level,
"//conditions:default": "sdk-%s" % default_api_level,
}

for api_level in api_levels:
sdk_alias_dict[":api_%d_enabled" % api_level] = "sdk-%d" % api_level
sdk_alias_dict[":api_%s_enabled" % api_level] = "sdk-%s" % api_level

native.alias(
name = "sdk",
Expand All @@ -282,12 +282,12 @@ def create_android_sdk_rules(

native.alias(
name = "sdk-toolchain",
actual = ":sdk-%d-toolchain" % default_api_level,
actual = ":sdk-%s-toolchain" % default_api_level,
)

java_import(
name = "core-for-system-modules-jar",
jars = ["platforms/android-%d/core-for-system-modules.jar" % default_api_level],
jars = ["platforms/android-%s/core-for-system-modules.jar" % default_api_level],
)

java_binary(
Expand Down
12 changes: 6 additions & 6 deletions rules/android_sdk_repository/rule.bzl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,8 @@ def _read_api_levels(repo_ctx, android_sdk_path):
name = entry.basename
if name.startswith("android-"):
level = name[len("android-"):]
if level.isdigit():
api_levels.append(int(level))
if len(level) > 0:
api_levels.append(level)
return api_levels

def _newest_build_tools(repo_ctx, android_sdk_path):
Expand Down Expand Up @@ -116,7 +116,7 @@ def _android_sdk_repository_impl(repo_ctx):
# Determine default SDK level.
default_api_level = max(api_levels)
if repo_ctx.attr.api_level:
default_api_level = int(repo_ctx.attr.api_level)
default_api_level = repo_ctx.attr.api_level
if default_api_level not in api_levels:
fail("Android SDK api level %s was requested but it is not installed in the Android SDK at %s. The api levels found were %s. Please choose an available api level or install api level %s from the Android SDK Manager." % (
default_api_level,
Expand Down Expand Up @@ -167,7 +167,7 @@ def _android_sdk_repository_impl(repo_ctx):
_android_sdk_repository = repository_rule(
implementation = _android_sdk_repository_impl,
attrs = {
"api_level": attr.int(default = 0),
"api_level": attr.string(default = ""),
"build_tools_version": attr.string(),
"path": attr.string(),
},
Expand All @@ -178,7 +178,7 @@ _android_sdk_repository = repository_rule(
def android_sdk_repository(
name,
path = "",
api_level = 0,
api_level = "",
build_tools_version = ""):
"""Create a repository with Android SDK toolchains.

Expand Down Expand Up @@ -228,7 +228,7 @@ android_sdk_repository_extension = module_extension(
tag_classes = {
"configure": tag_class(attrs = {
"path": attr.string(),
"api_level": attr.int(),
"api_level": attr.string(),
"build_tools_version": attr.string(),
}),
},
Expand Down
4 changes: 2 additions & 2 deletions rules/android_sdk_repository/template.bzl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,8 @@ create_android_sdk_rules(
name = "__repository_name__",
build_tools_version = "__build_tools_version__",
build_tools_directory = "__build_tools_directory__",
api_levels = [__api_levels__],
default_api_level = __default_api_level__,
api_levels = str(__api_levels__).split(","),
default_api_level = str(__default_api_level__),
)

alias(
Expand Down