- fix iOS CMake architecture

Signed-off-by: SSE4 <[email protected]>
conan-io · Jun 6, 2020 · d30579e · d30579e
1 parent b963b4d
commit d30579e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/conans/client/build/cmake_flags.py b/conans/client/build/cmake_flags.py
@@ -294,9 +294,9 @@ def get_definitions(self):
         definitions.update(build_type_definition(self._forced_build_type, build_type,
                                                  self._generator, self._output))
 
-        if str(os_) == "Macos":
-            if arch == "x86":
-                definitions["CMAKE_OSX_ARCHITECTURES"] = "i386"
+        if tools.is_apple_os(os_):
+            definitions["CMAKE_OSX_ARCHITECTURES"] = tools.to_apple_arch(arch)
+            definitions["CMAKE_OSX_SYSROOT"] = tools.XCRun(self._conanfile.settings).sdk_path
 
         definitions.update(self._cmake_cross_build_defines())
         definitions.update(self._get_cpp_standard_vars())

diff --git a/conans/test/unittests/client/build/cmake_test.py b/conans/test/unittests/client/build/cmake_test.py
@@ -1307,7 +1307,7 @@ def install_definitions_test(self):
                            ("iOS", "7.0",),
                            ("watchOS", "4.0",),
                            ("tvOS", "11.0",)])
-    @mock.patch('platform.system', return_value="Macos")
+    @mock.patch("conans.client.tools.apple.XCRun.sdk_path", return_value='/opt')
     def test_cmake_system_version_osx(self, the_os, os_version, _):
         settings = Settings.loads(get_default_settings_yml())
         settings.os = the_os