Refine dependency checking with unified 'dep'

This replaces all individual dependency macros with a single, concise
interface:
  $(call dep,flag-type,package-name[s])

Where:
- flag-type: cflags or libs
- package-name[s]: Single package or space-separated list

Author: jserv

Makefile

@@ -94,18 +94,18 @@ libtwin.a_files-y += src/screen-ops.c
 # Renderer implementations (draw-builtin.c includes all compositing operations)
 libtwin.a_files-$(CONFIG_RENDERER_BUILTIN) += src/draw-builtin.c
 libtwin.a_files-$(CONFIG_RENDERER_PIXMAN) += src/draw-pixman.c
-libtwin.a_cflags-$(CONFIG_RENDERER_PIXMAN) += $(shell pkg-config --cflags pixman-1)
+libtwin.a_cflags-$(CONFIG_RENDERER_PIXMAN) += $(call dep,cflags,pixman-1)
 ifeq ($(CONFIG_RENDERER_PIXMAN), y)
-TARGET_LIBS += $(shell pkg-config --libs pixman-1)
+TARGET_LIBS += $(call dep,libs,pixman-1)
 endif
 
 # Image loaders
 
 ifeq ($(CONFIG_LOADER_JPEG), y)
 libtwin.a_files-y += src/image-jpeg.c
 ifneq ($(CC_IS_EMCC), 1)
-libtwin.a_cflags-y += $(shell pkg-config --cflags libjpeg)
-TARGET_LIBS += $(shell pkg-config --libs libjpeg)
+libtwin.a_cflags-y += $(call dep,cflags,libjpeg)
+TARGET_LIBS += $(call dep,libs,libjpeg)
 else
 # Emscripten libjpeg port - flags needed for both compile and link
 libtwin.a_cflags-y += -sUSE_LIBJPEG=1
@@ -116,8 +116,8 @@ endif
 ifeq ($(CONFIG_LOADER_PNG), y)
 libtwin.a_files-y += src/image-png.c
 ifneq ($(CC_IS_EMCC), 1)
-libtwin.a_cflags-y += $(shell pkg-config --cflags libpng)
-TARGET_LIBS += $(shell pkg-config --libs libpng)
+libtwin.a_cflags-y += $(call dep,cflags,libpng)
+TARGET_LIBS += $(call dep,libs,libpng)
 else
 # Emscripten libpng port (includes zlib) - flags needed for both compile and link
 libtwin.a_cflags-y += -sUSE_LIBPNG=1 -sUSE_ZLIB=1
@@ -156,8 +156,8 @@ BACKEND := none
 ifeq ($(CONFIG_BACKEND_SDL), y)
 BACKEND = sdl
 libtwin.a_files-y += backend/sdl.c
-libtwin.a_cflags-y += $(shell sdl2-config --cflags)
-TARGET_LIBS += $(shell sdl2-config --libs)
+libtwin.a_cflags-y += $(call dep,cflags,sdl2)
+TARGET_LIBS += $(call dep,libs,sdl2)
 endif
 
 ifeq ($(CONFIG_BACKEND_FBDEV), y)
@@ -171,8 +171,8 @@ ifeq ($(CONFIG_BACKEND_VNC), y)
 BACKEND = vnc
 libtwin.a_files-y += backend/vnc.c
 libtwin.a_files-y += src/cursor.c
-libtwin.a_cflags-y += $(shell pkg-config --cflags neatvnc aml pixman-1)
-TARGET_LIBS += $(shell pkg-config --libs neatvnc aml pixman-1)
+libtwin.a_cflags-y += $(call dep,cflags,neatvnc aml pixman-1)
+TARGET_LIBS += $(call dep,libs,neatvnc aml pixman-1)
 endif
 
 ifeq ($(CONFIG_BACKEND_HEADLESS), y)
@@ -252,11 +252,11 @@ font-edit_files-y = \
 	tools/font-edit/font-edit.c
 font-edit_includes-y := tools/font-edit
 font-edit_cflags-y := \
-	$(call pkg-config-cflags,cairo) \
-	$(call sdl2-cflags)
+	$(call dep,cflags,cairo) \
+	$(call dep,cflags,sdl2)
 font-edit_ldflags-y := \
-	$(call pkg-config-libs,cairo) \
-	$(call sdl2-libs) \
+	$(call dep,libs,cairo) \
+	$(call dep,libs,sdl2) \
 	-lm
 
 # Headless control tool

mk/deps.mk

@@ -1,41 +1,41 @@
-# Dependency checking and flag retrieval macros
+# Unified dependency checking and flag retrieval system
+#
+# Usage: $(call dep,flag-type,package-name[s])
+#   flag-type: cflags or libs
+#   package-name[s]: single package or space-separated list (e.g., "cairo", "neatvnc aml")
+#
+# Automatically detects the right tool:
+# 1. Try package-config tool first (e.g., sdl2-config) - single package only
+# 2. Fall back to pkg-config if available - supports multiple packages
+# 3. Return empty string if package(s) not found
 
-# Check if pkg-config package exists and retrieve flags
-# Usage: $(call pkg-config-cflags,package-name)
-# Returns: CFLAGS or empty string if package not found
-define pkg-config-cflags
-$(shell pkg-config --cflags $(1) 2>/dev/null)
+# Internal: Try package-specific config tool (single package only)
+# Usage: $(call _dep-config,package-name,flag-type)
+define _dep-config
+$(shell command -v $(1)-config >/dev/null 2>&1 && $(1)-config --$(2) 2>/dev/null)
 endef
 
-# Usage: $(call pkg-config-libs,package-name)
-# Returns: LDFLAGS or empty string if package not found
-define pkg-config-libs
-$(shell pkg-config --libs $(1) 2>/dev/null)
+# Internal: Try pkg-config (supports multiple packages)
+# Usage: $(call _dep-pkg,package-names,flag-type)
+define _dep-pkg
+$(shell pkg-config --$(2) $(1) 2>/dev/null)
 endef
 
-# Check if sdl2-config exists and retrieve flags
-# Usage: $(call sdl2-cflags)
-# Returns: CFLAGS or empty string if sdl2-config not found
-define sdl2-cflags
-$(shell sdl2-config --cflags 2>/dev/null)
+# Internal: Check if input contains multiple packages
+# Usage: $(call _dep-multi,package-names)
+define _dep-multi
+$(filter-out $(firstword $(1)),$(1))
 endef
 
-# Usage: $(call sdl2-libs)
-# Returns: LDFLAGS or empty string if sdl2-config not found
-define sdl2-libs
-$(shell sdl2-config --libs 2>/dev/null)
-endef
-
-# Check if a command exists
-# Usage: $(call command-exists,command-name)
-# Returns: non-empty if command exists
-define command-exists
-$(shell command -v $(1) 2>/dev/null)
-endef
-
-# Check if pkg-config package exists
-# Usage: $(call pkg-exists,package-name)
-# Returns: "yes" if package exists, empty otherwise
-define pkg-exists
-$(shell pkg-config --exists $(1) 2>/dev/null && echo "yes")
+# Main entry point: Unified dependency checker
+# Usage: $(call dep,flag-type,package-name[s])
+# Example: $(call dep,cflags,cairo)
+#          $(call dep,libs,sdl2)
+#          $(call dep,cflags,neatvnc aml pixman-1)
+define dep
+$(if $(call _dep-multi,$(2)),\
+	$(call _dep-pkg,$(2),$(1)),\
+	$(if $(call _dep-config,$(2),$(1)),\
+		$(call _dep-config,$(2),$(1)),\
+		$(call _dep-pkg,$(2),$(1))))
 endef