summaryrefslogtreecommitdiffstats
path: root/Tests/RunCMake
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'check-keywords-only-if-used'Brad King2018-11-0222-0/+88
|\ | | | | | | | | | | | | | | | | | | 4babc9058a cmTargetPropCommandBase: check keywords after parsing 45a49ae58a cmTargetPropCommandBase: simplify code path 9f64974f5e cmTargetPropCommandBase: skip property setting if there's nothing to add 4201a11c2b Tests: add tests for empty-value keyword arguments in target_* Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2514
| * cmTargetPropCommandBase: check keywords after parsingBen Boeckel2018-10-2614-40/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | The following was disallowed: add_library(iface INTERFACE) target_link_libraries(iface PUBLIC) just due to the mention of the `PUBLIC` keyword. Instead, only error if there are actually `PUBLIC` dependencies specified (and analogously for other restrictions). Update tests to expect this new behavior.
| * Tests: add tests for empty-value keyword arguments in target_*Ben Boeckel2018-10-2636-0/+128
| | | | | | | | | | | | Not all of these commands accept non-compilable (i.e., IMPORTED) targets, so those calls are currently just commented out. If they ever do start to accept them, the tests should be enabled.
* | Merge topic 'fix-custom-target-with-csharp'Brad King2018-11-013-0/+13
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 9040df31e2 Merge branch 'backport-fix-custom-target-with-csharp' 1acd1c2b50 CSharp: Fix regression in VS project type selection for custom target a56edad6d6 CSharp: Fix regression in VS project type selection for custom target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2549
| * | CSharp: Fix regression in VS project type selection for custom targetBrad King2018-10-313-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A target created by `add_custom_target` should always be a `.vcxproj` file even if it has `.cs` sources involved in custom commands and such. The latter case was broken by refactoring in commit v3.12.0-rc1~160^2~7 (remove TargetIsCSharpOnly() and use methods from cmGeneratorTarget, 2018-03-19). The reason is that the `HasLanguage` method added by commit v3.12.0-rc1~239^2~6 (cmGeneratorTarget: add HasLanguage() as wrapper for GetLanguages(), 2018-03-19) does not check the target type and so is not a suitable check for deciding the project file extension. The `HasLanguage` method was an attempt at an abstraction that turns out not to work very well. Replace it with a dedicated `IsCSharpOnly` method that considers the target type, sources, and non-transitive `LINKER_LANGUAGE`. Fixes: #18515
| * | Merge branch 'GNUInstallDirs-kfreebsd-man' into release-3.13Brad King2018-10-221-1/+1
| |\ \ | | | | | | | | | | | | Merge-request: !2511
* | \ \ Merge topic 'set_directory_properties-script-mode'Brad King2018-10-303-0/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 50572d638b set_directory_properties: Restore in script mode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2544
| * | | | set_directory_properties: Restore in script modeBrad King2018-10-293-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.10.0-rc1~391^2~3 (Add directory property 'LABELS' and CMAKE_DIRECTORY_LABELS variable, 2017-06-23) this command was accidentally not allowed in script mode. It was dropped because `ctest -S` mode needs to start with CMake's normal script mode and then replace the `set_directory_properties` implementation. Restore the normal `set_directory_properties` in script mode and then add special logic to replace it in ctest. Also add a test case. Fixes: #18523
* | | | | Merge topic 'minusCparse'Brad King2018-10-2912-1/+22
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 379e5f93a9 Tests: Add cases for -{C,D,U} without a source tree 5873815fef cmake: distinguish '-Cpath' from '-C path' in source dir parsing Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2517
| * | | | | Tests: Add cases for -{C,D,U} without a source treeBrad King2018-10-2612-1/+22
| | |_|_|/ | |/| | |
* | | | | RPATH: Add option for using $ORIGIN in build treePeter Wu2018-10-264-1/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes binaries independent of the build directory by not embedding the build directory via RPATH. The tests are partially based on the existing RuntimePath test, but with the check moved into a POST_BUILD command such that it can be skipped when the platform lacks support. Fixes: #18413
* | | | | Merge topic 'GNUInstallDirs-kfreebsd-man'Brad King2018-10-231-1/+1
|\ \ \ \ \ | |/ / / / |/| | | / | | |_|/ | |/| | | | | | | | | | b2d7ab8bd1 GNUInstallDirs: Don't use BSD info and man paths on GNU/kFreeBSD Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2511
| * | | GNUInstallDirs: Don't use BSD info and man paths on GNU/kFreeBSDJames Clarke2018-10-221-1/+1
| | | |
* | | | Merge branch 'master' into deployqt4-cmp0080-fixKyle Edwards2018-10-19163-190/+340
|\ \ \ \ | |_|_|/ |/| | |
| * | | Merge topic 'better-empty-list-behavior'Brad King2018-10-1721-26/+38
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 121a036f73 cmListCommand: handle empty lists for list(REMOVE_AT) acfe53c588 cmListCommand: make list(ACTION not_a_list) succeed when idempotent bf572ac952 cmListCommand: check list(FILTER) operation before the list Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2478
| | * | | cmListCommand: handle empty lists for list(REMOVE_AT)Ben Boeckel2018-10-167-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | Treat an empty list as a list with no valid bounds and return an error message indicating that any given indices are out-of-bounds.
| | * | | cmListCommand: make list(ACTION not_a_list) succeed when idempotentBen Boeckel2018-10-1615-25/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The operations changed here all are no-ops on empty lists anyways, so just have them succeed when given non-extant lists. - `list(REMOVE_ITEM)` - `list(REMOVE_DUPLICATES)` - `list(SORT)` - `list(FILTER)` - `list(REVERSE)`
| * | | | Merge topic 'document-dollar-in-varnames'Brad King2018-10-166-0/+41
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 82a4822610 CMP0053: document that `$` is a valid literal variable character Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2479
| * \ \ \ \ Merge topic 'cppcheck-exit-code'Brad King2018-10-126-9/+18
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3b80cd77fb Fail the build if cppcheck returns a non-zero exit code Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2459
| | * | | | | Fail the build if cppcheck returns a non-zero exit codeHarald Brinkmann2018-10-116-9/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows the build failure to be tuned with cppcheck's options --error-exitcode=<n> and --exitcode-suppressions=<file>.
| * | | | | | Merge branch 'backport-revert-install-code-script-genex' into ↵Brad King2018-10-116-19/+0
| |\ \ \ \ \ \ | | |_|_|/ / / | |/| | | | | | | | | | | | revert-install-code-script-genex
| * | | | | | Merge topic 'cmake_policy-get_warning'Brad King2018-10-112-10/+16
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0d988f98e5 cmake_policy: Add undocumented GET_WARNING command f9f96598df Help: Convert FindOpenGL documentation to block comment Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2472
| | * | | | | | cmake_policy: Add undocumented GET_WARNING commandKyle Edwards2018-10-102-10/+16
| | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This command is intended for modules that issue policy warnings so they can get the warning string from CMake in a uniform manner, rather than duplicating the string. Several modules been updated to include an example of the usage of this new command.
| * | | | | | Merge topic 'install-subdirectory-order'Brad King2018-10-11131-145/+262
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b56f2db87a Testing: Add test for CMP0082 fc8955e889 add_subdirectory: Run subdirectory install rules in correct order 514f0b572e Testing: Update hard-coded line numbers to [0-9]+ in some tests Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2434
| | * | | | | | Testing: Add test for CMP0082Kyle Edwards2018-10-1035-2/+119
| | | | | | | |
| | * | | | | | Testing: Update hard-coded line numbers to [0-9]+ in some testsKyle Edwards2018-10-1096-143/+143
| | |/ / / / /
| * | | | | | Merge topic 'ctest-done'Brad King2018-10-113-0/+5
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a6e0158712 ctest_submit: Add support for a "Done" part f460bbd4c8 ctest_submit: Refactor file list to use a vector instead of a set Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Zack Galbreath <zack.galbreath@kitware.com> Merge-request: !2405
| | * | | | | ctest_submit: Add support for a "Done" partBetsy McPhail2018-10-103-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Teach CTest to submit Done.xml. Submission of this file indicates to CDash that a build is complete and no more files will be uploaded. It contains the build id returned by CDash and the current time. This file is submitted last for a given build when using the `ctest_submit()` command. If submitting by PARTS, use `ctest_submit(PARTS Done)`.
| * | | | | | Merge topic 'project-always-set-desc-url'Brad King2018-10-082-0/+27
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f8a086a86b project(): Ensure DESCRIPTION and HOMEPAGE_URL variables are set b27247c8df project(): Add tests verifying variables set by second project() call Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2457
* | | | | | | Merge branch 'document-dollar-in-varnames' into release-3.13Brad King2018-10-166-0/+41
|\ \ \ \ \ \ \ | | |_|_|/ / / | |/| | | | | | | | | | | | Merge-request: !2479
| * | | | | | CMP0053: document that `$` is a valid literal variable characterBen Boeckel2018-10-156-0/+41
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was overlooked in the initial implementation of CMP0053. However, an additional policy to reject it again is not worth it. Instead, add tests and document the behavior. Fixes: #17883
* | | | | | Merge branch 'backport-revert-install-code-script-genex' into release-3.13Brad King2018-10-116-19/+0
|\ \ \ \ \ \ | | |_|_|/ / | |/| | | | | | | | | | Merge-request: !2474
| * | | | | install: Revert CODE,SCRIPT support for generator expressionsBrad King2018-10-116-19/+0
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert commit v3.13.0-rc1~441^2 (install: Teach CODE,SCRIPT modes to evaluate generator expressions, 2018-05-29). Unfortunately it has been found to break existing code in a real project, e.g. install(CODE [[ message("$<FOOBAR>") ]]) Address this regression by reverting support for the 3.13 release series. Support can be restored later with a policy for compatibility. Issue: #15785 Fixes: #18435
* | | | | Merge branch 'project-always-set-desc-url' into release-3.13Brad King2018-10-082-0/+27
|\ \ \ \ \ | |_|/ / / |/| | / / | | |/ / | |/| | Merge-request: !2457
| * | | project(): Add tests verifying variables set by second project() callCraig Scott2018-10-072-0/+27
| |/ /
* | | FindPkgConfig: support also > and < operators for version comparesRolf Eike Beer2018-10-042-0/+84
|/ / | | | | | | Fixes: #18416
* | 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