Martin Koller wrote: > On Friday 22 February 2013 12:23:23 Koller, Martin wrote: > > I propose the attached patch for CMakeDetermineSystem.cmake > > Can someone add this to the mentioned mantis bug entry or shall I create a > > new one ? > Revised patch which ignores the case of "amd64" so that "AMD64" works as > well
No, this is wrong. As you can read in the blog post you refer to there is also
IA64, and Windows on ARM could be near, too. So for both of those platform we
now would have "x86" reported.
My code would look like this:
if ($ENV{PROCESSOR_ARCHITEW6432})
set(CMAKE_HOST_SYSTEM_PROCESSOR "$ENV{PROCESSOR_ARCHITEW6432}")
else()
set(CMAKE_HOST_SYSTEM_PROCESSOR "$ENV{PROCESSOR_ARCHITECTURE}")
endif()
This will also keep the casing to avoid confusion, just keep the values the OS
gives you as long as they are consistent in themself.
Eike
--
signature.asc
Description: This is a digitally signed message part.
-- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://www.cmake.org/mailman/listinfo/cmake
