On 09/03/2013 03:39 AM, Daniel Pfeifer wrote: > According to http://www.cmake.org/Bug/view.php?id=12930#c30721, > WindowsAppContainer will not pass /ZW to the command line. > Hence, setting VS_WINRT_EXTENSIONS will not enable C++/CX and the > documentation is incorrect. But that is good. > > Maybe also the name of that property is misleading. It might be better > to use the same name that is used as a define, which means > WINAPI_FAMILY as property with the possible values > WINAPI_PARTITION_APP and WINAPI_PARTITION_DESKTOP (and maybe more, > once Microsoft adds new partitions of the winapi). > > It is also important that WINAPI_PARTITION_DESKTOP is not supported > when compiling for ARM. Here, WINAPI_PARTITION_APP should be the > default, especially for the initial compile checks. > > Visual Studio generators should set WindowsAppContainer if > WINAPI_FAMILY is set to WINAPI_PARTITION_APP. > Makefile generators should pass -DWINAPI_FAMILY=${WINAPI_FAMILY} to > the command line.
As I don't do development in this area I have to depend on contributors for knowledge. If you have time, please prepare a patch series: 1. Add the better-named property you propose. 2. Fix the docs of VS_WINRT_EXTENSIONS and perhaps suggest using the alternative name instead. Thanks, -Brad -- 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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
