Skip to content

Commit cf47cc7

Browse files
committed
DO NOT SUBMIT
Delete fully rolled out ACLs
1 parent c04558f commit cf47cc7

File tree

3 files changed

+4
-43
lines changed
  • rules
  • test/rules/android_local_test/java/com/starlark_resources

3 files changed

+4
-43
lines changed

rules/acls.bzl

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,8 @@ load("//rules/acls:android_binary_resource_name_obfuscation_opt_out_allowlist.bz
4444
load("//rules/acls:android_binary_starlark_dex_desugar_proguard.bzl", "ANDROID_BINARY_STARLARK_DEX_DESUGAR_PROGUARD_FALLBACK", "ANDROID_BINARY_STARLARK_DEX_DESUGAR_PROGUARD_ROLLOUT")
4545
load("//rules/acls:android_binary_starlark_javac.bzl", "ANDROID_BINARY_STARLARK_JAVAC_FALLBACK", "ANDROID_BINARY_STARLARK_JAVAC_ROLLOUT")
4646
load("//rules/acls:android_binary_starlark_rollout.bzl", "ANDROID_BINARY_STARLARK_FALLBACK", "ANDROID_BINARY_STARLARK_ROLLOUT")
47-
load("//rules/acls:android_binary_starlark_split_transition.bzl", "ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_FALLBACK", "ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_ROLLOUT")
4847
load("//rules/acls:android_binary_with_sandboxed_sdks_allowlist.bzl", "ANDROID_BINARY_WITH_SANDBOXED_SDKS_ALLOWLIST")
4948
load("//rules/acls:android_build_stamping_rollout.bzl", "ANDROID_BUILD_STAMPING_FALLBACK", "ANDROID_BUILD_STAMPING_ROLLOUT")
50-
load("//rules/acls:android_device_plugin_rollout.bzl", "ANDROID_DEVICE_PLUGIN_FALLBACK", "ANDROID_DEVICE_PLUGIN_ROLLOUT")
5149
load("//rules/acls:android_feature_splits_dogfood.bzl", "ANDROID_FEATURE_SPLITS_DOGFOOD")
5250
load("//rules/acls:android_instrumentation_test_manifest_check_rollout.bzl", "ANDROID_INSTRUMENTATION_TEST_MANIFEST_CHECK_FALLBACK", "ANDROID_INSTRUMENTATION_TEST_MANIFEST_CHECK_ROLLOUT")
5351
load("//rules/acls:android_instrumentation_test_prebuilt_test_apk.bzl", "ANDROID_INSTRUMENTATION_TEST_PREBUILT_TEST_APK_FALLBACK", "ANDROID_INSTRUMENTATION_TEST_PREBUILT_TEST_APK_ROLLOUT")
@@ -59,7 +57,6 @@ load("//rules/acls:android_lint_rollout.bzl", "ANDROID_LINT_FALLBACK", "ANDROID_
5957
load("//rules/acls:android_local_test_jdk_sts_rollout.bzl", "ANDROID_LOCAL_TEST_JDK_STS_FALLBACK", "ANDROID_LOCAL_TEST_JDK_STS_ROLLOUT")
6058
load("//rules/acls:android_multidex_native_min_sdk_allowlist.bzl", "ANDROID_MULTIDEX_NATIVE_MIN_SDK_ALLOWLIST")
6159
load("//rules/acls:android_test_lockdown.bzl", "ANDROID_TEST_LOCKDOWN_GENERATOR_FUNCTIONS", "ANDROID_TEST_LOCKDOWN_TARGETS")
62-
load("//rules/acls:b122039567.bzl", "B122039567")
6360
load("//rules/acls:baseline_profiles_optimizer_integration.bzl", "BASELINE_PROFILES_OPTIMIZER_INTEGRATION")
6461
load("//rules/acls:baseline_profiles_rollout.bzl", "BASELINE_PROFILES_ROLLOUT")
6562
load("//rules/acls:databinding.bzl", "DATABINDING_ALLOWED", "DATABINDING_DISALLOWED")
@@ -106,15 +103,9 @@ def _in_android_archive_dogfood(fqn):
106103
def _in_android_archive_excluded_deps_denylist(fqn):
107104
return matches(fqn, ANDROID_ARCHIVE_EXCLUDED_DEPS_DENYLIST_DICT)
108105

109-
def _in_android_device_plugin_rollout(fqn):
110-
return not matches(fqn, ANDROID_DEVICE_PLUGIN_FALLBACK_DICT) and matches(fqn, ANDROID_DEVICE_PLUGIN_ROLLOUT_DICT)
111-
112106
def _in_android_binary_starlark_javac(fqn):
113107
return not matches(fqn, ANDROID_BINARY_STARLARK_JAVAC_FALLBACK_DICT) and matches(fqn, ANDROID_BINARY_STARLARK_JAVAC_ROLLOUT_DICT)
114108

115-
def _in_android_binary_starlark_split_transition(fqn):
116-
return not matches(fqn, ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_FALLBACK_DICT) and matches(fqn, ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_ROLLOUT_DICT)
117-
118109
def _in_android_binary_with_sandboxed_sdks_allowlist(fqn):
119110
return matches(fqn, ANDROID_BINARY_WITH_SANDBOXED_SDKS_ALLOWLIST_DICT)
120111

@@ -138,9 +129,6 @@ def _in_android_test_lockdown_allowlist(fqn, generator):
138129
return matches(fqn, ANDROID_TEST_LOCKDOWN_TARGETS)
139130
return generator in ANDROID_TEST_LOCKDOWN_GENERATOR_FUNCTIONS_DICT
140131

141-
def _in_b122039567(fqn):
142-
return matches(fqn, B122039567_DICT)
143-
144132
def _in_android_library_resources_without_srcs(fqn):
145133
return matches(fqn, ANDROID_LIBRARY_RESOURCES_WITHOUT_SRCS_DICT)
146134

@@ -247,12 +235,8 @@ AAR_PROPAGATE_RESOURCES_ROLLOUT_DICT = make_dict(AAR_PROPAGATE_RESOURCES_ROLLOUT
247235
ANDROID_APPLICATION_WITH_SANDBOXED_SDKS_ALLOWLIST_DICT = make_dict(ANDROID_APPLICATION_WITH_SANDBOXED_SDKS_ALLOWLIST)
248236
ANDROID_ARCHIVE_DOGFOOD_DICT = make_dict(ANDROID_ARCHIVE_DOGFOOD)
249237
ANDROID_ARCHIVE_EXCLUDED_DEPS_DENYLIST_DICT = make_dict(ANDROID_ARCHIVE_EXCLUDED_DEPS_DENYLIST)
250-
ANDROID_DEVICE_PLUGIN_ROLLOUT_DICT = make_dict(ANDROID_DEVICE_PLUGIN_ROLLOUT)
251-
ANDROID_DEVICE_PLUGIN_FALLBACK_DICT = make_dict(ANDROID_DEVICE_PLUGIN_FALLBACK)
252238
ANDROID_BINARY_STARLARK_JAVAC_ROLLOUT_DICT = make_dict(ANDROID_BINARY_STARLARK_JAVAC_ROLLOUT)
253239
ANDROID_BINARY_STARLARK_JAVAC_FALLBACK_DICT = make_dict(ANDROID_BINARY_STARLARK_JAVAC_FALLBACK)
254-
ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_ROLLOUT_DICT = make_dict(ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_ROLLOUT)
255-
ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_FALLBACK_DICT = make_dict(ANDROID_BINARY_STARLARK_SPLIT_TRANSITION_FALLBACK)
256240
ANDROID_BINARY_WITH_SANDBOXED_SDKS_ALLOWLIST_DICT = make_dict(ANDROID_BINARY_WITH_SANDBOXED_SDKS_ALLOWLIST)
257241
ANDROID_FEATURE_SPLITS_DOGFOOD_DICT = make_dict(ANDROID_FEATURE_SPLITS_DOGFOOD)
258242
ANDROID_LIBRARY_RESOURCES_WITHOUT_SRCS_DICT = make_dict(ANDROID_LIBRARY_RESOURCES_WITHOUT_SRCS)
@@ -270,7 +254,6 @@ ANDROID_BUILD_STAMPING_ROLLOUT_DICT = make_dict(ANDROID_BUILD_STAMPING_ROLLOUT)
270254
ANDROID_BUILD_STAMPING_FALLBACK_DICT = make_dict(ANDROID_BUILD_STAMPING_FALLBACK)
271255
ANDROID_TEST_LOCKDOWN_GENERATOR_FUNCTIONS_DICT = make_dict(ANDROID_TEST_LOCKDOWN_GENERATOR_FUNCTIONS)
272256
ANDROID_TEST_LOCKDOWN_TARGETS_DICT = make_dict(ANDROID_TEST_LOCKDOWN_TARGETS)
273-
B122039567_DICT = make_dict(B122039567)
274257
CAN_USE_DEX2OAT_OPTIONS_DICT = make_dict(CAN_USE_DEX2OAT_OPTIONS)
275258
FIX_EXPORT_EXPORTING_FALLBACK_DICT = make_dict(FIX_EXPORT_EXPORTING_FALLBACK)
276259
FIX_EXPORT_EXPORTING_ROLLOUT_DICT = make_dict(FIX_EXPORT_EXPORTING_ROLLOUT)
@@ -359,13 +342,10 @@ acls = struct(
359342
in_aar_import_explicit_exports_manifest = _in_aar_import_explicit_exports_manifest,
360343
in_aar_import_exports_r_java = _in_aar_import_exports_r_java,
361344
in_aar_propagate_resources = _in_aar_propagate_resources,
362-
in_b122039567 = _in_b122039567,
363345
in_android_application_with_sandboxed_sdks_allowlist_dict = _in_android_application_with_sandboxed_sdks_allowlist_dict,
364346
in_android_archive_dogfood = _in_android_archive_dogfood,
365347
in_android_archive_excluded_deps_denylist = _in_android_archive_excluded_deps_denylist,
366-
in_android_device_plugin_rollout = _in_android_device_plugin_rollout,
367348
in_android_binary_starlark_javac = _in_android_binary_starlark_javac,
368-
in_android_binary_starlark_split_transition = _in_android_binary_starlark_split_transition,
369349
in_android_binary_with_sandboxed_sdks_allowlist = _in_android_binary_with_sandboxed_sdks_allowlist,
370350
in_android_feature_splits_dogfood = _in_android_feature_splits_dogfood,
371351
in_android_library_starlark_resource_outputs_rollout = _in_android_library_starlark_resource_outputs_rollout,

rules/android_binary_internal/impl.bzl

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -120,11 +120,10 @@ def _validate_manifest(ctx, packaged_resources_ctx, **unused_ctxs):
120120

121121
def _process_native_libs(ctx, **_unusued_ctxs):
122122
providers = []
123-
if acls.in_android_binary_starlark_split_transition(str(ctx.label)):
124-
providers.append(_process_native_deps(
125-
ctx,
126-
filename = "nativedeps",
127-
))
123+
providers.append(_process_native_deps(
124+
ctx,
125+
filename = "nativedeps",
126+
))
128127
return ProviderInfo(
129128
name = "native_libs_ctx",
130129
value = struct(providers = providers),

test/rules/android_local_test/java/com/starlark_resources/BUILD

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -32,24 +32,6 @@ android_local_test(
3232
],
3333
)
3434

35-
android_local_test(
36-
name = "environment",
37-
srcs = ["SampleTestWithEnvSetting.java"],
38-
data = ["env_location_data.txt"],
39-
env = {
40-
"android_local_test_test_env": "peekaboo",
41-
"android_local_test_loc_env": "$(location :env_location_data.txt)",
42-
},
43-
manifest = "AndroidManifest.xml",
44-
test_class = "com.starlark_resources.SampleTestWithEnvSetting",
45-
deps = [
46-
"//java/com/google/thirdparty/robolectric",
47-
"//third_party/java/hamcrest:android",
48-
"//third_party/java/robolectric",
49-
"@rules_android_maven//:junit_junit",
50-
],
51-
)
52-
5335
rule_test(
5436
name = "no_deps_with_resources_rule_test",
5537
target_under_test = ":no_deps_with_resources",

0 commit comments

Comments
 (0)