View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0011461 | CMake | CMake | public | 2010-11-14 08:18 | 2012-03-16 13:07 | ||||
Reporter | Robert Lenhardt | ||||||||
Assigned To | David Cole | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | CMake 2.8.3 | ||||||||
Target Version | CMake 2.8.4 | Fixed in Version | CMake 2.8.4 | ||||||
Summary | 0011461: finish midl support for VS2010 Generator | ||||||||
Description | The support for processing idl files is missing from the VS2010 Generator. Therefore idl files are now added as headers without processing. I added a patch which adds idl files under <Midl> tags and inserts file output options to the midl options in backward correspondance to the previous VS Generators. This works for my idl files. I did not insert preprocessor definitions to the midl options, as there where problems with quoted defines in previous midl versions (see Issue 8165). The new midl seems to accept quoted defines though. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | vc10midl.patch [^] (2,332 bytes) 2010-11-14 08:18 [Show Content] | ||||||||
Relationships | ||||||||||||||||
|
Relationships |
Notes | |
(0023272) Bill Hoffman (manager) 2010-11-15 16:14 |
Is this comparable to what happens with VS 8 and 9 with idl files? |
(0023273) Robert Lenhardt (reporter) 2010-11-15 16:35 |
No VS10 was missing special treatment of idl files to compile it with midl. VS8/9 generator has put them correctly under midl, but midl can not handle quoted preprocessor defintions. I have attached a midl test to issue 8165, this should detect the missing midl support in VS2010 too. |
(0024412) David Cole (manager) 2011-01-05 09:23 |
I'm working on fixing this (and its related issues) and ran into the following known, but apparently not yet fixed, problem with VS10: https://connect.microsoft.com/VisualStudio/feedback/details/554333/midl-compiler-fails-when-include-directory-contains-space?wa=wsignin1.0 [^] This means the test added to verify that it works on all our Visual Studio dashboards will not work properly in its source tree location since the vast majority of our dashboards run in a path that contains "/My Tests/" as part of its path name. I'll have to copy the test source tree to a known location without spaces in the path in order to run this test successfully on the VS10 dashboards. |
(0024419) David Cole (manager) 2011-01-05 11:08 |
Fixed by this commit, just pushed to CMake 'next': http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=13caaa3eb74a11dbf067409ea129321718d34dfe [^] I expect this will make it into the upcoming CMake 2.8.4 release. I will update this and related issues with another note if that expectation turns out to be too optimistic... :-) |
(0026306) David Cole (manager) 2011-05-02 14:45 |
Closing resolved issues that have not been updated in more than 3 months. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2010-11-14 08:18 | Robert Lenhardt | New Issue | |
2010-11-14 08:18 | Robert Lenhardt | File Added: vc10midl.patch | |
2010-11-15 16:14 | Bill Hoffman | Note Added: 0023272 | |
2010-11-15 16:35 | Robert Lenhardt | Note Added: 0023273 | |
2010-12-15 12:13 | David Cole | Relationship added | related to 0008165 |
2010-12-15 12:14 | David Cole | Assigned To | => David Cole |
2010-12-15 12:14 | David Cole | Status | new => assigned |
2010-12-15 12:14 | David Cole | Relationship added | has duplicate 0010687 |
2010-12-17 07:35 | David Cole | Target Version | => CMake 2.8.4 |
2011-01-05 09:23 | David Cole | Note Added: 0024412 | |
2011-01-05 11:08 | David Cole | Note Added: 0024419 | |
2011-01-05 11:08 | David Cole | Status | assigned => resolved |
2011-01-05 11:08 | David Cole | Fixed in Version | => CMake 2.8.4 |
2011-01-05 11:08 | David Cole | Resolution | open => fixed |
2011-05-02 14:45 | David Cole | Note Added: 0026306 | |
2011-05-02 14:45 | David Cole | Status | resolved => closed |
2012-03-16 13:07 | Brad King | Relationship added | child of 0013043 |
2012-03-16 13:07 | Brad King | Relationship replaced | related to 0013043 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |