conan-io / cmake-conan

CMake wrapper for conan C and C++ package manager
MIT License
814 stars 247 forks source link

Add a fallback in case CMAKE_OSX_ARCHITECTURES isn't set #611

Closed Cogitri closed 5 months ago

Cogitri commented 6 months ago

528 introduced new behaviour to support iOS, tvOS and watchOS. This also added a reliance on CMAKE_OSX_ARCHITECTURES, which may be unset in some configurations when building for MacOS. In this case, we should still fallback to CMAKE_SYSTEM_PROCESSOR and assume we're trying to build for the current target only.

CLAassistant commented 6 months ago

CLA assistant check
All committers have signed the CLA.

Cogitri commented 5 months ago

Thanks for the quick merge! :)