summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge topic 'autogen-generators-refactor'Brad King2017-09-2246-274/+572
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 51fd7b71 Autogen: Tests: Add a change-not test to the mocRerun test c8f92db7 Autogen: Tests: Disable an AUTOMOC_DEPENDS_FILTER test for Ninja 3c77515e Autogen: Tests: Refactor the QtAutogenRebuild rccDepends test 04a0daee Autogen: Tests: Move each rerun test script to a NAME.cmake file e5c6610a Autogen: Tests: Extend CMAKE_AUTOMOC_DEPEND_FILTERS rebuild test 93265652 Autogen: Tests: Add <SUBDIR>/ui_view.h AUTOUIC includes to sameName test 4eb7d817 Autogen: Tests: Add <SUBDIR>/item.moc includes to sameName test 3f223743 Autogen: Read relative paths from rcc output ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1244
| * | | | | Autogen: Tests: Add a change-not test to the mocRerun testSebastian Holtermann2017-09-212-17/+36
| | | | | |
| * | | | | Autogen: Tests: Disable an AUTOMOC_DEPENDS_FILTER test for NinjaSebastian Holtermann2017-09-211-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ninja does not recognize changes in included files that are generated by AUTOMOC on the first run. This is an open issue: https://gitlab.kitware.com/cmake/cmake/issues/16776
| * | | | | Autogen: Tests: Refactor the QtAutogenRebuild rccDepends testSebastian Holtermann2017-09-2116-103/+151
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactors the QtAutogenRebuild rccDepends test script to be more readable.
| * | | | | Autogen: Tests: Move each rerun test script to a NAME.cmake fileSebastian Holtermann2017-09-214-217/+224
| | | | | |
| * | | | | Autogen: Tests: Extend CMAKE_AUTOMOC_DEPEND_FILTERS rebuild testSebastian Holtermann2017-09-2112-31/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The CMAKE_AUTOMOC_DEPEND_FILTERS must also detect filtered file changes from files that are registered to AUTOMOC by a `#include "moc_<NAME>.cpp"` statement.
| * | | | | Autogen: Tests: Add <SUBDIR>/ui_view.h AUTOUIC includes to sameName testSebastian Holtermann2017-09-2113-1/+118
| | | | | |
| * | | | | Autogen: Tests: Add <SUBDIR>/item.moc includes to sameName testSebastian Holtermann2017-09-216-9/+68
| | | | | |
* | | | | | Merge topic 'cpack-rpm-deb-version'Brad King2017-09-2216-13/+49
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 74b7457f CPack RPM and DEB: improved package version handling Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1296
| * | | | | | CPack RPM and DEB: improved package version handlingDomen Vrankar2017-09-2116-13/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds support for epoch version to CPackRPM and CPackDeb packagers. Also adds better version checking to CPackDeb that complies with Debian rules. Fixes: #17057
* | | | | | | test: Added additional unit test to variable_watchJustin Berger2017-09-212-0/+18
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | This tests adding a variable_watch inside the callback to an existing callback
* | | | | | Merge topic 'cpack-different-checksum-file-per-generator'Brad King2017-09-211-3/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b06870e5 CPack: use a distinct checksum file for each generator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1291
| * | | | | | CPack: use a distinct checksum file for each generatorDomen Vrankar2017-09-201-3/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Different CPack generators could produce checksum files with the same name which were overwritten by each other since only package name without extensions was used for checksum file name generation. This patch adds package extension to checksum files to prevent collisions. Fixes: #16840
* | | | | | Merge topic 'cpack-check-install-script'Brad King2017-09-213-0/+15
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 09166339 CPack: Add missing check for CPACK_INSTALL_SCRIPT variable Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1090
| * | | | | | CPack: Add missing check for CPACK_INSTALL_SCRIPT variableAlex Turbov2017-09-203-0/+15
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add a test case that uses CPACK_INSTALL_SCRIPT. Co-Author: Domen Vrankar <domen.vrankar@gmail.com> Fixes: #15005
* | | | | | Merge topic 'test_for_missing_sources'Brad King2017-09-2159-0/+209
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 55ed21e7 Tests: Add tests for missing sources in add_executable and add_library. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1269
| * | | | | Tests: Add tests for missing sources in add_executable and add_library.Deniz Bahadir2017-09-2059-0/+209
| | | | | |
* | | | | | Enable clang-tidy modernize-loop-convert lintBrad King2017-09-191-6/+3
| |/ / / / |/| | | | | | | | | | | | | | | | | | | Fix remaining diagnostics by this lint and remove it from our list of disabled lints.
* | | | | Merge topic 'gtest-fix-windows-linking'Brad King2017-09-191-0/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e2cca9f8 FindGTest: Avoid macro name collision 4636c64b FindGTest: Improve test to catch link error 35061791 FindGTest: Fix shared linking on Windows 9fd9e448 FindGTest: Avoid using find_dependency in a find module Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1267
| * | | | | FindGTest: Improve test to catch link errorMatthew Woehlke2017-09-181-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a reference to one of Google Test's command-line flags to the FindGTest test. This will ensure that we are using the correct compile definitions on Windows, as the test will otherwise fail to link. (IOW, this tests the changes made by the previous commit.)
* | | | | | Merge topic 'findmpi-overhaul'Brad King2017-09-181-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bf1e606f FindMPI: Add support for FreeBSD and SLES 2f673da6 FindMPI: Support components for various languages 9a58e69c FindMPI: Modernization from ground up Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !893
| * | | | | | FindMPI: Support components for various languagesChristian Pfeiffer2017-09-151-0/+5
| |/ / / / /
* | | | | | Tests: Fix CMake syntax warning in VSGNUFortran testBrad King2017-09-151-1/+1
| | | | | |
* | | | | | Tests: Fix some references to 'cmake' executableBrad King2017-09-151-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use `${CMAKE_CMAKE_COMMAND}` rather than `cmake` or `${CMAKE_COMMAND}` in order to ensure we always refer to the `cmake` command being tested.
* | | | | | Tests: Drop tests that have not run for over 10 yearsBrad King2017-09-1522-209/+0
| | | | | |
* | | | | | Merge topic 'refactor-iwyu-code'Brad King2017-09-1517-8/+30
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3bbe95f5 Clean up iwyu code to not be one big if statement. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1247
| * | | | | | Clean up iwyu code to not be one big if statement.Bill Hoffman2017-09-1317-8/+30
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes the internal -E__run_iwyu to be -E__run_co_compile. This is used for co-compile commands. These are tools that want to mirror the compiler. For each compiler invocation the tool will be invoked first. This started as a way to implement include what you use (iwyu), but has expanded to include cpplint, cppcheck and others. Likely there will be more in the future as well. This commit implements each one in its own function and provides a way to add additional ones in the future with less work.
* | | | | | Merge topic 'fphsa-fix-version-exact'Brad King2017-09-151-4/+4
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5dc215e0 FPHSA: fix typo in comment a8f70b32 FPHSA: replace left-over VERSION by _FOUND_VERSION. 549366cb FPHSA: Add (failing) unit test when checking EXACT on full version number Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1259
| * | | | | FPHSA: fix typo in commentYves Frederix2017-09-141-2/+2
| | | | | |
| * | | | | FPHSA: Add (failing) unit test when checking EXACT on full version numberYves Frederix2017-09-141-2/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | This test detects a regression introduced by commit aec27c152e (FPHSA: use more localized _FOUND_VERSION instead of VERSION, 2017-09-05).
* | | | | Tests: Update RunCMake/README.rst to clarify meaning of -result.txtBrad King2017-09-141-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The content of `-result.txt` files is a regex, but the documentation didn't make this clear. Suggested-by: Deniz Bahadir <dbahadir@benocs.com>
* | | | | get_filename_component: Revise PROGRAM/PROGRAM_ARGS split semanticsBrad King2017-09-131-0/+11
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The KWSys `SystemTools::SplitProgramFromArgs` implementation goes into an infinite loop when the value is just " " (a space). Since the "program path with unquoted spaces plus command-line arguments" operation it is trying to provide is poorly defined (string parsing should not depend on filesystem content), just stop using it. Instead consider the main two use cases the old approach tried to handle: * The value is the name or absolute path of a program with no quoting or escaping, but also no command-line arguments. In this case we can use the value as given with no parsing, and assume no arguments. * The value is a command-line string containing the program name/path plus arguments. In this case we now assume that the command line is properly quoted or escaped. Fixes: #17262
* | | | Merge topic 'vs_improve_custom_command'Brad King2017-09-113-0/+89
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9ed24280 VS: only add custom command line if it is not empty 34c4108b add HasOnlyEmptyCommandLines() method to cmCustomCommandGenerator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1050
| * | | | VS: only add custom command line if it is not emptyMichael Stürmer2017-09-043-0/+89
| | | | |
* | | | | Merge topic 'timestamp'Brad King2017-09-112-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 014ad298 Timestamp: support %A and %B Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1228
| * | | | | Timestamp: support %A and %BBernhard M. Wiedemann2017-09-072-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are defined both by [1] and [2] to give full names of a weekday and month. [1] http://pubs.opengroup.org/onlinepubs/009695399/functions/strftime.html [2] https://msdn.microsoft.com/de-de/library/fe06s4ak.aspx
* | | | | | CTest: Weight reported test times by PROCESSORS in summariesBill Hoffman2017-09-088-15/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes the times reported by labels and subprojects to be weighted by the PROCESSORS property. It is reported with `sec*proc` instead of just `sec`.
* | | | | | CTest: Split out labels and subproject labels in summary outputBill Hoffman2017-09-085-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit splits out the reporting of labels and labels used for sub projects. If a label is a sub project label it will not be included in the label summary. To implement this the commit creates PrintLabelOrSubprojectSummary which is able to do the work of both PrintLabelSummary and PrintSubprojectSummary avoiding code duplication.
* | | | | | Merge topic 'fphsa-version-cleanup'Brad King2017-09-086-0/+23
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 91935d9e FPHSA: lower-case for cmake_parse_arguments + move config-mode related code aec27c15 FPHSA: use more localized _FOUND_VERSION instead of VERSION. 0019bfeb FPHSA: Add (failing) unit test in case a cache variable VERSION is present 81844334 FPHSA: add (failing) unit test for found version being equal to 0. Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Rolf Eike Beer <eike@sf-mail.de> Merge-request: !1224
| * | | | | | FPHSA: Add (failing) unit test in case a cache variable VERSION is presentYves Frederix2017-09-054-1/+15
| | | | | | |
| * | | | | | FPHSA: add (failing) unit test for found version being equal to 0.Yves Frederix2017-09-053-0/+9
| | | | | | |
* | | | | | | Merge topic 'findfreetype-importedtargets'Brad King2017-09-084-0/+64
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b1d1c56c Help: Add notes for topic 'FindFreetype-importedtargets' 457c2021 FindFreetype: Add imported target, docs, and test Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1219
| * | | | | | | FindFreetype: Add imported target, docs, and testBen Morgan2017-09-074-0/+64
| | |/ / / / / | |/| | | | |
* | | | | | | Autogen: Tests: Add AUTOGEN_TARGET_DEPENDS test to mocDepends testsSebastian Holtermann2017-09-0710-54/+128
|/ / / / / /
* | | | | | Merge topic 'findexpat-importedtargets'Brad King2017-09-074-0/+51
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3a4f82e9 Help: Add notes for topic 'FindEXPAT-importedtargets' bfe51369 FindEXPAT: Add unit test of target and variables 78f166f8 FindEXPAT: Add imported target and documentation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1207
| * | | | | | FindEXPAT: Add unit test of target and variablesBen Morgan2017-09-014-0/+51
| | | | | | |
* | | | | | | Merge topic 'vs-csharp-ref-no-asm'Brad King2017-09-063-0/+25
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 94e70e53 Merge branch 'backport-vs-csharp-ref-no-asm' into vs-csharp-ref-no-asm 7e57e6ae VS: Do not reference output assemblies if not possible for CSharp target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1203
| * \ \ \ \ \ \ Merge branch 'backport-vs-csharp-ref-no-asm' into vs-csharp-ref-no-asmBrad King2017-09-053-0/+25
| |\ \ \ \ \ \ \ | | |_|_|/ / / / | |/| | | | | / | | | |_|_|_|/ | | |/| | | |
| | * | | | | VS: Do not reference output assemblies if not possible for CSharp targetMichael Stürmer2017-09-053-0/+25
| | | |_|/ / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.9.0-rc4~4^2 (Vs: allow CSharp targets to be linked to CXX targets, 2017-06-20) CSharp targets get `ProjectReference` entries to their dependencies. This causes VS to also reference the dependency's output assembly by default, which is incorrect for non-managed targets. Fix this by setting `ReferenceOutputAssembly` to `false` for targets that can't provide output assemblies. Unmanaged C++ targets (shared libs & executables) can still be referenced and a warning will be shown in the IDE but the build will not break anymore. Fixes: #17172
| | * | | | Merge branch 'fix-find-package-mode' into release-3.9Brad King2017-08-291-0/+2
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | Merge-request: !1208