Expliclity require OpenSSL and add its include directory not only for special cases BUNDLED and BREW for S2_SOURCE.
Using S2_SOURCE=SYSTEM, there may be cases where openssl and s2geometry were installed in different prefix paths (e.g., system-installed openssl and custom s2geometry installation like in github.com/benbovy/spherely/pull/35). This PR addresses those cases too.
I didn't see #21, but 56eb319 and e462b66 should make s2geography headers discoverable in both "add_subdirectory" vs. "find_package" cases in a modern cmake way.
benbovy
commented
1 week ago
Expliclity require OpenSSL and add its include directory not only for special cases
BUNDLEDandBREWforS2_SOURCE.Using
S2_SOURCE=SYSTEM, there may be cases where openssl and s2geometry were installed in different prefix paths (e.g., system-installed openssl and custom s2geometry installation like in github.com/benbovy/spherely/pull/35). This PR addresses those cases too.