Explicitly depend on //absl/strings:string_view where used. Traditionally, `//absl/strings` provided string_view.h, but it is its own target now and we should depend on it whenever we include `absl/strings/string_view.h`, see comment here: https://github.com/abseil/abseil-cpp/blob/a690167a55b9e210b7884d311759977c0f5669c9/absl/strings/BUILD.bazel#L94-L99 Apply this strict reading to absl itself. In some cases, this also allowed to remove a dependency on the bigger `//absl/strings` library. PiperOrigin-RevId: 971471096 Change-Id: Id981182ae760ca11361381f7f25857ae1382b1cf
diff --git a/absl/synchronization/BUILD.bazel b/absl/synchronization/BUILD.bazel index 57740ef..d5438ba 100644 --- a/absl/synchronization/BUILD.bazel +++ b/absl/synchronization/BUILD.bazel
@@ -146,6 +146,7 @@ "//absl/debugging:stacktrace", "//absl/debugging:symbolize", "//absl/meta:type_traits", + "//absl/strings:string_view", "//absl/time", ] + select({ "//conditions:default": [], @@ -368,7 +369,6 @@ deps = [ ":per_thread_sem_test_common", ":synchronization", - "//absl/strings", "//absl/time", "@googletest//:gtest_main", ],
diff --git a/absl/synchronization/CMakeLists.txt b/absl/synchronization/CMakeLists.txt index 2805000..32e8c98 100644 --- a/absl/synchronization/CMakeLists.txt +++ b/absl/synchronization/CMakeLists.txt
@@ -113,6 +113,7 @@ absl::nullability absl::raw_logging_internal absl::stacktrace + absl::string_view absl::symbolize absl::time absl::tracing_internal @@ -258,7 +259,6 @@ DEPS absl::per_thread_sem_test_common absl::synchronization - absl::strings absl::time GTest::gmock_main )