View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0011554

CMake

CTest

public

2010-12-01 16:09

2011-01-12 07:57

Reporter

Tommy Edvardsen

Assigned To

Brad King

Priority

normal

Severity

text

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

CMake 2.8.3

Target Version

CMake 2.8.4

Fixed in Version

CMake 2.8.4

Summary

0011554: Incorrect description for ctest_sleep command

Description

The description of ctest_sleep command is incorrect and confusing:
"With three arguments it will wait for time2 - time1 - duration seconds."

In source file "cmCTestSleepCommand.cxx" the calculation is as follows:
duration = (time1 + duration - time2);
cmCTestScriptHandler::SleepInSeconds(duration);

The correct description would then be:
"With three arguments it will wait for time1 + duration - time2 seconds."

It is also worth noting that the description should include that no sleep will be performed if time1 + duration is greater than time2.

Tags

No tags attached.

Attached Files

Relationships

Notes
(0023652) Tommy Edvardsen (reporter) 2010-12-01 16:30	Correction to my last sentence. Sleep will only be performed if time1 + duration is greater than time2. For all other situations no sleep will be performed. This should be clearified in the description.

(0023842) Brad King (manager) 2010-12-09 10:38	http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7145ca67 [^]

Issue History
Date Modified	Username	Field	Change
2010-12-01 16:09	Tommy Edvardsen	New Issue
2010-12-01 16:30	Tommy Edvardsen	Note Added: 0023652
2010-12-09 10:38	Brad King	Assigned To	=> Brad King
2010-12-09 10:38	Brad King	Status	new => assigned
2010-12-09 10:38	Brad King	Note Added: 0023842
2010-12-09 10:38	Brad King	Status	assigned => closed
2010-12-09 10:38	Brad King	Resolution	open => fixed
2011-01-12 07:57	David Cole	Fixed in Version	=> CMake 2.8.4
2011-01-12 07:57	David Cole	Target Version	=> CMake 2.8.4