Avoid uninitialized CMake GETOPT variable warnings

Only access GETOPT_INCLUDE_DIRS if GETOPT is found.

Fix a few warnings with --warn-uninitialized like:

```
CMake Warning (dev) at cmd/gvmap/CMakeLists.txt:20
(target_include_directories):
  uninitialized variable 'GETOPT_INCLUDE_DIRS'
```

Towards #2291

diff --git a/cmd/edgepaint/CMakeLists.txt b/cmd/edgepaint/CMakeLists.txt
index 8b3e464b3b0f23ca62f2dc473963b68c9439534b..58ebb81fd82922a0113f6109865ea33579b09750 100644 (file)
--- a/cmd/edgepaint/CMakeLists.txt
+++ b/cmd/edgepaint/CMakeLists.txt
@@ -9,9 +9,11 @@ if(with_sfdp)
     ../../lib/cdt
   )
 
-  target_include_directories(edgepaint SYSTEM PRIVATE
-    ${GETOPT_INCLUDE_DIRS}
-  )
+  if(GETOPT_FOUND)
+    target_include_directories(edgepaint SYSTEM PRIVATE
+      ${GETOPT_INCLUDE_DIRS}
+    )
+  endif()
 
   target_link_libraries(edgepaint PRIVATE
     cdt

diff --git a/cmd/gvmap/CMakeLists.txt b/cmd/gvmap/CMakeLists.txt
index e3d91a1cfcbe12c858ea53ba226e776993ef96df..3f02bab712ab5db891796494a0c30441797d72cc 100644 (file)
--- a/cmd/gvmap/CMakeLists.txt
+++ b/cmd/gvmap/CMakeLists.txt
@@ -17,9 +17,11 @@ if(with_sfdp)
     ../../lib/cdt
   )
 
-  target_include_directories(gvmap SYSTEM PRIVATE
-    ${GETOPT_INCLUDE_DIRS}
-  )
+  if(GETOPT_FOUND)
+    target_include_directories(gvmap SYSTEM PRIVATE
+      ${GETOPT_INCLUDE_DIRS}
+    )
+  endif()
 
   target_link_libraries(gvmap
     cgraph

diff --git a/cmd/tools/CMakeLists.txt b/cmd/tools/CMakeLists.txt
index 39a4930f2f68e78c13de3c51a2b36d9272c5989a..7f5f45aca3535e573ea9acb1da9062840efb1acb 100644 (file)
--- a/cmd/tools/CMakeLists.txt
+++ b/cmd/tools/CMakeLists.txt
@@ -54,9 +54,11 @@ function(add_simple_tool name)
     ../../lib/pathplan
   )
 
-  target_include_directories(${name} SYSTEM PRIVATE
-    ${GETOPT_INCLUDE_DIRS}
-  )
+  if(GETOPT_FOUND)
+    target_include_directories(${name} SYSTEM PRIVATE
+      ${GETOPT_INCLUDE_DIRS}
+    )
+  endif()
 
   target_link_libraries(${name}
     cgraph
@@ -97,9 +99,11 @@ target_include_directories(dijkstra PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(dijkstra SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(dijkstra SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(dijkstra
   cgraph
@@ -140,9 +144,11 @@ target_include_directories(gml2gv PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(gml2gv SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(gml2gv SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(gml2gv
   cgraph
@@ -159,9 +165,11 @@ target_include_directories(gv2gml PRIVATE
   ../../lib/gvc
   ../../lib/pathplan
 )
-target_include_directories(gv2gml SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(gv2gml SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 target_link_libraries(gv2gml cgraph gvc ingraphs)
 if(NOT HAVE_GETOPT_H)
   target_link_libraries(gv2gml ${GETOPT_LINK_LIBRARIES})
@@ -190,9 +198,14 @@ if(EXPAT_FOUND)
 
   target_include_directories(graphml2gv SYSTEM PRIVATE
     ${EXPAT_INCLUDE_DIRS}
-    ${GETOPT_INCLUDE_DIRS}
   )
 
+  if(GETOPT_FOUND)
+    target_include_directories(graphml2gv SYSTEM PRIVATE
+      ${GETOPT_INCLUDE_DIRS}
+    )
+  endif()
+
   target_link_libraries(graphml2gv
     cgraph
     ${EXPAT_LIBRARIES}
@@ -224,9 +237,11 @@ target_include_directories(gvcolor PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(gvcolor SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(gvcolor SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(gvcolor
   cgraph
@@ -254,9 +269,11 @@ target_include_directories(gvgen PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(gvgen SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if (GETOPT_FOUND)
+  target_include_directories(gvgen SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(gvgen cgraph)
 
@@ -284,9 +301,11 @@ target_include_directories(gvpack PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(gvpack SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(gvpack SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(gvpack
   cgraph
@@ -321,8 +340,12 @@ if(EXPAT_FOUND)
 
   target_include_directories(gxl2gv SYSTEM PRIVATE
     ${EXPAT_INCLUDE_DIRS}
-    ${GETOPT_INCLUDE_DIRS}
   )
+  if(GETOPT_FOUND)
+    target_include_directories(gxl2gv SYSTEM PRIVATE
+      ${GETOPT_INCLUDE_DIRS}
+    )
+  endif()
 
   target_link_libraries(gxl2gv
     cgraph
@@ -379,9 +402,11 @@ target_include_directories(mm2gv PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(mm2gv SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(mm2gv SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(mm2gv
   sparse
@@ -409,9 +434,11 @@ target_include_directories(sccmap PRIVATE
   ../../lib/pathplan
 )
 
-target_include_directories(sccmap SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(sccmap SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 target_link_libraries(sccmap
   cgraph

diff --git a/lib/gvpr/CMakeLists.txt b/lib/gvpr/CMakeLists.txt
index 02c1c25f44b5d460d655993be993d839ddcaa331..15637aa025541135589f1b51d426714cb72a29b9 100644 (file)
--- a/lib/gvpr/CMakeLists.txt
+++ b/lib/gvpr/CMakeLists.txt
@@ -34,9 +34,11 @@ target_include_directories(gvpr PRIVATE
   ../expr
 )
 
-target_include_directories(gvpr SYSTEM PRIVATE
-  ${GETOPT_INCLUDE_DIRS}
-)
+if(GETOPT_FOUND)
+  target_include_directories(gvpr SYSTEM PRIVATE
+    ${GETOPT_INCLUDE_DIRS}
+  )
+endif()
 
 if(WIN32 AND NOT MINGW)
   target_include_directories(gvpr PRIVATE ../../windows/include/unistd)