View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014298CMakeCMakepublic2013-07-17 05:142013-12-02 08:51
ReporterMathieu Malaterre 
Assigned ToBrad King 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake 2.8.9 
Target VersionCMake 2.8.12Fixed in VersionCMake 2.8.12 
Summary0014298: add_test and test with space
DescriptionPlease document that add_test does not deal with test name with spaces.

Eg:

 get_filename_component(filename "${dcmfile}" NAME)
 add_test(NAME "my_app_${filename}"
   COMMAND ${EXECUTABLE_OUTPUT_PATH}/my_app "${dcmfile}"
 )

should really be:

 get_filename_component(filename "${dcmfile}" NAME)
 string(REPLACE " " "_" filename_nospace "${filename}")
 add_test(NAME "my_app_${filename_nospace}"
   COMMAND ${EXECUTABLE_OUTPUT_PATH}/my_app "${dcmfile}"
 )
TagsNo tags attached.
Attached Files

 Relationships

  Notes
(0033563)
Brad King (manager)
2013-07-17 08:16

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d0170584 [^]
(0034649)
Robert Maynard (manager)
2013-12-02 08:51

 Closing resolved issues that have not been updated in more than 4 months.

 Issue History
Date Modified Username Field Change
2013-07-17 05:14 Mathieu Malaterre New Issue
2013-07-17 08:16 Brad King Note Added: 0033563
2013-07-17 08:16 Brad King Assigned To => Brad King
2013-07-17 08:16 Brad King Status new => resolved
2013-07-17 08:16 Brad King Resolution open => fixed
2013-07-17 08:16 Brad King Fixed in Version => CMake 2.8.12
2013-07-17 08:16 Brad King Target Version => CMake 2.8.12
2013-12-02 08:51 Robert Maynard Note Added: 0034649
2013-12-02 08:51 Robert Maynard Status resolved => closed

Copyright © 2000 - 2018 MantisBT Team