From cd84eea18084e3fce38c93d729ae442045417717 Mon Sep 17 00:00:00 2001 From: "M. Chornyi" <99709299+mc-nv@users.noreply.github.com> Date: Tue, 4 Aug 2026 22:02:46 -0700 Subject: [PATCH 1/2] build: Align with other Cmake configuration --- protobuf/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/protobuf/CMakeLists.txt b/protobuf/CMakeLists.txt index 35bf6a2..518a261 100644 --- a/protobuf/CMakeLists.txt +++ b/protobuf/CMakeLists.txt @@ -113,7 +113,7 @@ if(${TRITON_COMMON_ENABLE_GRPC}) ) if(TRITON_COMMON_ENABLE_PROTOBUF_PYTHON) - find_package(Python REQUIRED COMPONENTS Interpreter) + find_package(Python3 REQUIRED COMPONENTS Interpreter) add_custom_command( OUTPUT "${GRPC_PY}" COMMAND ${Python_EXECUTABLE} From d195feec0304bd6e7d8dd8e8cbbd74adff729ab7 Mon Sep 17 00:00:00 2001 From: "M. Chornyi" <99709299+mc-nv@users.noreply.github.com> Date: Wed, 5 Aug 2026 13:28:07 -0700 Subject: [PATCH 2/2] fix: Use Python3_EXECUTABLE for the gRPC Python codegen cd84eea switched the lookup to find_package(Python3 ...) but left the custom command on ${Python_EXECUTABLE}, which that package never populates. The variable expands to empty, so the grpc_tools.protoc command runs with no interpreter and grpc-service-py-library cannot generate its output. TRITON_COMMON_ENABLE_PROTOBUF_PYTHON defaults to ON, so this is on the default build path rather than an opt-in one. Only Interpreter is requested, which is all the codegen needs. --- protobuf/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/protobuf/CMakeLists.txt b/protobuf/CMakeLists.txt index 518a261..cec5f87 100644 --- a/protobuf/CMakeLists.txt +++ b/protobuf/CMakeLists.txt @@ -116,7 +116,7 @@ if(${TRITON_COMMON_ENABLE_GRPC}) find_package(Python3 REQUIRED COMPONENTS Interpreter) add_custom_command( OUTPUT "${GRPC_PY}" - COMMAND ${Python_EXECUTABLE} + COMMAND ${Python3_EXECUTABLE} ARGS -m grpc_tools.protoc -I "${grpc_service_proto_dir}"