summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'gtest_add_tests-empty-file'Craig Scott2018-09-112-3/+4
|\ | | | | | | | | | | | | | | 31c82143bf GoogleTest: gtest_add_tests() fails if any source file is empty d6b06d8d87 GoogleTest: Modify test to verify that empty files can be scanned Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2342
| * GoogleTest: Modify test to verify that empty files can be scannedCraig Scott2018-09-102-3/+4
| | | | | | The modified test confirms the bug described in issue #18321.
* | Merge topic 'imported-same-name'Brad King2018-09-117-0/+47
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | f35be59961 Fix transitive usage requirements through same-name imported targets 1b57f49586 genex: Simplify cmGeneratorExpressionInterpreter bea390e9bd Fix dependency propagation through same-name imported targets fc7e4d1ed8 cmLinkItem: Convert to a "sum type" over a string and target pointer Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2359
| * | Fix transitive usage requirements through same-name imported targetsBrad King2018-09-103-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If two imported targets in different directories have the same name we should still be able to propagate transitive usage requirements from both. Fix the DAG checker to work with target pointers instead of target names since the pointers will not be duplicated even if the names are. Fixes: #18345
| * | Fix dependency propagation through same-name imported targetsBrad King2018-09-077-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If two imported targets in different directories have the same name we should still be able to propagate transitive link dependencies from both. Fix the target and link dependency analyzers to de-duplicate targets using target pointers rather than target names since the pointers will not be duplicated even if the names are. Issue: #18345
* | | Merge topic 'fix-ctest_start-track'Brad King2018-09-113-1/+10
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 09f0325eaf CTest: Fix regression in ctest_start() Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2366
| * | | CTest: Fix regression in ctest_start()Kyle Edwards2018-09-103-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The format for the TAG file was changed in 3.12, and the way it was read caused a regression which changed how the track was decided. This commit fixes the regression. Fixes #18347.
* | | | Merge topic 'ctest-fix-test-load'Brad King2018-09-1114-18/+30
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 292ec157b6 CTest: Fix --test-load regression Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2362
| * | | | CTest: Fix --test-load regressionBrad King2018-09-1014-18/+30
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `ctest --test-load` option is implemented in `StartNextTests` by not starting any tests when the load is too high and instead sleeping and then returning. Prior to commit v3.11.0-rc1~117^2 (CTest: Re-implement test process handling using libuv, 2017-12-10) our outer loop in `RunTests` would immediately call `StartNextTests` again. However, now the `uv_run` loop may simply terminate if there are no tests running because no events are left pending. Fix this by converting the sleep in `StartNextTests` into a libuv timer that it starts instead. This avoids leaving `uv_run` with no pending events. In the case that there are other running tests this also allows CTest to detect when they finish even if it during the wait period where we previously slept. This regression was not caught by the test suite because it only verified that we do not start new tests when the load was too high and not that we proceed to start tests when the load drops. Revise the test suite to cover both. Fixes: #18338
| * | | Merge branch 'android-ndk-r18' into release-3.12Brad King2018-08-282-6/+12
| |\ \ \ | | | | | | | | | | | | | | | Merge-request: !2319
| * \ \ \ Merge branch 'cuda-no-cublas_device' into release-3.12Brad King2018-08-231-0/+6
| |\ \ \ \ | | | | | | | | | | | | | | | | | | Merge-request: !2296
* | \ \ \ \ Merge topic 'ExternalProject-check-explicit-include'Craig Scott2018-09-077-0/+43
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | df1ddeec12 ExternalProject: Report error if local variables are not defined Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Brad King <brad.king@kitware.com> Merge-request: !2281
| * | | | | ExternalProject: Report error if local variables are not definedJean-Christophe Fillion-Robin2018-09-067-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since in some situations, ExternalProject module may be included in a sub-directory, functions will be available in the global scope but local variables like "_ep_keywords_<keyword>" will not be defined, this commit checks and reports an error indicating that the ExternalProject module must be explicitly included before using any of the ExternalProject_* functions that require the module's inclusion within the current scope or above. Co-authored-by: Pablo Hernandez <pablo.hernandez@kitware.com> Co-authored-by: Craig Scott <craig.scott@crascit.com>
* | | | | | Merge topic 'cpack_test'Brad King2018-09-042-1/+20
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7a7e94d6f7 CPack (DEB/RPM): add test for per-component description/summmary. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2333
| * | | | | CPack (DEB/RPM): add test for per-component description/summmary.David Faure2018-08-302-1/+20
| | | | | |
* | | | | | Merge topic 'no-sanitizer-logfile'Brad King2018-08-3110-10/+0
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5aaaee5e9e ctest_memcheck: Change failure to find log file from error to warning Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2323
| * | | | | ctest_memcheck: Change failure to find log file from error to warningBetsy McPhail2018-08-2810-10/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sanitizers do not create a log file when no defects are found. Therefore, it is currently impossible for ctest_memcheck to set both `CAPTURE_CMAKE_ERROR` and `RETURN_VALUE` to zero. With defects, `CAPTURE_CMAKE_ERROR`=0 and `RETURN_VALUE`=-1, as expected. With no defects, `CAPTURE_CMAKE_ERROR`=-1 and `RETURN_VALUE`=0.
* | | | | | Merge topic 'cpack-ext-stage-and-run'Brad King2018-08-305-2/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4a0f664aaf CPackExt: Add CPACK_EXT_ENABLE_STAGING and CPACK_EXT_PACKAGE_SCRIPT Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2272
| * | | | | | CPackExt: Add CPACK_EXT_ENABLE_STAGING and CPACK_EXT_PACKAGE_SCRIPTNils Gladitz2018-08-145-2/+31
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CPACK_EXT_ENABLE_STAGING enables optional staging and CPACK_EXT_PACKAGE_SCRIPT allows to specify an optional script file that can package staged files via an external packaging tool. Issue: #18236
* | | | | | Merge topic 'FindPackageModeMakefileTest-use-explicit-linkage'Brad King2018-08-302-0/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 03f22cd188 FindPackageModeMakefileTest: Explicitly use C linkage Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2328
| * | | | | | FindPackageModeMakefileTest: Explicitly use C linkageChuck Atkins2018-08-292-0/+10
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This explicitly uses C linkage for the library code used by the test to work around some compilers (Solaris Studio 12.4) that implicitly assume incorrect linkage information. Since something else entirely is being tested here, this allows the test to proceed as needed on the affected platforms.
* | | | | | Merge topic 'android-ndk-r18'Brad King2018-08-302-6/+12
|\ \ \ \ \ \ | |/ / / / / |/| | | | / | | |_|_|/ | |/| | | | | | | | | | | | | ca97d4cb5f Android: Add support for NDK r18 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2319
| * | | | Android: Add support for NDK r18Brad King2018-08-282-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NDK r18 drops GCC toolchains and some STL types. We need to choose a clang toolchain by default when no gcc toolchains are available. Switch the STL type default to `c++_static` when the old `gnustl_static` default is not available. Update the test suite to not run tests for STL types that do not exist. Also do not expect the gcc toolchain `cpp` tool to be available because r18 does not provide it. Also teach it to tolerate `gcc -dumpmachine` output like `arm--linux-android` that differs from the toolchain prefix. Fixes: #18301
* | | | | Merge topic 'static-lib-options'Brad King2018-08-279-0/+73
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 974de0e199 static library: add property STATIC_LIBRARY_OPTIONS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2290
| * | | | | static library: add property STATIC_LIBRARY_OPTIONSMarc Chevrier2018-08-159-0/+73
| | | | | | | | | | | | | | | | | | | | | | | | issue: #18251
* | | | | | Merge topic 'ninja-generator-do-not-add-alias-matching-custom-command-output'Brad King2018-08-272-0/+48
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 180c2f8de8 Ninja: Fix dupbuild error skipping alias with conflicting custom command Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Isaiah <isaiah.norton@gmail.com> Merge-request: !2276
| * | | | | | Ninja: Fix dupbuild error skipping alias with conflicting custom commandJean-Christophe Fillion-Robin2018-08-092-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See-also: https://issues.slicer.org/view.php?id=4595 Reported-by: Isaiah Norton <inorton@bwh.harvard.edu>
* | | | | | | Merge topic 'cuda-no-cublas_device'Brad King2018-08-241-0/+6
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 122c79748c CUDA: Avoid using deprecated cublas_device to identify device lib dirs Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2296
| * | | | | | CUDA: Avoid using deprecated cublas_device to identify device lib dirsRobert Maynard2018-08-231-0/+6
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use `curand_static` to identify directories containing device libraries because `cublas_device` is deprecated and will be removed in the future. Issue: #18290
* | | | | | Merge topic 'export-properties-undefined'Craig Scott2018-08-182-0/+12
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dbd3e2c53d EXPORT_PROPERTIES: Prevent null dereference for undefined property b88bf6796e EXPORT_PROPERTIES: Add test for an undefined property Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2294
| * | | | | EXPORT_PROPERTIES: Add test for an undefined propertyCraig Scott2018-08-162-0/+12
| |/ / / / | | | | | | | | | | | | | | | The added test verifies that a property listed in EXPORT_PROPERTIES can be undefined. It confirms the crash recorded in issue #18260.
| * | | | Merge branch 'cpack-default-package-version-zero' into release-3.12Craig Scott2018-07-297-0/+24
| |\ \ \ \ | | | | | | | | | | | | | | | | | | Merge-request: !2239
| * \ \ \ \ Merge branch 'project-injected-no-cmp0048' into release-3.12Craig Scott2018-07-254-0/+16
| |\ \ \ \ \
* | | | | | | LINK_OPTIONS property: add test for static library.Marc Chevrier2018-08-155-1/+23
| |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check that property INTERFACE_LINK_OPTIONS is correctly propagated from static libraries. Issue: #18251
* | | | | | Merge topic 'doc-consistently-reference-packagename'Craig Scott2018-08-151-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 30d08bdce9 doc: Consistently use <PackageName> for describing the name of a package Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2283
| * | | | | | doc: Consistently use <PackageName> for describing the name of a packageJean-Christophe Fillion-Robin2018-08-141-1/+1
| | |_|_|_|/ | |/| | | |
* | | | | | Merge topic 'better_static_link_flags_docs'Craig Scott2018-08-154-0/+16
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6b17d85873 Help: Document when LINK_FLAGS and STATIC_LIBRARY_FLAGS are used 9561d42f95 target_link_options: Add test to verify static library behavior Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Robert Maynard <robert.maynard@kitware.com> Merge-request: !2274
| * | | | | target_link_options: Add test to verify static library behaviorRobert Maynard2018-08-094-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | Issue: #18251
* | | | | | Merge topic 'test_MocOsMacros_fix'Brad King2018-08-107-20/+36
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 52676e3794 Autogen: Tests: Use non inverting Qt version tests e7fb85d00d Autogen: Check if we can run the MocOsMacros test Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2270
| * | | | | Autogen: Tests: Use non inverting Qt version testsSebastian Holtermann2018-08-096-11/+11
| | | | | |
| * | | | | Autogen: Check if we can run the MocOsMacros testSebastian Holtermann2018-08-072-9/+25
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The MocOsMacros test requires moc to support moc_predefs.h on some platforms (e.g. Mac). moc supports moc_predefs.h inclusion since Qt 5.8, therefore run the MocOsMacros only if Qt >= 5.8 and if CMAKE_CXX_COMPILER_PREDEFINES_COMMAND is defined. Fixes #18245
* | | | | Help: Document the $CACHE{} syntaxRobert Maynard2018-08-073-0/+11
|/ / / / | | | | | | | | | | | | Also add a test case for the behavior.
* | | | Xcode: Add variables and properties to configure schemesGregor Jasny2018-08-013-0/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add `XCODE_SCHEME_*` target properties and associated variables `CMAKE_XCODE_SCHEME_*` to initialize them on target creation. Map each target property value to an associated Xcode scheme entry. Co-Author: Martin Sander <mail@martin-sander.de> Fixes: #17919
* | | | Autogen: Add test for Q_OS_ macrosSebastian Holtermann2018-07-305-0/+180
| | | |
* | | | Merge topic 'cpack-default-package-version-zero'Craig Scott2018-07-297-0/+24
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | ed015bde2b CPack: Restore support for 0-valued version components Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2239
| * | | CPack: Restore support for 0-valued version componentsBrad King2018-07-277-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit v3.12.0-rc1~136^2 (CPack: Use project version as default for `CPACK_PACKAGE_VERSION`, 2018-04-29) we did not account for the value of `CMAKE_PROJECT_VERSION_{MAJOR,MINOR,PATCH}` having `0`. Fix the logic to distinguish between unprovided version components and `0` components. While at it, add a test case covering the behavior described in the documentation by the original commit. The number of version components in the package name should match those provided to the `project()` command `VERSION` option. Fixes: #18199
* | | | Merge topic 'UseSWIG-target-name-policy'Brad King2018-07-2720-12/+50
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a3a0c3aa71 UseSWIG: add policy to manage target naming strategy. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2232
| * | | | UseSWIG: add policy to manage target naming strategy.Marc Chevrier2018-07-2520-12/+50
| | | | |
* | | | | Merge topic 'reproducible-tarballs'Brad King2018-07-276-2/+72
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | 548ac51d8e CPack/Deb: Support SOURCE_DATE_EPOCH when packaging files Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2226
| * | | | CPack/Deb: Support SOURCE_DATE_EPOCH when packaging filesAndrew Fuller2018-07-256-2/+72
| | | | |
generated by cgit v0.12 at 2025-02-05 09:03:11 (GMT)