From f826bce37a30dce3841209340f58493e9d45ce21 Mon Sep 17 00:00:00 2001 From: Remi NGUYEN VAN Date: Wed, 26 Jan 2022 17:43:53 +0900 Subject: [PATCH] Conditionally disable TetheringApiCurrentLib The "current" targets are only necessary in branches that can use non-stable APIs, and can fail to build otherwise. Apply ConnectivityNextEnableDefaults so they are disabled on such branches. Also deprecate NetworkStackNextEnableDefaults, so ConnectivityNextEnableDefaults is used everywhere. Bug: 190249673 Test: m Change-Id: I1b152f3cc22f1bed6444d280f0066e480980e949 --- Tethering/Android.bp | 3 ++- tests/cts/net/Android.bp | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Tethering/Android.bp b/Tethering/Android.bp index 041a3ae12c..0e04dbea04 100644 --- a/Tethering/Android.bp +++ b/Tethering/Android.bp @@ -64,6 +64,7 @@ java_defaults { android_library { name: "TetheringApiCurrentLib", defaults: [ + "ConnectivityNextEnableDefaults", "TetheringAndroidLibraryDefaults", "TetheringApiLevel" ], @@ -158,7 +159,7 @@ java_defaults { // Non-updatable tethering running in the system server process for devices not using the module android_app { name: "InProcessTethering", - defaults: ["TetheringAppDefaults", "TetheringApiLevel"], + defaults: ["TetheringAppDefaults", "TetheringApiLevel", "ConnectivityNextEnableDefaults"], static_libs: ["TetheringApiCurrentLib"], certificate: "platform", manifest: "AndroidManifest_InProcess.xml", diff --git a/tests/cts/net/Android.bp b/tests/cts/net/Android.bp index 81c30b1e47..07a319d628 100644 --- a/tests/cts/net/Android.bp +++ b/tests/cts/net/Android.bp @@ -69,7 +69,7 @@ java_defaults { // devices. android_test { name: "CtsNetTestCases", - defaults: ["CtsNetTestCasesDefaults", "NetworkStackNextEnableDefaults"], + defaults: ["CtsNetTestCasesDefaults", "ConnectivityNextEnableDefaults"], // TODO: CTS should not depend on the entirety of the networkstack code. static_libs: [ "NetworkStackApiCurrentLib",