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.k...@kitware.com] 
Sent: Thursday, January 29, 2015 4:46 PM
To: Robert Goulet
Cc: cmake-developers@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

-- 

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