tree 417e162d09321a927defa568fb5631c21aae18b3
parent 8c4ec3b7b0d69649c04778a39eed5cdca338c1c0
author David Benjamin <davidben@google.com> 1677519439 -0500
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> 1677529590 +0000

Fix the generated CMake build

This fixes two issues introduced in
0e68520eb27b1f37038e9d0772cfee1d015b50c3. First, libssl never had a
dependency on libcrypto, so the include directory did not get passed
along. Second, although this build (unlike the other CMake build)
doesn't define an install target, gRPC includes it and then pulls it
into its own install target. That then runs afoul of CMake's check
against include directories in install targets.

To avoid this, condition the src/include directory on
$<BUILD_INTERFACE:...> but omit a corresponding
$<INSTALL_INTERFACE:...>. Since we're not the ones providing an install
target, we don't actually know the value to use.

Per [0], using the generator expression means we need to manually make
it absolute ourselves.

[0] https://cmake.org/cmake/help/latest/command/target_include_directories.html

Change-Id: I2a25cc8382116c5957d94f65641220559b7af87d
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/57685
Reviewed-by: Bob Beck <bbe@google.com>
Auto-Submit: David Benjamin <davidben@google.com>
Commit-Queue: Bob Beck <bbe@google.com>