Fix gRPC header include path.

ChrisCummins · ChrisCummins · commit 9a8d4f7b5d76 · 2022-03-04T15:44:39.000Z
diff --git a/compiler_gym/util/GrpcStatusMacros.h b/compiler_gym/util/GrpcStatusMacros.h
@@ -9,7 +9,7 @@
 #include <boost/current_function.hpp>
 
 #include "glog/logging.h"
-#include "include/grpcpp/grpcpp.h"
+#include "grpcpp/grpcpp.h"
 
 using grpc::Status;
 
diff --git a/external/external.cmake b/external/external.cmake
@@ -174,7 +174,6 @@ if(COMPILER_GYM_GRPC_PROVIDER STREQUAL "internal")
   )
   FetchContent_MakeAvailable(grpc)
   set(_GRPC_CPP_PLUGIN_EXECUTABLE $<TARGET_FILE:grpc_cpp_plugin>)
-  #TODO(boian): remove this when GrpcStatusMacros.h uses the correct include path.
   target_include_directories(grpc++ INTERFACE "${grpc_SOURCE_DIR}")
 else()
   find_package(gRPC REQUIRED)

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,6 @@ if(COMPILER_GYM_GRPC_PROVIDER STREQUAL "internal")`
`174`	`174`	`)`
`175`	`175`	`FetchContent_MakeAvailable(grpc)`
`176`	`176`	`set(_GRPC_CPP_PLUGIN_EXECUTABLE $<TARGET_FILE:grpc_cpp_plugin>)`
`177`		`- #TODO(boian): remove this when GrpcStatusMacros.h uses the correct include path.`
`178`	`177`	`target_include_directories(grpc++ INTERFACE "${grpc_SOURCE_DIR}")`
`179`	`178`	`else()`
`180`	`179`	`find_package(gRPC REQUIRED)`