google / s2geometry

Computational geometry and spatial indexing on the sphere
http://s2geometry.io/
Apache License 2.0
2.29k stars 302 forks source link

Allow already have abseil #285

Closed MBkkt closed 1 year ago

MBkkt commented 1 year ago

We don't install abseil, we use add_subdirectory. so for us I needs patch like this.

Is it ok for you? The main idea if we already have abseil targets we don't need to find them

smcallis commented 1 year ago

This seems minor enough to be fine. Abseil should probably support this workflow upstream too. Wish CMake made this easier.

MBkkt commented 1 year ago

Thanks!