-
Notifications
You must be signed in to change notification settings - Fork 16
Open
Description
According to
Lines 22 to 23 in ff3f727
| if test -z "$CMAKE"; then CMAKE="`which cmake`"; fi | |
| if test -z "$CMAKE"; then CMAKE=/Applications/CMake.app/Contents/bin/cmake; fi |
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.
Metadata
Metadata
Assignees
Labels
No labels