According to
|
if test -z "$CMAKE"; then CMAKE="`which cmake`"; fi |
|
if test -z "$CMAKE"; then CMAKE=/Applications/CMake.app/Contents/bin/cmake; fi |
we are checking if
CMAKE env variable is empty and then first set
CMAKE to
which cmake (appropriate for Linux envs) but in the next line we overwrite the same with the MacOS equivalent without checking if we are on MacOS or not.
It would be great to make those two checks work on both Linux and MacOS so that s2 can use Linux's cmake to build Abseil as a system dependency.
According to
s2/tools/build_absl.sh
Lines 22 to 23 in ff3f727
CMAKEenv variable is empty and then first setCMAKEtowhich cmake(appropriate for Linux envs) but in the next line we overwrite the same with the MacOS equivalent without checking if we are on MacOS or not.It would be great to make those two checks work on both Linux and MacOS so that
s2can use Linux'scmaketo buildAbseilas a system dependency.