fix: cmake-cxxopts by using similar approach as glm (#44)

* fix: cmake-cxxopts by using similar approach as glm

Signed-off-by: Peter Staar <[email protected]>

* fixed the loguru cmake

Signed-off-by: Peter Staar <[email protected]>

* trying to be compatible with qpdf v10 and v11

Signed-off-by: Peter Staar <[email protected]>

---------

Signed-off-by: Peter Staar <[email protected]>

Author: PeterStaar-IBM

cmake/extlib_cxxopts.cmake

@@ -7,10 +7,14 @@ if(USE_SYSTEM_DEPS)
     message(STATUS "using system-deps in extlib_cxxopts.cmake")
 
     # this will define the cxxopts target
-    find_package(cxxopts REQUIRED)
+    # find_package(cxxopts REQUIRED)
 
-    add_library(${ext_name_cxxopts} INTERFACE IMPORTED)
-    add_dependencies(${ext_name_cxxopts} cxxopts)
+    # add_library(${ext_name_cxxopts} INTERFACE IMPORTED)
+    # add_dependencies(${ext_name_cxxopts} cxxopts)
+
+    find_package(PkgConfig)		   
+    pkg_check_modules(libcxxopts REQUIRED IMPORTED_TARGET cxxopts)
+    add_library(${ext_name_cxxopts} ALIAS PkgConfig::libcxxopts)
 
 else()
     message(STATUS "ignoring system-deps extlib_cxxopts.cmake")

cmake/extlib_loguru.cmake

@@ -7,10 +7,13 @@ if(USE_SYSTEM_DEPS)
     message(STATUS "using system-deps in extlib_loguru.cmake")
 
     # this will define the loguru target
-    find_package(loguru REQUIRED)
+    # find_package(loguru REQUIRED)
 
-    add_library(${ext_name_loguru} INTERFACE IMPORTED)
-    add_dependencies(${ext_name_loguru} loguru)
+    # add_library(${ext_name_loguru} INTERFACE IMPORTED)
+    # add_dependencies(${ext_name_loguru} loguru)
+
+    find_package(loguru CONFIG REQUIRED)
+    add_library(${ext_name_loguru} ALIAS loguru::loguru)
 
 else()
     message(STATUS "ignoring system-deps extlib_loguru.cmake")

src/v2/pdf_decoders/document.h

@@ -57,8 +57,12 @@ namespace pdflib
     timings({}),
     qpdf_document(),
 
-    qpdf_root(NULL),
-    qpdf_pages(NULL),
+    //qpdf_root(NULL),
+    //qpdf_pages(NULL),
+    
+    // have compatibulity between QPDF v10 and v11
+    qpdf_root(),
+    qpdf_pages(),
 
     number_of_pages(-1),
     json_document(nlohmann::json::value_t::null)
@@ -71,8 +75,12 @@ namespace pdflib
     timings(timings_),
     qpdf_document(),
 
-    qpdf_root(NULL),
-    qpdf_pages(NULL),
+    //qpdf_root(NULL),
+    //qpdf_pages(NULL),
+
+    // have compatibulity between QPDF v10 and v11
+    qpdf_root(),
+    qpdf_pages(),
 
     number_of_pages(-1),
     json_document(nlohmann::json::value_t::null)