MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0009884CMakeCMakepublic2009-11-11 15:532016-06-10 14:31
ReporterThomas Harning 
Assigned ToKitware Robot 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionmoved 
PlatformOSOS Version
Product VersionCMake-2-6 
Target VersionFixed in Version 
Summary0009884: Commas in the path to a project do not force output to be quoted for VS2008 projects
DescriptionVisual Studio 2008 interprets both semicolon and commas as separators for include paths (at the least).

If a path to a project contains a comma, ex:
C:\MyProject-1,0\
then paths such as the 'include' path are split in such a way when included..

Ex: include_directories(include) in that project will yield:
include paths:
C:\MyProject-1
0\include

CMake needs to know the VS2008 rules for project output. The same is likely to apply to VS 20**
Steps To Reproduce
Additional Information
Tagsstudio, visual
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-11-11 15:53Thomas HarningNew Issue
2009-11-11 15:54Thomas HarningTag Attached: studio
2009-11-11 15:54Thomas HarningTag Attached: visual
2010-12-14 16:13David ColeAssigned To => David Cole
2010-12-14 16:13David ColeStatusnew => assigned
2010-12-17 07:41David ColeTarget Version => CMake 2.8.4
2011-01-03 17:33Bill HoffmanNote Added: 0024370
2011-01-10 14:50David ColeNote Added: 0024550
2011-01-10 14:50David ColeTarget VersionCMake 2.8.4 =>
2011-10-25 22:25David ColeStatusassigned => backlog
2011-10-25 22:27David ColeAssigned ToDavid Cole =>
2016-06-10 14:27Kitware RobotNote Added: 0041615
2016-06-10 14:27Kitware RobotStatusbacklog => resolved
2016-06-10 14:27Kitware RobotResolutionopen => moved
2016-06-10 14:27Kitware RobotAssigned To => Kitware Robot
2016-06-10 14:31Kitware RobotStatusresolved => closed

Notes
(0024370)
Bill Hoffman   
2011-01-03 17:33   
I was not able to reproduce this.... It might be fixed.

foo.vcproj: AdditionalIncludeDirectories=""C:\Users\hoffman\Work\My Builds\test,1\include";"

hoffman@metaverse ~/Work/My Builds/test,1/b
$ cat ../CMakeLists.txt
project(foo)
include_directories(include)
add_library(foo foo.c)

Works with both these versions:
cmake version 2.8.3
cmake version 2.8.3.20110103-ga67fd
(0024550)
David Cole   
2011-01-10 14:50   
Not enough time to get a fix for this into CMake 2.8.4 -- deferring until later.
(0041615)
Kitware Robot   
2016-06-10 14:27   
Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current CMake Issues page linked in the banner at the top of this page.