[CMake] CMAKE_SYSTEM_PROCESSOR says x86 on Win64 -> Patch

[Martin Koller]

On Saturday 23 February 2013 14:58:11 Rolf Eike Beer wrote:
> 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.

Great, even better.
What to do so this solution will make its way into the next official cmake version ?

-- 
Best regards/Schöne Grüße

Martin

A: Because it breaks the logical sequence of discussion
Q: Why is top posting bad?

()  ascii ribbon campaign - against html e-mail 
/\  www.asciiribbon.org   - against proprietary attachments

This mail was not scanned before sending.
It was sent from a secure Linux desktop.