summaryrefslogtreecommitdiff
path: root/Tests/RunCMake
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'no_buildid_for_files'Brad King2018-10-014-0/+4
|\ | | | | | | | | | | | | c49d13f94b ctest: only create buildid when submitting from Testing/ dir Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2423
| * ctest: only create buildid when submitting from Testing/ dirZack Galbreath2018-09-284-0/+4
| | | | | | | | | | | | | | | | | | | | | | In 7f530cc we taught CTest to pass extra information to CDash at submit time. This extra info is used by CDash to initialize a buildid. `ctest_submit(FILES)` can be used to send specific files to CDash. These files are not necessarily associated with the build currently being performed. For this reason, we modify the behavior of ctest_submit() to only specify this extra info when we are submitting files from the current build's Testing directory.
* | Ninja,Makefile: Add tests for handling of byproducts by clean operationsPedro Navarro2018-09-286-0/+173
| |
* | Merge topic 'vs-global-props-for-all-targets'Brad King2018-09-283-0/+53
|\ \ | |/ |/| | | | | | | | | | | 36489b85aa VS: Add test for CMAKE_VS_GLOBALS 22e670a306 VS: Add option to set VS_GLOBAL_* for all targets Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2345
| * VS: Add test for CMAKE_VS_GLOBALSMikhail Korolev2018-09-283-0/+53
| |
* | Merge topic 'link-directories'Craig Scott2018-09-2617-0/+55
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | f9717725f9 link_directories(): enhance capabilities b5915744eb LINK_DIRECTORIES target property: add policy for absolute paths check. a71caab46b LINK_DIRECTORIES: Add new properties and commands 5ca130e223 Refactoring: introduce function to check if a string is a generator expression Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2403
| * | LINK_DIRECTORIES target property: add policy for absolute paths check.Marc Chevrier2018-09-2514-0/+49
| | |
| * | LINK_DIRECTORIES: Add new properties and commandsMarc Chevrier2018-09-253-0/+6
| |/ | | | | | | | | | | | | | | | | | | | | | | These new capabilities enable to manage link directories Two new properties: * target properties: LINK_DIRECTORIES and INTERFACE_LINK_DIRECTORIES One new command * target_link_directories(): to populate target properties Fixes: #17215
* | Merge topic 'dbgsym-packaging'Brad King2018-09-265-11/+38
|\ \ | |/ |/| | | | | | | | | | | | | d8a3939aef CPack/Deb: Add ability to split out debug symbols into .ddeb package 42fbff45e4 CPack/Deb: Use CMAKE_COMMAND to set the environment 66ab24a4c5 Help: Fix typo Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2399
| * CPack/Deb: Add ability to split out debug symbols into .ddeb packageAndrew Fuller2018-09-215-11/+38
| |
* | Merge topic 'deprecate-policy-old'Craig Scott2018-09-2010-3/+106
|\ \ | | | | | | | | | | | | | | | | | | 0c709cb2a2 Add deprecation warnings for policies CMP0063 and below Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2397
| * | Add deprecation warnings for policies CMP0063 and belowBrad King2018-09-1710-3/+106
| |/ | | | | | | | | | | The OLD behaviors of all policies are deprecated, but only by documentation. Add an explicit deprecation diagnostic for some policies to encourage projects to port away from setting policies to OLD.
* | BundleUtilities: Disallow inclusion at configure timeKyle Edwards2018-09-1910-0/+32
| | | | | | | | | | | | This commit adds a new CMake policy, CMP0080, which prohibits the inclusion of BundleUtilities at configure time. The old behavior is to allow the inclusion.
* | Merge topic 'symlink'Brad King2018-09-193-31/+42
|\ \ | | | | | | | | | | | | | | | | | | afb7f6e4ff cmake: Add '-E create_symlink' support on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2144
| * | cmake: Add '-E create_symlink' support on WindowsJon Chronopoulos2018-09-183-31/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The allows `-E create_symlink` to work on Windows. It utilizes `uv_fs_symlink`. I am still unsure exactly which Windows platforms will work without requiring Administrator privileges or needing a user/group with the "Create Symbolic Links" User Rights. It does work with my Windows 10 Pro with Developer Mode turned on. In the test suite check that the symlink either worked or failed with a permissions error. Use recent changes in cmSystemTools::FileExists to check that a symlink is broken.
* | | Merge topic 'provide_explicit_source_and_build_command_line_options'Brad King2018-09-1911-0/+42
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 638f00117a Add release note for the -S and -B options. de962cc00d CMake: Internally uses -S instead of -H to specify source directory a10d63d578 cmake: -S and -B can be used to specify source and build directories Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2358
| * | | cmake: -S and -B can be used to specify source and build directoriesRobert Maynard2018-09-1511-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | Document the previously internal option of '-B' and provide a matching source directory option with '-S'. Both '-B', and '-S' can be used independently of each other.
* | | | Merge topic 'ctest-more_submit_params'Brad King2018-09-195-0/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7f530cc54e ctest_submit: pass additional info to CDash Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2380
| * | | | ctest_submit: pass additional info to CDashZack Galbreath2018-09-175-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | Specify buildname, site, and buildstamp to CDash upon submission. CDash will use this extra info to assign and report back a buildid.
* | | | | Merge topic 'vs-ipo'Brad King2018-09-182-2/+2
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | bef80e6623 VS: Do not specify incremental linking if LTCG is enabled 567fabe88e IPO: INTERPROCEDURAL_OPTIMIZATION (LTCG) for Visual Studio Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2363
| * | | | IPO: INTERPROCEDURAL_OPTIMIZATION (LTCG) for Visual StudioNiels Dekker2018-09-172-2/+2
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add IPO support for Visual Studio (which is referred to by VS as "Link Time Code Generation" and "Whole Program Optimization"), for VS version >= 10. This allows CMake/VS users to enable IPO by setting property `INTERPROCEDURAL_OPTIMIZATION`. Fixes: #16748
* | | | Merge topic 'out-of-dir-linking'Brad King2018-09-1427-11/+77
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a1ad0a699b target_link_libraries: Allow use with targets in other directories 9bbae5ae28 cmTarget: Future-proof AddLinkLibrary target lookup scope f9cb6f618a cmExportFileGenerator: Use cmGeneratorTarget::ResolveTargetReference 18441a6269 cmGeneratorTarget: Factor target name resolution out of link item resolution 2f708f5d65 Make internal TARGET_PROPERTY generator expressions more robust 94a75801c8 Android.mk: De-duplicate link libraries logic during export 8a63b23d16 cmGlobalGenerator: Remove unused FindLocalGenerator method Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Patrick Stotko <stotko@cs.uni-bonn.de> Merge-request: !2370
| * | | | target_link_libraries: Allow use with targets in other directoriesBrad King2018-09-1227-11/+77
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the command did not allow naming targets on the LHS that were not created in the calling directory. Lift this restriction to enable more flexible use by projects. Targets named on the RHS will need to be looked up during generation in the scope of the call site rather than the scope of the LHS target. Introduce an internal syntax in `[INTERFACE_]LINK_LIBRARIES` properties to specify target names that need to be looked up in a directory other than that containing the target on which the property is set. Add minimal documentation of the syntax to help users that encounter it. Unfortunately CMake previously did allow such calls in the case that only `INTERFACE` libraries are specified, but those libraries would be looked up in the target's directory rather than the caller's. Add policy `CMP0079` to enable the new behavior with new lookup scope in a compatible way. Fixes: #17943
* | | | Tests: Extend RunCMake.Ninja filesystem delaysBrad King2018-09-121-4/+5
|/ / / | | | | | | | | | | | | | | | On filesystems with 1s resolution the `run_sub_cmake` cases fail occasionally when the 1 second sleep does not cause files to have a different time. Use 3 seconds instead.
* | | 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 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 '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 'export-properties-undefined'Craig Scott2018-08-172-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-154-0/+15
| |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check that property INTERFACE_LINK_OPTIONS is correctly propagated from static libraries. Issue: #18251
* | | | | | target_link_options: Add test to verify static library behaviorRobert Maynard2018-08-094-0/+16
| |_|_|_|/ |/| | | | | | | | | | | | | | Issue: #18251
* | | | | Help: Document the $CACHE{} syntaxRobert Maynard2018-08-073-0/+11
| |_|_|/ |/| | | | | | | | | | | Also add a test case for the behavior.