Martin Koller wrote:
> 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 ?http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=add8d22acc9417cb144a0b497f4f5ef330bfc680 Please wait a day or 2 and then download a snapshot and test if it behaves properly. 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
