You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In switching these statements around, for macOS builds (albeit still experimental) at least, this proposal changes the policy that identifies the compiler as AppleClang rather than Clang. See https://cmake.org/cmake/help/latest/policy/CMP0025.html
So a change to cmake/defaults/CXXDefaults.cmake is also required, i.e.
Description of Issue
The top-level CMakeLists.txt (in the dev branch) currently begins with:
The documentation for CMake says that
project
should come aftercmake_minimum_required
. See https://cmake.org/cmake/help/latest/command/project.html#usageand https://cmake.org/cmake/help/latest/command/cmake_minimum_required.html#cmake-minimum-required
In switching these statements around, for macOS builds (albeit still experimental) at least, this proposal changes the policy that identifies the compiler as AppleClang rather than Clang. See https://cmake.org/cmake/help/latest/policy/CMP0025.html
So a change to
cmake/defaults/CXXDefaults.cmake
is also required, i.e.to change the
STREQUAL
toMATCHES
.All the other cases I have found for clang already use
MATCHES
or explicitly check for AppleClang.System Information (OS, Hardware)
I've tried this out on a macOS build on a much older version of USD, and has made my builds behave as I would expect when using CMake 3+.
I've not tried it on other platforms (yet), but it appears to be making the USD CMake scripts more inline with recommended practices.
Thanks
The text was updated successfully, but these errors were encountered: