diff options
author | Alex Humesky <ahumesky@google.com> | 2021-03-18 20:28:36 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-03-18 20:28:36 -0400 |
commit | 87690250441364c8dfab01b2f4ee6fe67a44277a (patch) | |
tree | a24a93acc44655fa11884ea9a3a4356ad56ac242 | |
parent | bd8fc648167ff573f1415a80cc4a72c98fc62335 (diff) | |
download | bazelbuild-rules_android-87690250441364c8dfab01b2f4ee6fe67a44277a.tar.gz |
Fix up visibility and some labels to make the android rules more embeddable into a repository. (#35)
-rw-r--r-- | toolchains/android/BUILD | 7 | ||||
-rw-r--r-- | toolchains/android_sdk/BUILD | 5 | ||||
-rw-r--r-- | tools/android/BUILD | 14 | ||||
-rw-r--r-- | tools/android/defs.bzl | 15 |
4 files changed, 37 insertions, 4 deletions
diff --git a/toolchains/android/BUILD b/toolchains/android/BUILD index 6c731a9..3741ee8 100644 --- a/toolchains/android/BUILD +++ b/toolchains/android/BUILD @@ -12,7 +12,10 @@ filegroup( ) # Android Toolchain Type -toolchain_type(name = "toolchain_type") +toolchain_type( + name = "toolchain_type", + visibility = ["//visibility:public"], +) # Default Android Toolchain android_toolchain( @@ -44,9 +47,11 @@ EOF sh_binary( name = "zip", srcs = [":zip.sh"], + visibility = ["//visibility:public"], ) sh_binary( name = "unzip", srcs = [":unzip.sh"], + visibility = ["//visibility:public"], ) diff --git a/toolchains/android_sdk/BUILD b/toolchains/android_sdk/BUILD index 10e40ca..d675cbe 100644 --- a/toolchains/android_sdk/BUILD +++ b/toolchains/android_sdk/BUILD @@ -9,7 +9,10 @@ filegroup( ) # Android SDK Toolchain Type -toolchain_type(name = "toolchain_type") +toolchain_type( + name = "toolchain_type", + visibility = ["//visibility:public"], +) toolchain( name = "android_sdk_tools", diff --git a/tools/android/BUILD b/tools/android/BUILD index c0f554c..7c33f67 100644 --- a/tools/android/BUILD +++ b/tools/android/BUILD @@ -8,6 +8,18 @@ android_jar( # TODO(b/175833857): This is a stub, should remove. alias( name = "merge_manifests", - actual = "@androidsdk//:fail", + actual = ":fail", visibility = ["//visibility:public"], ) + +genrule( + name = "gen_fail", + outs = ["fail.sh"], + cmd = "echo 'exit 1' > $@", + executable = 1, +) + +sh_binary( + name = "fail", + srcs =[":fail.sh"], +) diff --git a/tools/android/defs.bzl b/tools/android/defs.bzl index 6aa1896..07c7a13 100644 --- a/tools/android/defs.bzl +++ b/tools/android/defs.bzl @@ -21,7 +21,20 @@ def _android_jar_impl(ctx): android_jar = rule( implementation = _android_jar_impl, + # TODO: Should use a toolchain instead of a configuration_field on + # --android_sdk as below, however that appears to be broken when used + # from an local_repository: b/183060658. + #toolchains = [ + # "//toolchains/android_sdk:toolchain_type", + #], attrs = { - "_sdk": attr.label(default = "@androidsdk//:sdk"), + "_sdk": attr.label( + allow_rules = ["android_sdk"], + default = configuration_field( + fragment = "android", + name = "android_sdk_label", + ), + providers = [AndroidSdkInfo], + ), }, ) |