View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0012930 | CMake | CMake | public | 2012-02-03 07:16 | 2014-08-21 15:46 | ||||
Reporter | Eugene Golushkov | ||||||||
Assigned To | Brad King | ||||||||
Priority | normal | Severity | feature | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | CMake 2.8.8 | ||||||||
Target Version | CMake 2.8.10 | Fixed in Version | CMake 2.8.10 | ||||||
Summary | 0012930: [patch] CMake does not support Visual Studio 11 WinRT project type | ||||||||
Description | CMake currently does not support setting project type as needed to develop WinRT applications for Windows 8 with the Visual Studio 10 generator. Patch is based on a similar patch for .NET project types from here: http://public.kitware.com/Bug/view.php?id=12586 [^] | ||||||||
Additional Information | The attached patch adds support for two new project properties: VS_WINRT_EXTENSIONS: Boolean property that correspond to "Enable Tailored Features" in Visual Studio 11 IDE. VS_WINRT_REFERENCES: Semicolon-delimited list of *.winmd references to add to the project, which creates a new <ItemGroup>. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | cmake_winrt.patch.txt [^] (3,796 bytes) 2012-02-03 07:17 [Show Content]
0001-VS11-Rename-Immersive-to-WindowsAppContainer-12930.patch [^] (1,425 bytes) 2012-08-21 12:10 [Show Content] | ||||||||
Relationships | |||||||||||||||||||||
|
Relationships |
Notes | |
(0028459) Brad King (manager) 2012-02-03 10:23 |
The patch looks pretty good but needs a few tweaks. First, a compilation error: - this->WriteString("<Immersive>true</Immersive>"); + this->WriteString("<Immersive>true</Immersive>\n", 2); Second, the WriteWinRTReferences method parses ;-separated lists on its own. CMake makes broad use of ;-separated lists and has a standard internal method called cmSystemTools::ExpandListArgument. Please convert the implementation to use that. |
(0028463) Eugene Golushkov (reporter) 2012-02-03 11:04 |
I will try to fix this, but I can not test my changes, as CMake can not bootstrap itself on Win8+VS11, and as primarily Mac developer I have no access to other Windows machines. |
(0028464) Eugene Golushkov (reporter) 2012-02-03 11:09 |
BTW, on Win8+VS11 it decide that >//Does the compiler support headers like iostream. >CMAKE_NO_ANSI_STREAM_HEADERS:INTERNAL=1 |
(0028467) Brad King (manager) 2012-02-03 14:30 |
Re 0012930:0028463: I just noticed that your pattern was taken from the change in 0012586 where I missed this ;-parsing issue. I'll take your patch as-is and then add a commit to fix up the list handling. |
(0028468) Brad King (manager) 2012-02-03 14:46 |
Applied patch with compilation fix: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=9e01aefd [^] and fixed the ;-separated value parsing: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=a03447b3 [^] Thanks for your contribution! Re 0012930:0028464: While interesting, the problem is not in the scope of this issue. |
(0030717) Eugene Golushkov (reporter) 2012-08-21 11:36 |
In VS 2012 RTM things changed. VS_WINRT_EXTENSIONS should create "<WindowsAppContainer>true</WindowsAppContainer>" tag instead of "<Immersive>true</Immersive>". Also ability to specify set_target_properties(X PROPERTIES COMPILE_FLAGS "/ZW") is crucial. |
(0030718) Brad King (manager) 2012-08-21 11:42 |
Re 0012930:0030717: VS 2012 is the official VS version 11 release. The VS 11 preview has expired so no one should be using it anymore. So we need "Immersive" for VS 10 and "WindowsAppContainer" for VS 11? |
(0030719) Eugene Golushkov (reporter) 2012-08-21 11:46 |
No, we don`t need Immersive at all - it was prerelease name of what is now known as WindowsAppContainer, and it always was VS11 aka VS2012 feature - no VS10 aka VS2010 involved. |
(0030720) Brad King (manager) 2012-08-21 12:11 |
How does "0001-VS11-Rename-Immersive-to-WindowsAppContainer-12930.patch" look? |
(0030721) Eugene Golushkov (reporter) 2012-08-21 12:15 |
"<WindowsAppContainer>true</WindowsAppContainer>" causes MSBuild to use another set of rules to build the project, another defaults, defines, API subset and so on. This setting is called "Windows Store App Support" in the VS2012 RTM. In VS11 beta it also causes flag "/ZW" ("Consume Windows Runtime Extension") to be passed to the C++ compiler automatically. But in RTM this flag is not longer passed automatically, as there can be dependencies that are compatible with WinRT but not depended on it, like zlib and other. Therefore for projects that are actually depended on WinRT such flag should be passed manually, via set_target_properties(X PROPERTIES COMPILE_FLAGS "/ZW") for example, but now this flag is unrecognized and lost, despite that for example set_target_properties(X PROPERTIES COMPILE_FLAGS "/bigobj") flag is passed. And last issue - if "<WindowsAppContainer>true</WindowsAppContainer>" is specified and no "Create/Use/Don`t use Precompiled Headers" ("/Yc", "/Yu", "/Y-") flag is specified explicitly - than MSBuild uses implicit default of "Use Precompiled Header" instead of usual "Don`t use" default. This behavior can not be suppressed via set_target_properties(X PROPERTIES COMPILE_FLAGS "/Y-"), so it would be nice if CMake generated files would always explicitly disable precompiled headers if they was not enabled by the user via "/Yc" or "/Yu' option. This can be done via default "<PrecompiledHeader>NotUsing</PrecompiledHeader>" subtag of <ClCompile> tag |
(0030722) Eugene Golushkov (reporter) 2012-08-21 12:17 |
Yeah, patch solves first of the three problems |
(0030723) Eugene Golushkov (reporter) 2012-08-21 12:24 edited on: 2012-08-21 12:24 |
BTW, I`m author of the Ogre`s WinRT port, and want to simplify bootstrapping process, as https://bitbucket.org/sinbad/ogre/src/2aea1c30ffa5/BuildingOgreWinRT.txt [^] is already too big. |
(0030724) Brad King (manager) 2012-08-21 12:48 edited on: 2012-08-21 13:03 |
Re 0012930:0030722: Patch for that problem applied: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=4c9ae472 [^] Re 0012930:0030721: The /ZW flag does not appear in our table of compiler flags: http://cmake.org/gitweb?p=cmake.git;a=blob;f=Source/cmVS10CLFlagTable.h;hb=v2.8.9 [^] probably because it hasn't been updated for VS 11. We may need to generate a new one. Try this patch for the precompiled header problem: diff --git a/Source/cmVisualStudio10TargetGenerator.cxx b/Source/cmVisualStudio10TargetGenerator.cxx index 53b6a9b..45bb948 100644 --- a/Source/cmVisualStudio10TargetGenerator.cxx +++ b/Source/cmVisualStudio10TargetGenerator.cxx @@ -1492,6 +1492,7 @@ void cmVisualStudio10TargetGenerator::WriteLinkOptions(std::string const& linkOptions.AddFlag("ImportLibrary", imLib.c_str()); linkOptions.AddFlag("ProgramDataBaseFile", pdb.c_str()); + linkOptions.AddFlag("PrecompiledHeader", "NotUsing"); linkOptions.Parse(flags.c_str()); if(!this->GeneratorTarget->ModuleDefinitionFile.empty()) { |
(0030725) Eugene Golushkov (reporter) 2012-08-21 12:55 edited on: 2012-08-21 12:55 |
No no no, it should be inside cmVisualStudio10TargetGenerator::WriteClOptions(), not inside cmVisualStudio10TargetGenerator::WriteLinkOptions() |
(0030726) Brad King (manager) 2012-08-21 13:07 |
Re 0012930:0030725: Oops! How about this:diff --git a/Source/cmVisualStudio10TargetGenerator.cxx b/Source/cmVisualStudio10TargetGenerator.cxx index 53b6a9b..b8b57e6 100644 --- a/Source/cmVisualStudio10TargetGenerator.cxx +++ b/Source/cmVisualStudio10TargetGenerator.cxx @@ -1204,6 +1204,7 @@ bool cmVisualStudio10TargetGenerator::ComputeClOptions( // Get preprocessor definitions for this directory. std::string defineFlags = this->Target->GetMakefile()->GetDefineFlags(); clOptions.FixExceptionHandlingDefault(); + clOptions.AddFlag("PrecompiledHeader", "NotUsing"); clOptions.Parse(flags.c_str()); clOptions.Parse(defineFlags.c_str()); clOptions.AddDefines |
(0030732) Eugene Golushkov (reporter) 2012-08-22 06:38 edited on: 2012-08-22 06:39 |
> + clOptions.AddFlag("PrecompiledHeader", "NotUsing"); Works like a charm, in both cases, when user ask to use pch, and when say nothing about it. How about the last issue, "/ZW" flag? Can I help you somehow with it? |
(0030733) Eugene Golushkov (reporter) 2012-08-22 06:46 edited on: 2012-08-22 06:53 |
BTW, as "<WindowsAppContainer>true</WindowsAppContainer>" corresponds to the UI setting "Windows Store App Support", and "/ZW" flag corresponds to the "Consume Windows Runtime Extension" - may be rename VS_WINRT_EXTENSIONS that influence first but not the second setting to the VS_WINRT_STORE_APP or similar, to remove confusion? BTW2: In VS2012 RTM all system API metadata was consolidated into the implicitly referenced Platfrom.winmd and Windows.winmd files, that makes VS_WINRT_REFERENCES unnecessary in many cases, when you need only system metadata. But to use such metadata (even those that are implicitly referenced) you need "/ZW" flag, so may be generate it implicitly if VS_WINRT_REFERENCES is set, even if it is empty ? |
(0030734) Brad King (manager) 2012-08-22 09:19 |
Re 0012930:0030732: Thanks for testing. Patch applied: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=f0ae381c [^] The /ZW flag problem is more general in that we haven't generated flag tables for the VS 11 tools. It's now a work in progress. |
(0030735) Brad King (manager) 2012-08-22 15:42 |
Re 0012930:0030734: This commit should add the ZW flag support: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=9b2dda5d [^] One may use this command: $ git show -C -C 9b2dda5d in a CMake Git work tree to see the differences from the VS 10 flag table. ZW is among them. |
(0030794) Brad King (manager) 2012-08-29 09:55 |
Resolving as fixed for now because the main fixes have been merged to CMake's Git master branch. Re 0012930:0030733: If you want to do this please base work off master as of commit 78b30939 or later. |
(0032886) Robert Maynard (manager) 2013-04-17 08:54 |
Closing resolved issues that have not been updated in more than 4 months. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2012-02-03 07:16 | Eugene Golushkov | New Issue | |
2012-02-03 07:17 | Eugene Golushkov | File Added: cmake_winrt.patch.txt | |
2012-02-03 09:04 | Brad King | Relationship added | related to 0012586 |
2012-02-03 10:23 | Brad King | Note Added: 0028459 | |
2012-02-03 11:04 | Eugene Golushkov | Note Added: 0028463 | |
2012-02-03 11:09 | Eugene Golushkov | Note Added: 0028464 | |
2012-02-03 14:30 | Brad King | Note Added: 0028467 | |
2012-02-03 14:30 | Brad King | Assigned To | => Brad King |
2012-02-03 14:30 | Brad King | Status | new => assigned |
2012-02-03 14:46 | Brad King | Note Added: 0028468 | |
2012-02-03 14:46 | Brad King | Status | assigned => resolved |
2012-02-03 14:46 | Brad King | Resolution | open => fixed |
2012-04-19 15:48 | David Cole | Fixed in Version | => CMake 2.8.8 |
2012-04-19 15:48 | David Cole | Target Version | => CMake 2.8.8 |
2012-08-21 11:36 | Eugene Golushkov | Note Added: 0030717 | |
2012-08-21 11:36 | Eugene Golushkov | Status | resolved => feedback |
2012-08-21 11:36 | Eugene Golushkov | Resolution | fixed => reopened |
2012-08-21 11:42 | Brad King | Note Added: 0030718 | |
2012-08-21 11:46 | Eugene Golushkov | Note Added: 0030719 | |
2012-08-21 11:46 | Eugene Golushkov | Status | feedback => assigned |
2012-08-21 12:10 | Brad King | File Added: 0001-VS11-Rename-Immersive-to-WindowsAppContainer-12930.patch | |
2012-08-21 12:11 | Brad King | Note Added: 0030720 | |
2012-08-21 12:11 | Brad King | Product Version | CMake 2.8.7 => CMake 2.8.8 |
2012-08-21 12:11 | Brad King | Fixed in Version | CMake 2.8.8 => |
2012-08-21 12:11 | Brad King | Target Version | CMake 2.8.8 => CMake 2.8.10 |
2012-08-21 12:15 | Eugene Golushkov | Note Added: 0030721 | |
2012-08-21 12:17 | Eugene Golushkov | Note Added: 0030722 | |
2012-08-21 12:24 | Eugene Golushkov | Note Added: 0030723 | |
2012-08-21 12:24 | Eugene Golushkov | Note Edited: 0030723 | |
2012-08-21 12:48 | Brad King | Note Added: 0030724 | |
2012-08-21 12:55 | Eugene Golushkov | Note Added: 0030725 | |
2012-08-21 12:55 | Eugene Golushkov | Note Edited: 0030725 | |
2012-08-21 13:03 | Brad King | Note Edited: 0030724 | |
2012-08-21 13:07 | Brad King | Note Added: 0030726 | |
2012-08-22 06:38 | Eugene Golushkov | Note Added: 0030732 | |
2012-08-22 06:39 | Eugene Golushkov | Note Edited: 0030732 | |
2012-08-22 06:46 | Eugene Golushkov | Note Added: 0030733 | |
2012-08-22 06:53 | Eugene Golushkov | Note Edited: 0030733 | |
2012-08-22 09:19 | Brad King | Note Added: 0030734 | |
2012-08-22 15:42 | Brad King | Note Added: 0030735 | |
2012-08-29 09:55 | Brad King | Note Added: 0030794 | |
2012-08-29 09:55 | Brad King | Status | assigned => resolved |
2012-08-29 09:55 | Brad King | Resolution | reopened => fixed |
2012-08-29 09:55 | Brad King | Fixed in Version | => CMake 2.8.10 |
2012-08-30 13:03 | Brad King | Relationship added | related to 0013498 |
2012-09-03 10:21 | Brad King | Relationship added | related to 0013511 |
2012-09-03 10:22 | Brad King | Relationship deleted | related to 0013511 |
2012-09-03 10:25 | Brad King | Relationship added | related to 0013511 |
2012-11-28 08:33 | Brad King | Relationship added | related to 0013749 |
2013-04-17 08:54 | Robert Maynard | Note Added: 0032886 | |
2013-04-17 08:54 | Robert Maynard | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |