StarlingMonkey
StarlingMonkey copied to clipboard
Allow specifying dependencies from subprojects
This patch makes it possible for users of the add_builtins() CMake function to specify a per-builtin dependency on another target. This has been used to fix #42.
I can confirm this resolves the issue for us.
Force-pushed changes that:
- Remove
LINK_LIBS - Establish
DEPENDENCIESas a more general-purpose way of declaring the dependencies of built-ins - Installs OpenSSL in
${CMAKE_BINARY_DIR}/deps/OpenSSL, so it's clearer that it's a dependency - Patches usages of
LINK_LIBSinbuiltins.cmaketo useDEPENDENCIESinstead - Uses a
VERBOSEmessage when printing information about a builtin's dependencies
Again I can confirm that the latest updates all work correctly in the Fastly branch.
Rebased and force-pushed with the clarifications.