MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012567CMakeDocumentationpublic2011-11-09 20:522013-03-04 08:38
ReporterMateusz Loskot 
Assigned ToRolf Eike Beer 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformWindowsOSWindows 7 ProfessionalOS VersionWindows 6.1
Product VersionCMake 2.8.6 
Target VersionCMake 2.8.10Fixed in VersionCMake 2.8.10 
Summary0012567: MSVC11 variable is not documented, nor some other versions
DescriptionThis version of CMake supports generator for Visual Studio 11 (current Developer Preview):

C:\>cmake --help | grep "Visual Studio 11"
  Visual Studio 11 = Generates Visual Studio 11 project files.
  Visual Studio 11 Win64 = Generates Visual Studio 11 Win64 project

However, MSVC11 variable is not set.
So, CMake-provided modules like FindBoost.cmake can not perform libraries lookup effectively, because they rely on MSVCXY variable:

...
elseif (MSVC90)
   set(_boost_COMPILER "-vc90")
elseif (MSVC10)
   set(_boost_COMPILER "-vc100")
...


It also looks FindBoost.cmake is lacking of

elseif (MSVC11)
  set(_boost_COMPILER "-vc110")


By the way, the documentation mentions only MSVC and MSVC80 variables:
http://cmake.org/cmake/help/cmake-2-8-docs.html#variable:MSVC80 [^]
It does not document variables for other versions of Visual C++ like MSVC71.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch 0001-add-documentation-for-all-MSVCxxx-version-variables-.patch (2,164) 2012-09-09 08:03
https://public.kitware.com/Bug/file/4469/0001-add-documentation-for-all-MSVCxxx-version-variables-.patch
Issue History
Date ModifiedUsernameFieldChange
2011-11-09 20:52Mateusz LoskotNew Issue
2011-11-09 21:03Mateusz LoskotNote Added: 0027741
2011-11-10 07:59Brad KingStatusnew => resolved
2011-11-10 07:59Brad KingResolutionopen => no change required
2011-11-10 07:59Brad KingAssigned To => Brad King
2011-11-10 10:59David ColeFixed in Version => CMake 2.8.7
2011-11-10 10:59David ColeTarget Version => CMake 2.8.7
2011-11-10 11:02David ColeNote Added: 0027746
2011-11-10 12:00Brad KingAssigned ToBrad King => David Cole
2011-11-10 12:00Brad KingStatusresolved => feedback
2011-11-10 12:00Brad KingResolutionno change required => reopened
2011-11-10 12:01Brad KingStatusfeedback => assigned
2011-11-10 12:01Brad KingSummaryMSVC11 variable is not set => MSVC11 variable is not documented, nor some other versions
2011-12-13 19:26David ColeNote Added: 0027969
2011-12-13 19:26David ColeFixed in VersionCMake 2.8.7 =>
2011-12-13 19:26David ColeTarget VersionCMake 2.8.7 =>
2012-08-13 15:40David ColeStatusassigned => backlog
2012-08-13 15:40David ColeNote Added: 0030620
2012-09-09 08:03Rolf Eike BeerFile Added: 0001-add-documentation-for-all-MSVCxxx-version-variables-.patch
2012-09-09 08:04Rolf Eike BeerNote Added: 0030963
2012-09-10 08:45Brad KingNote Added: 0030974
2012-09-10 08:45Brad KingAssigned ToDavid Cole => Rolf Eike Beer
2012-09-10 08:45Brad KingStatusbacklog => assigned
2012-09-10 08:45Brad KingResolutionreopened => open
2012-09-10 08:45Brad KingTarget Version => CMake 2.8.10
2012-09-12 15:55Rolf Eike BeerNote Added: 0031019
2012-09-12 15:55Rolf Eike BeerStatusassigned => resolved
2012-09-12 15:55Rolf Eike BeerResolutionopen => fixed
2012-09-12 15:55Rolf Eike BeerCategoryCMake => Documentation
2012-09-12 15:55Rolf Eike BeerFixed in Version => CMake 2.8.10
2013-03-04 08:38Robert MaynardNote Added: 0032447
2013-03-04 08:38Robert MaynardStatusresolved => closed

Notes
(0027741)
Mateusz Loskot   
2011-11-09 21:03   
I apologies for this premature report. CMake does set the boolean variable MSVC11.

The problem is only with lack of update in FindBoost.cmake. I'll submit patch.

Please, close this ticket, it's invalid.
(0027746)
David Cole   
2011-11-10 11:02   
You do have a valid point about MSVC80 being the only documented version-specific MSVC* variable, though... We should add docs for the others, too.
(0027969)
David Cole   
2011-12-13 19:26   
Unset target version field; too late for a fix to go into 2.8.7; deferred until a future version.
(0030620)
David Cole   
2012-08-13 15:40   
Sending old, not-recently-updated issues to the backlog.

(The age of the bug alone means that nobody is actively working on it...)

If an issue you care about is sent to the backlog when you feel it should have been addressed in a different manner, please bring it up on the CMake mailing list for discussion. Sign up for the mailing list here, if you're not already on it: http://www.cmake.org/mailman/listinfo/cmake [^]

It's easy to re-activate a bug here if you can find a CMake developer who has the bandwidth to take it on, and ferry a fix through to our 'next' branch for dashboard testing.
(0030963)
Rolf Eike Beer   
2012-09-09 08:04   
David, is this patch enough to fix this issue?
(0030974)
Brad King   
2012-09-10 08:45   
Thanks, Eike, that looks good to us.
(0031019)
Rolf Eike Beer   
2012-09-12 15:55   
Fix merged to next: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c10cb2fed7916af9983df814870d6d8de49a868a [^]
(0032447)
Robert Maynard   
2013-03-04 08:38   
Closing resolved issues that have not been updated in more than 4 months.