commit 41923aea20a19ee8d497b5576d50b710967427c4
author Miroslav Zagorac <mzagorac@haproxy.com> Thu Jul 29 11:10:08 2021 +0200
committer Willy Tarreau <w@1wt.eu> Wed Aug 11 16:24:09 2021 +0200
tree a0367012c09f5204a7291b0a8890301644319835
parent c1485bfbdf049908bc3923deac4c01503e04f7dd

BUILD: opentracing: fixed build when using pkg-config utility

In case the OpenTracing C Wrapper library was installed as part of the
system (ie in a directory that pkg-config considers part of the system),
HAProxy building was not possible because in that case pkg-config did
not set the value of the OT_CFLAGS variable in the addon Makefile.

This resolves GitHub issue #1323.

(cherry picked from commit 3571111de3c21bb6b88cb3637a6f5850cb810de7)
Signed-off-by: Willy Tarreau <w@1wt.eu>

addons/ot/Makefile

diff --git a/addons/ot/Makefile b/addons/ot/Makefile
index 9d9b1bf..ef48ce4 100644
--- a/addons/ot/Makefile
+++ b/addons/ot/Makefile
@@ -8,6 +8,7 @@
 OT_CFLAGS    =
 OT_LDFLAGS   =
 OT_DEBUG_EXT =
+OT_PKGSTAT   =
 OTC_WRAPPER  = opentracing-c-wrapper
 
 ifneq ($(OT_DEBUG),)
@@ -16,6 +17,7 @@
 endif
 
 ifeq ($(OT_INC),)
+OT_PKGSTAT = $(shell pkg-config --exists $(OTC_WRAPPER)$(OT_DEBUG_EXT); echo $$?)
 OT_CFLAGS = $(shell pkg-config --silence-errors --cflags $(OTC_WRAPPER)$(OT_DEBUG_EXT))
 else
 ifneq ($(wildcard $(OT_INC)/$(OTC_WRAPPER)/.*),)
@@ -23,9 +25,15 @@
 endif
 endif
 
+ifeq ($(OT_PKGSTAT),)
 ifeq ($(OT_CFLAGS),)
 $(error OpenTracing C wrapper : can't find headers)
 endif
+else
+ifneq ($(OT_PKGSTAT),0)
+$(error OpenTracing C wrapper : can't find package)
+endif
+endif
 
 ifeq ($(OT_LIB),)
 OT_LDFLAGS = $(shell pkg-config --silence-errors --libs $(OTC_WRAPPER)$(OT_DEBUG_EXT))