[cmake-developers] confusing documentation: VS_WINRT_EXTENSIONS

[Daniel Pfeifer]

2013/9/3 Brad King <brad.king at kitware.com>:
> 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.

OK, I will do that.