fixup cpp.cmake

PROTO_DIRS is used by python/dotnet/java cmake file too
google · Sep 22, 2023 · dd86fc3 · dd86fc3
1 parent a05782c
commit dd86fc3
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/cmake/cpp.cmake b/cmake/cpp.cmake
@@ -129,6 +129,16 @@ endif()
 ##################
 ##  PROTO FILE  ##
 ##################
+# Get Protobuf include dir
+set(PROTO_DIRS)
+get_target_property(protobuf_dirs protobuf::libprotobuf INTERFACE_INCLUDE_DIRECTORIES)
+foreach(dir IN LISTS protobuf_dirs)
+  if (NOT "${dir}" MATCHES "INSTALL_INTERFACE|-NOTFOUND")
+    #message(STATUS "Adding proto path: ${dir}")
+    list(APPEND PROTO_DIRS "--proto_path=${dir}")
+  endif()
+endforeach()
+
 # Generate C++ OBJECT library from proto files,
 # e.g
 # generate_proto_library(
@@ -153,15 +163,6 @@ function(generate_proto_library)
  if(NOT PROTOC_PRG)
    message(FATAL_ERROR "protoc binary not found.")
  endif()
- # Get Protobuf include dir
- set(PROTO_DIRS)
- get_target_property(protobuf_dirs protobuf::libprotobuf INTERFACE_INCLUDE_DIRECTORIES)
- foreach(dir IN LISTS protobuf_dirs)
-   if (NOT "${dir}" MATCHES "INSTALL_INTERFACE|-NOTFOUND")
-     #message(STATUS "Adding proto path: ${dir}")
-     list(APPEND PROTO_DIRS "--proto_path=${dir}")
-   endif()
- endforeach()
 
  # Generate proto C++ files.
  set(PROTO_HDRS)