From 13ac9f80f1e9398e7981c5198cc381cfd6824f12 Mon Sep 17 00:00:00 2001 From: John Chadwick Date: Wed, 12 Mar 2025 18:35:52 -0400 Subject: [PATCH] Remove unnecessary dependency on cel_proto_wrap_util For some reason, cel_proto_wrap_util #includes "eval/testutil/test_message.pb.h" even though I don't think it is used anywhere. This results in an otherwise unnecessary dependency on the test protos in a non-test target. --- eval/public/structs/BUILD | 1 - eval/public/structs/cel_proto_wrap_util.cc | 1 - 2 files changed, 2 deletions(-) diff --git a/eval/public/structs/BUILD b/eval/public/structs/BUILD index bdc926861..4ee28f6e7 100644 --- a/eval/public/structs/BUILD +++ b/eval/public/structs/BUILD @@ -57,7 +57,6 @@ cc_library( deps = [ ":protobuf_value_factory", "//eval/public:cel_value", - "//eval/testutil:test_message_cc_proto", "//internal:overflow", "//internal:proto_time_encoding", "//internal:status_macros", diff --git a/eval/public/structs/cel_proto_wrap_util.cc b/eval/public/structs/cel_proto_wrap_util.cc index 3aaa205bf..a039cc330 100644 --- a/eval/public/structs/cel_proto_wrap_util.cc +++ b/eval/public/structs/cel_proto_wrap_util.cc @@ -43,7 +43,6 @@ #include "absl/types/variant.h" #include "eval/public/cel_value.h" #include "eval/public/structs/protobuf_value_factory.h" -#include "eval/testutil/test_message.pb.h" #include "internal/overflow.h" #include "internal/proto_time_encoding.h" #include "internal/status_macros.h"