From 7cf119fb6b622f075f45c87ec7ad09fc175dabec Mon Sep 17 00:00:00 2001
From: Jim Hague <jim@sinodun.com>
Date: Mon, 28 Oct 2019 11:05:54 +0000
Subject: [PATCH] Only include libidn/idn2 includes on building getdns objects
 if they are set.

---
 CMakeLists.txt | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 91489e8e..ca396734 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -520,8 +520,6 @@ target_include_directories(getdns_objects
   src/yxml
 
   ${CMAKE_CURRENT_BINARY_DIR}
-  ${LIBIDN_INCLUDE_DIR}
-  ${LIBIDN2_INCLUDE_DIR}
   ${OPENSSL_INCLUDE_DIR}
   )
 if (NOT HAVE_SSL_DANE_ENABLE)
@@ -529,6 +527,12 @@ if (NOT HAVE_SSL_DANE_ENABLE)
   target_include_directories(getdns_objects PRIVATE src/ssl_dane)
   set(USE_DANESSL 1)
 endif ()
+if (Libidn_FOUND)
+  target_include_directories(getdns_objects PRIVATE Libidn::Libidn)
+endif ()
+if (Libidn2_FOUND)
+  target_include_directories(getdns_objects PRIVATE Libidn2::Libidn2)
+endif ()
 
 # Don't compile separate objects for shared and static libraries.
 # Yes, -fPIC is slightly suboptimal for static libraries, but it looks