summaryrefslogtreecommitdiffstats
path: root/Source/CTest
Commit message (Collapse)AuthorAgeFilesLines
* clang-tidy: Enable performance-inefficient-string-concatenationSebastian Holtermann2019-08-055-17/+13
| | | | | | | Enables the clang-tidy test performance-inefficient-string-concatenation and replaces all inefficient string concatenations with `cmStrCat`. Closes: #19555
* Merge topic 'control-block3'Brad King2019-08-051-31/+2
|\ | | | | | | | | | | | | | | | | | | | | | | 41364824ad cmFunctionBlocker: Recycle functions 6491270e0d cmFunctionBlocker: Move check for matching args af24e4ef6e cmFunctionBlocker: Move common logic to base ef38ff22f7 cm*FunctionBlocker: Extract function Replay b51fba6298 cmMakefile: Add OnExecuteCommand callback c76500949d cm*FunctionBlocker: Move to source file Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3632
| * cmMakefile: Add OnExecuteCommand callbackRegina Pfeifer2019-07-301-31/+2
| | | | | | | | | | In cmCTestScriptHandler, port away from cmFunctionBlocker and update the elapsed time with the new callback instead.
* | Cleanups: Use cmHas{Prefix,Suffix} instead of String{Starts,Ends}WithSebastian Holtermann2019-08-013-3/+6
| |
* | Merge topic 'clang-tidy-8'Brad King2019-07-312-2/+3
|\ \ | |/ |/| | | | | | | | | 4af094c8df clang-tidy: Blacklist violations for version 8 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3627
| * clang-tidy: Blacklist violations for version 8Regina Pfeifer2019-07-302-2/+3
| | | | | | | | | | Check the codebase with clang-tidy version 8, fix the low hanging fruits, blacklist the rest.
* | cmStringAlgorithms: Move string functions to the new cmStringAlgorithms.hSebastian Holtermann2019-07-295-3/+5
|/ | | | | This adds the `cmStringAlgorithms.h` header and moves all string functions from `cmAlgorithms.h` to `cmStringAlgorithms.h`.
* Merge topic 'definitions_string_view'Brad King2019-07-267-22/+17
|\ | | | | | | | | | | | | | | | | | | | | e91bfe440c cmMakefile: Let AddDefinition accept a value as cm::string_view f2ba968ef2 cmMakefile: Simplify and rename AddDefinitionBool 9b5cc42531 cmDefinitions: Remove const char* based Set method e268840c0a cmDefinitions: Add Unset and cm::string_view based Set methods 451fd329a8 cmDefinitions: Cleanups Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3577
| * cmMakefile: Let AddDefinition accept a value as cm::string_viewSebastian Holtermann2019-07-247-21/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes `cmMakefile::AddDefinition` to take a `cm::string_view` as value argument instead of a `const char *`. Benefits are: - `std::string` can be passed to `cmMakefile::AddDefinition` directly without the `c_str()` plus string length recomputation fallback. - Lengths of literals passed to `cmMakefile::AddDefinition` can be computed at compile time. In various sources uses of `cmMakefile::AddDefinition` are adapted to avoid `std::string::c_str` calls and the `std::string` is passed directly. Uses of `cmMakefile::AddDefinition`, where a `nullptr` `const char*` might be passed to `cmMakefile::AddDefinition` are extended with `nullptr` checks.
| * cmMakefile: Simplify and rename AddDefinitionBoolSebastian Holtermann2019-07-241-1/+1
| | | | | | | | | | | | | | This simplifies the `cmMakefile::AddDefinition` method with bool value overload to call the string based `cmMakefile::AddDefinition` method with either an "ON" or "OFF" string. Also the method is renamed to `cmMakefile::AddDefinitionBool`
* | cmCTestHandlerCommand: fix typo of local variableBen Boeckel2019-07-241-7/+7
|/
* Merge topic 'command-refactoring3'Brad King2019-07-231-9/+8
|\ | | | | | | | | | | | | | | | | | | | | | | dfaa87f1b3 cmState: Support BuiltinCommands as free functions 28f2d12a05 cmCommand: De-virtualize function InvokeInitialPass de77d355ac cmState: Add scripted commands by value 0101ace131 cmUnexpectedCommand: Replace with lambda expression 015001aaf1 cmState: Hold commands by value 1eebc29563 cmCommand: deprecate functions GetMakefile and SetError Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3574
| * cmCommand: deprecate functions GetMakefile and SetErrorDaniel Pfeifer2019-07-211-9/+8
| | | | | | | | | | | | | | | | | | Replace the members for the Makefile and the Error with a cmExecutionStatus. Re-implement GetMakefile and SetError based on that. Both functions should be called directly on the cmExecutionStatus that is passed to InitialPass. This will help us make all Commands immutable and remove the need for cloning.
* | Refactor: Break cmCTestTestHandler::ProcessHandler() into smaller functionsKyle Edwards2019-07-192-137/+181
|/
* Merge topic 'skip_regular_expression'Kyle Edwards2019-07-194-5/+38
|\ | | | | | | | | | | | | | | | | | | 5dbd9c8583 Vim: Add SKIP_REGULAR_EXPRESSION test property to vim syntax file 407dd1a910 Help: Add documentation for SKIP_REGULAR_EXPRESSION test property d7955d4e5d Tests: Create test for SKIP_REGULAR_EXPRESSION test property 4f1dec86a5 CTest: Add SKIP_REGULAR_EXPRESSION test property Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3570
| * CTest: Add SKIP_REGULAR_EXPRESSION test propertyGabor Bencze2019-07-184-5/+38
| |
* | Merge topic 'ctest-done-hash'Brad King2019-07-191-9/+9
|\ \ | | | | | | | | | | | | | | | | | | da79075c4d CTest: Generate Done.xml before calculating its hash Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3563
| * | CTest: Generate Done.xml before calculating its hashBetsy McPhail2019-07-191-9/+9
| | | | | | | | | | | | Fixes: #19489
* | | cmMakefile: Modernize AddFunctionBlocker method to accept a std::unique_ptrSebastian Holtermann2019-07-171-10/+14
| |/ |/|
* | modernize: manage cmCommand instances using unique_ptr.Marc Chevrier2019-07-1417-100/+153
| |
* | Introduce memory management helper: cm_memory.hxxMarc Chevrier2019-07-142-3/+4
| |
* | Merge topic 'ctest_make_program__propagated_to_cmake_configure_pass'Brad King2019-07-121-0/+5
|\ \ | | | | | | | | | | | | | | | | | | cd681f1f41 ctest: propagate make program to cmake Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3483
| * | ctest: propagate make program to cmakeRobert Maynard2019-07-111-0/+5
| |/
* | Merge topic 'iwyu-memory'Brad King2019-07-115-4/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | 71fbebd1dc IWYU: Fix handling of <memory> standard header Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Marc Chevrier <marc.chevrier@gmail.com> Merge-request: !3536
| * | IWYU: Fix handling of <memory> standard headerBrad King2019-07-105-4/+6
| |/ | | | | | | | | | | | | | | An old workaround for `std::allocator_traits<>::value_type` lints from IWYU on `std::vector<>` usage breaks IWYU's handling of `<memory>`. Convert the workaround to use the same approach we already use for a workaround of `std::__decay_and_strip<>::::__type` lints. Then update the `<memory>` inclusions to follow the now-correct IWYU lints.
* | CTest: Clean up stdout and logfile output for Pass/Fail regexStephen Manz2019-07-091-9/+12
|/ | | | | | | | | | For pass regex, display only the element that was found, rather than all elements Rename loop variable for fail regex, from pass to fail For consistency, add space in output for pass Add tests that find and don't find PASS_REGULAR_EXPRESSION, and a test that finds FAIL_REGULAR_EXPRESSION, whose LastTest.log files are checked using *-check.cmake.
* Use cmAppend to append ranges to std::vector instancesSebastian Holtermann2019-05-2310-51/+35
|
* cmSystemTools::Error(): remove const char* overloadVitaly Stakhovsky2019-05-221-1/+2
|
* ctest: Add new variable CTEST_UPDATE_VERSION_OVERRIDEZack Galbreath2019-05-075-0/+25
| | | | | Set this in a CTest script to explicitly define what the current revision will be reported as in Update.xml.
* Refactor: Move/rename cmProcessGetPipes() to cmGetPipes()Kyle Edwards2019-04-251-44/+2
|
* ctest: consistently format generator nameZack Galbreath2019-04-231-1/+1
| | | | | | | Notes.xml and Upload.xml were missing a hyphen that the rest of our XML files included. ctest3.14.0-rc1 vs. ctest-3.14.0-rc1
* ctest: Include ChangeId in Update.xmlZack Galbreath2019-04-231-0/+4
|
* ctest: drop incremental progress-only messages from build-and-test outputBrad King2019-04-121-3/+4
| | | | | | The cmake progress callback is called with non-negative progress values to provide incremental updates on "Configuring" and "Generating" steps. These are useful in interactive views but are just noise in test output.
* Modernize: Enable modernize-raw-string-literal in clang-tidyArtur Ryt2019-04-023-21/+21
|
* Merge topic 'utilize-substr'Brad King2019-04-015-8/+8
|\ | | | | | | | | | | | | | | 2d66567dca Modernize: Prefer .substr in place of .c_str() + int Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Sebastian Holtermann <sebholt@xwmw.org> Merge-request: !3169
| * Modernize: Prefer .substr in place of .c_str() + intArtur Ryt2019-03-305-8/+8
| | | | | | | | | | | | A lot of temporary/local strings were created out of C-strings substr can utilize current string size, so in theory be a little more efficient.
* | Fix invalid ///! doxygen comment line startsSebastian Holtermann2019-03-313-6/+6
|/ | | | | In various places `///!` was used to start a comment line. This is not valid Doygen syntax. This patch replaces `///!` comment starts with `//!`.
* Merge topic 'jacoco_parser_crash'Brad King2019-03-271-0/+1
|\ | | | | | | | | | | | | 1d16eae868 ctest_coverage: fix out-of-bounds index in Jacoco parser Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3147
| * ctest_coverage: fix out-of-bounds index in Jacoco parserSylvain Joubert2019-03-261-0/+1
| | | | | | | | | | | | | | | | | | | | When the current source file is not found the FilePath variable was left with the previous path content. In case the previous file had less lines than the current one and there are 'line' entries for the current one with higher number we ended up in a buffer overflow while indexing the previous file entry with a line number higher. By clearing the FilePath, the empty() guard triggers correctly on an empty path and it avoid modifying the wrong data.
* | ctest_submit: Add option BUILD_IDZack Galbreath2019-03-222-21/+40
| | | | | | | | Fixes: #18968
* | Merge topic 'ctest-devirtualize-handlers'Brad King2019-03-2115-93/+55
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5a72dbd40c cmCTest: Remove dead code a5eeb0310d cmCTest: Cleanup typedefs 71a3391b5f cmCTest: Use default member initialization b172a81d55 cmCTest: Use concrete accessor functions for TestingHandlers 46090c2337 cmCTest: Store TestingHandlers by value da5ee509b2 cmCTest: Move all data members to private implementation struct 670d27da64 cmCTest: Remove friend declarations 9406844616 cmCTest: De-inline all member functions Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Daniel Pfeifer <daniel@pfeifer-mail.de> Merge-request: !3114
| * | cmCTest: Cleanup typedefsRegina Pfeifer2019-03-207-21/+15
| | |
| * | cmCTest: Use concrete accessor functions for TestingHandlersRegina Pfeifer2019-03-209-69/+34
| | |
| * | cmCTest: Remove friend declarationsRegina Pfeifer2019-03-181-3/+6
| | |
* | | Rename cmFileTimeCache::FileTimeCompare to cmFileTimeCache::CompareSebastian Holtermann2019-03-181-1/+1
| | |
* | | Rename cmFileTimeComparison to cmFileTimeCacheSebastian Holtermann2019-03-181-4/+4
|/ / | | | | | | | | The name `cmFileTimeCache` reflects the functionality of the class more appropriately.
* | Merge topic 'feature/cleanup-ctest'Kyle Edwards2019-03-073-21/+18
|\ \ | | | | | | | | | | | | | | | | | | 1166aa5ce7 ctest: refactor some code Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3064
| * | ctest: refactor some codeGregor Jasny2019-03-053-21/+18
| | |
* | | cmake: Teach --build mode to support multiple targetsBartosz Kosiorek2019-03-051-1/+1
|/ / | | | | | | Fixes: #16136
* | Merge topic 'cmrange-improvements'Brad King2019-02-252-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b8031308f3 cmRange: Add unit tests a8d51ef8b7 cmRange: Add functions filter and transform da4773e8b8 cmRange: Add functions all_of, any_of, none_of 17a367e77f cmRange: Stylistic cleanup 9eb0e73f46 cmRange: Move to dedicated header file Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Artur Ryt <artur.ryt@gmail.com> Merge-request: !2972