View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004524CMakeCMakepublic2007-03-02 06:542008-10-07 16:24
ReporterMarco Wedekind 
Assigned ToBill Hoffman 
PrioritylowSeverityfeatureReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0004524: support for source code control integration in Visual Studio projects
DescriptionIt would be nice to be able to configure source control bindings for Visual Studio project files. You need to specify three parameters on a per-target basis: SCCPROJECTNAME, SCCLOCALPATH and SCCPROVIDER.

I propose to add three according target properties: VS_SCCPROJECTNAME, VS_SCCLOCALPATH and VS_SCCPROVIDER. They could be set via SET_TARGET_PROPERTIES. In cmLocalVisualStudio7Generator::WriteProjectStart() they would be read and their values added to the generated project files. A patch of cmLocalVisualStudio7Generator::WriteProjectStart() to achieve this behaviour is attached.
TagsNo tags attached.
Attached Filestxt file icon diff_cmLocalVisualStudio7Generator_WriteProjectStart.txt [^] (1,249 bytes) 1969-12-31 19:00 [Show Content]

 Relationships
related to 0005813closedBill Hoffman Support for Visual Source Safe plugin in MSVC 
related to 0010237closedBrad King Target properties for visual sourcesafe bindings do not work on Fortran (vfproj) projects 

  Notes
(0013696)
James Bigler (developer)
2008-10-01 17:38

Perhaps this could be generalized to allow for arbitrary key value pairs, so that the user could be free to add whatever attributes to the project's XML node they wish. This way one could do:

set(target PROPERTIES
  VS_KEYVALUE "SccProjectName=\"Perforce Project\""
  VS_KEYVALUE "SccLocalPath=\".\""
  VS_KEYVALUE "SccProvider=\"MSSCCI:Perforce SCM\""
  )

BTW, the patch submitted has a minor bug:

+ << providerString

Should be:

+ << "\tSccProvider=\"" << providerString << "\"\n"

Anyway, I would be happy to add a patch for this, if it would help it get integrated.
(0013758)
Bill Hoffman (manager)
2008-10-07 16:24

I changed the property names a bit, but decided against the keyvalue stuff as you can not have more than one property on a target with the same name. Anyway, here is the commit:

 cvs commit -m "BUG: fix for 4524, add support for target properties to set vs source code control information"
Committer: Bill Hoffman <bill.hoffman@kitware.com>
/cvsroot/CMake/CMake/Source/cmLocalVisualStudio7Generator.cxx,v <-- Source/cmLocalVisualStudio7Generator.cxx
new revision: 1.232; previous revision: 1.231
/cvsroot/CMake/CMake/Source/cmSetTargetPropertiesCommand.h,v <-- Source/cmSetTargetPropertiesCommand.h
new revision: 1.32; previous revision: 1.31

 Issue History
Date Modified Username Field Change
2008-10-01 17:38 James Bigler Note Added: 0013696
2008-10-07 15:14 Bill Hoffman Relationship added related to 0005813
2008-10-07 16:24 Bill Hoffman Status assigned => closed
2008-10-07 16:24 Bill Hoffman Note Added: 0013758
2008-10-07 16:24 Bill Hoffman Resolution open => fixed
2011-01-17 15:34 Brad King Relationship added related to 0010237


Copyright © 2000 - 2018 MantisBT Team