[cmake-developers] confusing documentation: VS_WINRT_EXTENSIONS

[Brad King]

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