| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
Arguably, many of these are bugs in `clang-tidy`. An if/else tree with
other conditionals between cloned blocks may be relying on the
intermediate logic to fall out of the case and inverting this logic may
be non-trivial.
See: https://bugs.llvm.org/show_bug.cgi?id=44165
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
c84cf42897 cmState::GetGlobalProperty: return cmProp
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4521
|
| | |
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Since commit d1976cd1f2 (CTest: Fix timeout when grandchild keeps pipes
open, 2020-01-13, v3.17.0-rc1~169^2) we no longer hang, but the test
duration we report after the timeout is the amount of time the immediate
child ran before exiting. Fix the logic to instead report the actual
amount of time we spent monitoring the test before the timeout.
Fixes: #20509
|
| | |
| | |
| | |
| | | |
This should avoid the runtime strlen() call.
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Modify the original string instead of creating a new copy with substr() when it
is not used for anything else afterwards.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Only one key can match per iteration, avoid any further compares when one match
was already found. While at it entirely avoid that the key and value strings are
copied.
|
| | | |
|
| | | |
|
| | | |
|
| |/
|/|
| |
| | |
While at it avoid creating a new string.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
f964739ead cmCTestRunTest: modernize memory management
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4454
|
| | | |
|
|\ \ \
| | |/
| |/|
| | |
| | |
| | |
| | | |
7a1cce210b CTest: Fix our internal CURL_DEBUGFUNCTION to conform to CURL docs
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4459
|
| | |
| | |
| | |
| | |
| | |
| | | |
The curl debug callback function must return ``0``.
Fixes: #20449
|
| |/
|/| |
|
|/
|
|
| |
Update internals of various classes.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
If no tests were found, the default behavior of CTest is to always log an
error message but to return an error code in script mode only. This option
unifies the behavior of CTest by either returning an error code if no tests
were found or by ignoring it.
Signed-off-by: Stefan Dinkelacker <s.dinkelacker@dkfz-heidelberg.de>
|
|
|
|
|
|
|
|
|
|
|
| |
When a test's process creates its own child and exits, the grandchild
may keep pipes open. Fix CTest logic to correctly timeout if the
grandchild does not exit and close the pipes before the timeout expires.
This was broken by commit b5e21d7d2e (CTest: Re-implement test process
handling using libuv, 2017-12-10, v3.11.0-rc1~117^2) which added an
unnecessary condition to the timeout handling.
Fixes: #20116
|
| |
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
b393b32b4b CTest: Improve error handling when reading resource spec file
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4162
|
| |
| |
| |
| | |
Fixes: #20079
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
676befdf52 ctest: add support for memcheck using Dr. Memory
2db0a65f56 cmCTestMemCheckHandler.cxx: minor refactoring
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4119
|
| | |
| | |
| | |
| | | |
Fixes: #19788
|
| | | |
|
|/ / |
|
|\ \
| |/
| |
| |
| |
| |
| |
| |
| | |
a033bafbe0 Help: Clarify how tests are run if no resource spec file is specified
a64ba0235f CTest: Clarify that resource requirements can be split
f9f294f5fa CTest: Add version field to resource spec file
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4080
|
| |
| |
| |
| | |
Fixes: #20007
|
| | |
|
| | |
|
| |
| |
| |
| | |
Replace use of the term "rerun" with "repeat" to match the public names.
|
|\ \
| |/ |
|
| | |
|
| |
| |
| |
| |
| |
| | |
Renaming these ahead of other refactoring which will use the
previous names. The previous names more accurately reflect their
purpose after this commit anyway (talking about locking and
unlocking rather than allocating and deallocating).
|
| |
| |
| |
| |
| |
| | |
Only changes the user-visible effects of renaming hardware
allocation to resource allocation. Code changes are the minimum
needed to achieve that.
|
| |
| |
| |
| |
| | |
The corresponding test property `PROCESSES` has been renamed to
`RESOURCE_GROUPS`.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The `PROCESSES` test property name added for CMake 3.16 is too close to
the existing `PROCESSORS` test property. Furthermore, the property in
principle specifies groups of resources organized in a way that is
meaningful to a particular test. The groups may often correspond to
processes but they could have other meanings. Since the property name
`PROCESSES` has not been in a final 3.16 release yet, simply rename it
to `RESOURCE_GROUPS`.
Fixes: #19914
|