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.
--
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
