On 09/03/2014 01:21 PM, Bach, Pascal wrote:
> This is only a first draft and I would like to hear if I am on the right 
> track.

Yes, it looks good so far.

> +  else if (this->SystemName == "WindowsCE")
> +    {
> +    this->SystemIsWindowsCE = true;
> +    if (!this->InitializeWindowsCE(mf))

At the beginning of this block you should check/reject when
the generator name specified a platform name.  Something like:

    if(this->PlatformName != "Win32")
      {
      cmOStringStream e;
      e << "CMAKE_SYSTEM_NAME is 'WindowsCE' but CMAKE_GENERATOR "
        << "specifies a platform too: '" << this->GetName() << "'";
      mf->IssueMessage(cmake::FATAL_ERROR, e.str());
      return false;
      }

Thanks,
-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to