android-tools: fix deprecated header warning from clang

Remove .h header files from the SOURCES compilation list inside adbd.mk
to resolve a Clang warning regarding treating 'c-header' input as 'c++-header'

Moved the header files into a separate HEADERS variable to act purely as a
GNU Make dependency trigger, keeping them off the direct compiler
execution string.

Signed-off-by: AshishKumar Mishra <ashishkumar.mishra@bmwtechworks.in>
Signed-off-by: Khem Raj <khem.raj@oss.qualcomm.com>

1 files changed, 9 insertions, 6 deletions

diff --git a/meta-oe/recipes-devtools/android-tools/android-tools/adbd.mk b/meta-oe/recipes-devtools/android-tools/android-tools/adbd.mk
index 3282216b8e..709a166887 100644
--- a/meta-oe/recipes-devtools/android-tools/android-tools/adbd.mk
+++ b/meta-oe/recipes-devtools/android-tools/android-tools/adbd.mk
@@ -5,16 +5,12 @@ SOURCES = \
         adb/adbconnection/adbconnection_server.cpp \
         adb/daemon/auth.cpp \
         adb/daemon/file_sync_service.cpp \
-        adb/daemon/file_sync_service.h \
         adb/daemon/framebuffer_service.cpp \
-        adb/daemon/framebuffer_service.h \
         adb/daemon/jdwp_service.cpp \
         adb/daemon/main.cpp \
         adb/daemon/restart_service.cpp \
-        adb/daemon/restart_service.h \
         adb/daemon/services.cpp \
         adb/daemon/shell_service.cpp \
-        adb/daemon/shell_service.h \
         adb/daemon/usb_ffs.cpp \
         adb/daemon/usb_legacy.cpp \
         adb/daemon/usb.cpp \
@@ -41,7 +37,14 @@ SOURCES = \
         diagnose_usb/diagnose_usb.cpp \
         libasyncio/AsyncIO.cpp \
 
+HEADERS = \
+        adb/daemon/file_sync_service.h \
+        adb/daemon/framebuffer_service.h \
+        adb/daemon/restart_service.h \
+        adb/daemon/shell_service.h
+
 SOURCES := $(foreach source, $(SOURCES), system/core/$(source))
+HEADERS := $(foreach header, $(HEADERS), system/core/$(header))
 
 SOURCES += \
     frameworks/native/libs/adbd_auth/adbd_auth.cpp
@@ -79,9 +82,9 @@ ifneq ($(filter armel mipsel,$(DEB_HOST_ARCH)),)
   LDFLAGS += -latomic
 endif
 
-build: $(SOURCES)
+build: $(SOURCES) $(HEADERS)
         mkdir --parents debian/out/system/core
-        $(CXX) $^ -o debian/out/system/core/adbd $(CXXFLAGS) $(CPPFLAGS) $(LDFLAGS)
+        $(CXX) $(SOURCES) -o debian/out/system/core/adbd $(CXXFLAGS) $(CPPFLAGS) $(LDFLAGS)
 
 clean:
         $(RM) debian/out/system/core/adbd