summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* FindBoost: Improve messages when a Boost CMake package is foundBrad King2017-09-297-0/+19
| | | | | | Add a test for this case to verify the messages. This test will also be valuable to cover this code path in which we've had several regressions recently.
* Merge topic 'add-vs-shader-properties'Brad King2017-09-291-2/+4
|\ | | | | | | | | | | | | 49dab3eb VS: Add VS_SHADER_OUTPUT_HEADER_FILE and VS_SHADER_VARIABLE_NAME properties. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1319
| * VS: Add VS_SHADER_OUTPUT_HEADER_FILE and VS_SHADER_VARIABLE_NAME properties.Kevin M. Godby2017-09-271-2/+4
| | | | | | | | | | | | | | | | | | | | The VS_SHADER_OUTPUT_HEADER_FILE property is the name of the generated header file containing the object code of the shader. The VS_SHADER_VARIABLE_NAME property is the name of the variable containing the object code in the above header file. Signed-off-by: Kevin M. Godby <kevin@godby.org>
* | Autogen: Tests: Update AUTOMOC_MACRO_NAMES testSebastian Holtermann2017-09-2815-49/+139
| |
* | Merge topic 'GetPrerequisites-no-clear-on-missing'Brad King2017-09-274-0/+11
|\ \ | | | | | | | | | | | | | | | | | | 1a7b8c83 GetPrerequisites: Restore behavior on missing binary of not clearing list Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1312
| * | GetPrerequisites: Restore behavior on missing binary of not clearing listBrad King2017-09-264-0/+11
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to commit v3.4.0-rc1~264^2~1 (GetPrerequisites: Add error checks for execute_process() calls, 2015-07-29), `get_prerequisites` would simply warn on a missing binary and not update the result list at all. That commit accidentally made the case an error. This was fixed by commit v3.8.0-rc1~110^2 (GetPrerequisites: Do not fail on files we cannot find, 2017-01-10), but the fix also cleared the result list. Clearing the list is incorrect because it is supposed to be able to accumulate results over multiple calls. Remove the list clearing behavior to restore the original behavior on a missing binary. Fixes: #17306
* | Merge topic 'ninja-rc-depfile-quoting'Brad King2017-09-271-0/+7
|\ \ | | | | | | | | | | | | | | | | | | df965cb9 Ninja: Fix quoting of RC language depfile in cmcldeps call Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1309
| * | Ninja: Fix quoting of RC language depfile in cmcldeps callBrad King2017-09-251-0/+7
| |/ | | | | | | | | | | | | | | | | | | | | Since commit v3.0.0-rc1~448^2 (Ninja: use deps = gcc/msvc feature, 2013-10-18) the value of the `DEP_FILE` binding already includes the needed quoting to refer to the file. However, that commit forgot to update one of the `$DEP_FILE` references to not be quoted anymore. The offending code path currently only affects cmcldeps for RC. Remove the extra quoting now. Fixes: #17298
* | Merge topic 'FindOpenGL-glvnd'Brad King2017-09-272-1/+64
|\ \ | | | | | | | | | | | | | | | | | | e2e8a690 FindOpenGL: Add support for GLVND on Linux Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !782
| * | FindOpenGL: Add support for GLVND on LinuxTom Fogal2017-09-252-1/+64
| |/ | | | | | | | | | | | | | | | | | | | | | | Find GLVND components if available. Add `GLX` and `EGL` options for COMPONENTS that allow requesting these libraries explicitly. Introduce new import targets for these windowing-system-specific libraries. On a GLVND system, populate the legacy `OPENGL_LIBRARIES` variable and the `OpenGL::GL` target using the `OpenGL` and `GLX` components. On non-GLVND systems, continue to use the legacy `GL` library and simply do not provide the GLVND components. Application code can choose to adapt based on the availability of GLVND components as imported targets.
* | Merge topic 'revert-perf-source-lookup'Brad King2017-09-272-1/+17
|\ \ | |/ |/| | | | | | | | | | | a7005c98 Tests: Add case for legacy source file property behavior 1604716d Revert "Performance: Improve efficiency of source file lookup in cmMakefile" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1308
| * Tests: Add case for legacy source file property behaviorBrad King2017-09-252-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | The change in commit 3b95ab5693 (Performance: Improve efficiency of source file lookup in cmMakefile, 2017-08-17) broke some legacy behavior of source file properties in which the order sources are first resolved with extensions affects how setting properties without extensions works. It has been reverted for now, but the discovery was made after merging because the broken case was not covered by our test suite. Add a test case representing the legacy behavior. Issue: #15208
* | Merge topic 'test-fix-msvc-standard-default'Brad King2017-09-251-13/+31
|\ \ | |/ |/| | | | | | | | | a5dd1599 Tests: Fix RunCMake.try_compile CxxStandard case on MSVC Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1302
| * Tests: Fix RunCMake.try_compile CxxStandard case on MSVCBrad King2017-09-221-13/+31
| | | | | | | | | | | | | | | | In post-3.9 development we've taught CMake to understand C++ language standards for MSVC. The RunCMake.try_compile test needs to know whether to expect support or not. Previously we depended on the host CMake version to know this, but CMake 3.9 and below do not. Add special logic to the test to account for this.
* | 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
generated by cgit v0.12 at 2025-01-11 03:53:23 (GMT)