Skip to content

Conversation

@cblichmann
Copy link
Contributor

If the absl::strings already exists at configure time, this means protobuf was likely included via add_subdirectory() from a parent project that also uses Abseil. Do not check protobuf_ABSL_PROVIDER in that case.

@cblichmann
Copy link
Contributor Author

Should fix #10435

@mkruskal-google mkruskal-google merged commit 6affd51 into protocolbuffers:main Aug 22, 2022
@mkruskal-google mkruskal-google linked an issue Aug 22, 2022 that may be closed by this pull request
@cblichmann cblichmann deleted the patch-1 branch August 23, 2022 14:18
copybara-service bot pushed a commit to protocolbuffers/utf8_range that referenced this pull request Oct 26, 2022
This is similar in spirit to my [Protobuf PR #10436](protocolbuffers/protobuf#10436) (as per [Protobuf Issue #10435](protocolbuffers/protobuf#10435))

PiperOrigin-RevId: 483934830
copybara-service bot pushed a commit to protocolbuffers/utf8_range that referenced this pull request Oct 26, 2022
This is similar in spirit to my [Protobuf PR #10436](protocolbuffers/protobuf#10436) (as per [Protobuf Issue #10435](protocolbuffers/protobuf#10435))

PiperOrigin-RevId: 483934830
copybara-service bot pushed a commit to protocolbuffers/utf8_range that referenced this pull request Oct 26, 2022
This is similar in spirit to my [Protobuf PR #10436](protocolbuffers/protobuf#10436) (as per [Protobuf Issue #10435](protocolbuffers/protobuf#10435))

PiperOrigin-RevId: 484007937
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

protobuf CMake dependency management using add_subdirectory

4 participants