Merge "Revert "Revert "[incremental] change build rules to not to use aidl_interface""" into sc-dev

Android.bp

@@ -344,8 +344,8 @@ filegroup {
 genrule {
     name: "statslog-telephony-common-java-gen",
     tools: ["stats-log-api-gen"],
-    cmd: "$(location stats-log-api-gen) --java $(out) --module telephony_common"
+    cmd: "$(location stats-log-api-gen) --java $(out) --module telephony_common" +
-        + " --javaPackage com.android.internal.telephony --javaClass TelephonyCommonStatsLog",
+        " --javaPackage com.android.internal.telephony --javaClass TelephonyCommonStatsLog",
     out: ["com/android/internal/telephony/TelephonyCommonStatsLog.java"],
 }
 
@@ -1161,7 +1161,6 @@ cc_library {
     },
 }
 
-
 // This is the full proto version of libplatformprotos. It may only
 // be used by test code that is not shipped on the device.
 cc_library {
@@ -1227,68 +1226,57 @@ filegroup {
     path: "core/java",
 }
 
-aidl_interface {
+cc_defaults {
-    name: "libincremental_aidl",
+    name: "incremental_default",
-    unstable: true,
+    cflags: [
+        "-Wall",
+        "-Wextra",
+        "-Wextra-semi",
+        "-Werror",
+        "-Wzero-as-null-pointer-constant",
+        "-DANDROID_BASE_UNIQUE_FD_DISABLE_IMPLICIT_CONVERSION",
+    ],
+    shared_libs: [
+        "libbinder",
+        "libutils",
+    ],
+    aidl: {
+        include_dirs: [
+            "frameworks/native/aidl/binder",
+        ],
+        export_aidl_headers: true,
+    },
+}
+
+cc_library {
+    name: "libincremental_aidl-cpp",
     srcs: [
         ":incremental_aidl",
     ],
-    backend: {
+    defaults: ["incremental_default"],
-        java: {
-            sdk_version: "28",
-        },
-        cpp: {
-            enabled: true,
-        },
-        ndk: {
-            enabled: true,
-        },
-    },
 }
 
-aidl_interface {
+cc_library {
-    name: "libdataloader_aidl",
+    name: "libdataloader_aidl-cpp",
-    unstable: true,
     srcs: [
         ":dataloader_aidl",
     ],
-    imports: [
+    defaults: ["incremental_default"],
-        "libincremental_aidl",
+    shared_libs: [
+        "libincremental_aidl-cpp",
     ],
-    backend: {
-        java: {
-            sdk_version: "28",
-        },
-        cpp: {
-            enabled: true,
-        },
-        ndk: {
-            enabled: false,
-        },
-    },
 }
 
-aidl_interface {
+cc_library {
-    name: "libincremental_manager_aidl",
+    name: "libincremental_manager_aidl-cpp",
-    unstable: true,
     srcs: [
         ":incremental_manager_aidl",
     ],
-    imports: [
+    defaults: ["incremental_default"],
-        "libincremental_aidl",
+    shared_libs: [
-        "libdataloader_aidl",
+        "libincremental_aidl-cpp",
+        "libdataloader_aidl-cpp",
     ],
-    backend: {
-        java: {
-            sdk_version: "28",
-        },
-        cpp: {
-            enabled: true,
-        },
-        ndk: {
-            enabled: false,
-        },
-    },
 }
 
 // TODO(b/77285514): remove this once the last few hidl interfaces have been
@@ -1536,4 +1524,5 @@ java_library {
         ":protolog-common-src",
     ],
 }
+
 // protolog end

PREUPLOAD.cfg

@@ -1,5 +1,6 @@
 [Builtin Hooks]
 clang_format = true
+bpfmt = true
 
 [Builtin Hooks Options]
 # Only turn on clang-format check for the following subfolders.
@@ -15,7 +16,7 @@ clang_format = --commit ${PREUPLOAD_COMMIT} --style file --extensions c,h,cc,cpp
                services/incremental/
                tests/
                tools/
-
+bpfmt = -d
 [Hook Scripts]
 checkstyle_hook = ${REPO_ROOT}/prebuilts/checkstyle/checkstyle.py --sha ${PREUPLOAD_COMMIT}