[cmake-developers] Windows Store generated solution also containing win32 console app?

[Robert Goulet]

Ok. :)

That sounds silly, but how do I merge patches with git?

Not setting VS_WINRT_COMPONENT on an executable at all, when system name is 'WindowsStore', will produce an executable with everything properly set as a Windows Store app. I don't use VS_WINRT_COMPONENT (except for my new special case to turn it off explicitly) and everything seems to work properly. Essentially, if you set system name to 'WindowsStore', then all projects are assumed to have the Windows Store Support turned ON in Visual Studio.


-----Original Message-----
From: Brad King [mailto:brad.king at kitware.com] 
Sent: Thursday, January 29, 2015 4:46 PM
To: Robert Goulet
Cc: cmake-developers at cmake.org; Gilles Khouzam
Subject: Re: [cmake-developers] Windows Store generated solution also containing win32 console app?

On 01/28/2015 05:08 PM, Robert Goulet wrote:
> Please review my patch

Good start.

>> On Windows Store it should be an error to set VS_WINRT_COMPONENT 
>> explicitly to false for anything but an EXECUTABLE target.

Actually, what does it mean to build targeting Windows Store and call add_executable or add_library without setting VS_WINRT_COMPONENT right now?  Gilles?

Robert, please look at updating the VSWinPhoneStore test to try this feature on Store, with a custom command running an executable to generate a source or something.  Regardless of how we modify the interface we will still need a test for the functionality.

Please wrap C++ source lines to <= 79 characters.  Please avoid trailing whitespace or blank lines at EOF, but make sure there is a newline at EOF.

Please squash this all back into one commit for the next round of review.

Thanks,
-Brad