commit d6659983335082d4b54ee006626a1ebc29423bcc
author Roman Stratiienko <r.stratiienko@gmail.com> Sat Feb 27 22:08:51 2021 +0200
committer Roman Stratiienko <r.stratiienko@gmail.com> Wed Mar 03 12:14:14 2021 +0200
tree ea551fdfcc1d64157b497fae7af2bd641bee2df3
parent d518a0594a3535a450128102163529e1f319fb37

drm_hwcomposer: CI: Use bpfmt to check Android.bp files style

Fix available errors by executing:
$ find -name "*.bp" -exec bpfmt -d -s -w {} \;

Suggested-by: Marijn Suijten <marijn.suijten@somainline.org>
Signed-off-by: Roman Stratiienko <r.stratiienko@gmail.com>

.ci/.gitlab-ci-checkcommit.sh

diff --git a/.ci/.gitlab-ci-checkcommit.sh b/.ci/.gitlab-ci-checkcommit.sh
index 314238f..817afa7 100755
--- a/.ci/.gitlab-ci-checkcommit.sh
+++ b/.ci/.gitlab-ci-checkcommit.sh
@@ -55,4 +55,10 @@
 		cat /tmp/format-fixup.patch >&2
 		exit 1
 	fi
+
+	find -name "*.bp" -exec bpfmt -d -s {} \; > /tmp/bpfmt.patch
+	if [ -s  /tmp/bpfmt.patch ]; then
+		cat /tmp/bpfmt.patch >&2
+		exit 1
+	fi
 done

.gitlab-ci.yml

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 63cedb6..7fd7367 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1,8 +1,8 @@
-image: ubuntu:20.04
+image: ubuntu:20.10
 
 before_script:
   - apt-get --quiet update --yes >/dev/null
-  - apt-get --quiet install --yes clang-11 clang-tidy-11 clang-format-11 git libdrm-dev >/dev/null
+  - apt-get --quiet install --yes clang-11 clang-tidy-11 clang-format-11 git libdrm-dev blueprint-tools >/dev/null
 
 stages:
   - build

Android.bp

diff --git a/Android.bp b/Android.bp
index 20b22ce..9bed9e4 100644
--- a/Android.bp
+++ b/Android.bp
@@ -21,8 +21,8 @@
     srcs: ["utils/Worker.cpp"],
 
     include_dirs: [
-        "external/drm_hwcomposer/include",
         "external/drm_hwcomposer",
+        "external/drm_hwcomposer/include",
     ],
 
     cflags: [
@@ -52,8 +52,8 @@
     ],
 
     include_dirs: [
-        "external/drm_hwcomposer/include",
         "external/drm_hwcomposer",
+        "external/drm_hwcomposer/include",
     ],
 
     static_libs: ["libdrmhwc_utils"],
@@ -64,8 +64,8 @@
     ],
 
     cppflags: [
-        "-DHWC2_USE_CPP11",
         "-DHWC2_INCLUDE_STRINGIFICATION",
+        "-DHWC2_USE_CPP11",
     ],
 
     product_variables: {
@@ -77,6 +77,7 @@
     relative_install_path: "hw",
     vendor: true,
 }
+
 cc_library_static {
     name: "drm_hwcomposer",
     defaults: ["hwcomposer.drm_defaults"],
@@ -105,9 +106,9 @@
         "utils/autolock.cpp",
         "utils/hwcutils.cpp",
 
-        "backend/BackendManager.cpp",
         "backend/Backend.cpp",
         "backend/BackendClient.cpp",
+        "backend/BackendManager.cpp",
         "backend/BackendRCarDu.cpp",
     ],
 }

tests/Android.bp

diff --git a/tests/Android.bp b/tests/Android.bp
index 3bb494a..e30898c 100644
--- a/tests/Android.bp
+++ b/tests/Android.bp
@@ -29,7 +29,7 @@
     static_libs: ["libdrmhwc_utils"],
     shared_libs: ["hwcomposer.drm"],
     include_dirs: [
-        "external/drm_hwcomposer/include",
         "external/drm_hwcomposer",
+        "external/drm_hwcomposer/include",
     ],
 }