summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'install-name-tool-bug'Brad King2020-01-282-5/+9
|\ | | | | | | | | | | | | 162e8aeb9e Tests: Disable test on problematic versions of install_name_tool Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4276
| * Tests: Disable test on problematic versions of install_name_toolKyle Edwards2020-01-272-5/+9
| | | | | | | | | | | | | | According to https://cmake.org/pipermail/cmake-developers/2015-September/026462.html, there is a bug in Xcode 7.0's install_name_tool that manifests when we try to test file(GET_RUNTIME_DEPENDENCIES). Add a cache option to disable this test.
* | Merge topic 'fix-ninja-multi-framework-header'Brad King2020-01-272-0/+29
|\ \ | | | | | | | | | | | | | | | | | | 63c9cd2088 Ninja Multi-Config: Fix bug with MacOS frameworks Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4279
| * | Ninja Multi-Config: Fix bug with MacOS frameworksKyle Edwards2020-01-242-0/+29
| | |
* | | Merge topic 'depend-on-target-real-name'Brad King2020-01-272-0/+12
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 08699cec68 add_custom_command: Fix dependency on macOS framework target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4273
| * | | add_custom_command: Fix dependency on macOS framework targetBrad King2020-01-242-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the `DEPENDS` option names a logical target name, convert to the "real" path of that target rather than any of the symlink-provided alternative file names. This makes the dependencies consistent with the outputs named by link rules producing the target artifacts. Fixes: #20264
* | | | Merge topic 'csharp-no-valued-macros'Brad King2020-01-271-4/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | effd4d0569 CSharp: Do not pass definitions with values Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4267
| * | | | CSharp: Do not pass definitions with valuesSumit Bhardwaj2020-01-241-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Microsoft C# tooling does not accept definitions with values. Filter them out. Fixes: #19817
* | | | | Merge topic 'stl-support'Brad King2020-01-271-0/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 73d1da4f86 Stl support: cm::append now supports any sequential container Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4251
| * | | | | Stl support: cm::append now supports any sequential containerMarc Chevrier2020-01-241-0/+1
| | | | | |
* | | | | | Merge topic 'FindGTest-upstream-lib-names'Brad King2020-01-271-0/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fbd3ea2047 FindGTest: Add GTest::{gtest,gtest_main} library names Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4268
| * | | | | | FindGTest: Add GTest::{gtest,gtest_main} library namesPaweł Bylica2020-01-241-0/+4
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces 2 new INTERFACE IMPORTED libraries: GTest::gtest and GTest::gtest_main. They link to GTest::GTest and GTest::Main targets respectively, therefore working as aliases. These new names map the names of the targets from upstream GTest's CMake package config. Fixes: #20255
* | | | | | Merge topic 'cpack-deb-fix-description'Brad King2020-01-274-7/+16
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | baec299ecd CPack: Fix regression in Deb description Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4271
| * | | | | CPack: Fix regression in Deb descriptionKyle Edwards2020-01-244-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: #20254
| * | | | | file(GET_RUNTIME_DEPENDENCIES): Tolerate empty list argumentsKyle Edwards2020-01-222-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also list all keywords that were missing values instead of only the first one.
* | | | | | Merge topic 'ninja-multi-cross-configs'Brad King2020-01-2415-0/+377
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | b7a2baf38c Ninja Multi-Config: Add variable to control configs used in cross-config build Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4269
| * | | | | Ninja Multi-Config: Add variable to control configs used in cross-config buildKyle Edwards2020-01-2315-0/+377
| | |/ / / | |/| | |
* | | | | Merge topic 'vs-dotnet-standard-core'Brad King2020-01-248-0/+134
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | ae1e1909a1 VS: Add support for .NET Standard and .NET Core Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4240
| * | | | VS: Add support for .NET Standard and .NET CoreJoerg Bornemann2020-01-248-0/+134
| | |_|/ | |/| | | | | | | | | | Fixes: #20105
* | | | Merge topic 'trace_json_timestamp'Brad King2020-01-241-4/+8
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | c829f0cfca trace: Add time and stack level to JSON output format Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Volo Zyko <volo.zyko@gmail.com> Merge-request: !4242
| * | | trace: Add time and stack level to JSON output formatFrank Winklmeier2020-01-221-4/+8
| |/ / | | | | | | | | | | | | | | | | | | | | | Add the timestamp and stack depth of the function call to the JSON trace output format. This information can be useful for cmake profiling and call stack inspection (see e.g. https://github.com/volo-zyko/cmake-profile-stats). Improve unit test to allow for varying set of keys to check in trace lines.
* | | Merge topic 'get-runtime-dependencies-keyword-args'Brad King2020-01-232-2/+20
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3573f62a16 Merge branch 'backport-get-runtime-dependencies-keyword-args' c998c8d560 file(GET_RUNTIME_DEPENDENCIES): Tolerate empty list arguments 4c8a917a87 file(GET_RUNTIME_DEPENDENCIES): Tolerate empty list arguments Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4262
| * | | file(GET_RUNTIME_DEPENDENCIES): Tolerate empty list argumentsKyle Edwards2020-01-222-2/+20
| | | | | | | | | | | | | | | | | | | | Also list all keywords that were missing values instead of only the first one.
* | | | Ninja Multi-Config: Add variable to control aliases in build.ninjaKyle Edwards2020-01-225-0/+140
| | | |
* | | | Refactor: Split Ninja files into impl-<Config>.ninja and build-<Config>.ninjaKyle Edwards2020-01-224-0/+142
|/ / /
* | | Merge topic 'mark_as_advanced-without-cache'Brad King2020-01-229-0/+87
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3ec82b713e cmMarkAsAdvancedCommand: ignore variables which don't exist in the cache 701a5c60e0 cmake: avoid marking local or unused variables as advanced af158149e7 FindOpenSSL: do not mark a local variable as advanced 74f659f1f2 FindCurses: only mark CURSES_EXTRA_LIBRARY when it is used 7e2ae4e96d FindOpenGL: only mark declared cache variables as advanced 7cc02a0c29 FindLua: only mark LUA_MATH_LIBRARY as advanced if required 85cd26b8a6 FindBoost: only mark Boost_DIR as advanced if defined 338c7916ba CTest: avoid marking undeclared cache variables as advanced ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4150
| * | | cmMarkAsAdvancedCommand: ignore variables which don't exist in the cacheBen Boeckel2020-01-209-0/+87
| | | | | | | | | | | | | | | | Fixes: #18331
* | | | Tests: Make CMake.FileDownload invalid host name case more robustBrad King2020-01-211-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Use a well-defined `.invalid` top-level domain instead of assuming there is no `.png` top-level domain. Extend the timeout of this one case to 30 seconds to give DNS lookups a chance to fail instead of timing out.
* | | | Merge topic 'multi-ninja-pch'Brad King2020-01-209-2/+43
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a55df20499 Multi-Ninja: Add precompile headers support Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4201
| * | | | Multi-Ninja: Add precompile headers supportCristian Adam2020-01-179-2/+43
| | | | | | | | | | | | | | | | | | | | Fixes: #19789
* | | | | Merge topic 'fphsa-name-mismatch'Brad King2020-01-208-0/+55
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | be4d1bdf9a FPHSA: acknowledge the name mismatches in CMake-owned modules ee4673c1ae FPHSA: detect package name mismatches Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Eric Noulard <eric.noulard@gmail.com> Merge-request: !4123
| * | | | | FPHSA: detect package name mismatchesBen Boeckel2020-01-148-0/+55
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | The `FPHSA_NAME_MISMATCHED` variable may be set if this is intentional (but should be cleared after the call to not affect other FPHSA calls). It may also be passed via the `NAME_MISMATCHED` option for new-signature FPHSA calls.
* | | | | Merge topic 'acc-fix-in-source-depends-path'Brad King2020-01-201-0/+17
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ba3a417dce Tests/CustomCommand: add a test for depending on a `./path` e23475dc73 Tests/CustomCommand: fix custom command line to actually make its output db4780d584 cmGeneratorTarget: search for relative paths to the binary directory ec479f101f cmLocalGenerator: collapse the path after construction fd0ba705ce add_custom_command: check if a relative path should be an in-source path fd84f510f8 cmLocalGenerator: simplify the current source dir query Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4195
| * | | | Tests/CustomCommand: add a test for depending on a `./path`Ben Boeckel2020-01-161-0/+8
| | | | |
| * | | | Tests/CustomCommand: fix custom command line to actually make its outputBen Boeckel2020-01-161-1/+1
| | | | |
| * | | | cmLocalGenerator: collapse the path after constructionBen Boeckel2020-01-161-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | The path may contain `..` or `.` components naïvely, so compute them out before letting the generator handle them.
| * | | | add_custom_command: check if a relative path should be an in-source pathBen Boeckel2020-01-161-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This still is broken for dependencies on generated paths where they get generated to the source directory rather than the build directory however, but there's no way to determine that is the case. Fixes: #20194
* | | | | Merge topic 'out-of-dir-link-list'Brad King2020-01-1711-11/+31
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f0e67da061 target_link_libraries: Fix out-of-dir linking of a list of targets acee629103 cmTargetLinkLibrariesCommand: Move HandleLibrary to helper struct ba675f1ecc Tests: Enable CMP0022 in ExportImport out-of-dir linking case Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4226
| * | | | | target_link_libraries: Fix out-of-dir linking of a list of targetsBrad King2020-01-1610-11/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In a case like target_link_libraries(targetInOtherDir PUBLIC "$<1:a;b>") then all entries in the list need to be looked up in the caller's scope. Previously our `::@(directory-id)` suffix would apply only to the last entry. Instead surround the entire entry by a pair `::@(directory-id);...;::@` so that the `::@` syntax can encode a directory lookup scope change evaluated as the list is processed. Fixes: #20204
| * | | | | Tests: Enable CMP0022 in ExportImport out-of-dir linking caseBrad King2020-01-141-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since out-of-dir linking is enabled by CMP0079, which is newer than CMP0022, it is likely that both will be set in practice when out-of-dir linking is used.
* | | | | | Merge topic 'cmake-ctest-arguments'Brad King2020-01-173-0/+19
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4153d8445b Add CMAKE_CTEST_ARGUMENTS variable to pass command-line arguments to ctest Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4168
| * | | | | | Add CMAKE_CTEST_ARGUMENTS variable to pass command-line arguments to ctestRobert Goulet2020-01-153-0/+19
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Fixes: #20172
* | | | | | Merge topic 'rename-cross-config-variable'Brad King2020-01-171-7/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 310b58e961 Ninja Multi-Config: Rename variable to be more consistent Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4234
| * | | | | | Ninja Multi-Config: Rename variable to be more consistentKyle Edwards2020-01-161-7/+7
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | Also make some tweaks to the documentation.
* | | | | | Merge topic 'cpack-custom-dmg-names'Craig Scott2020-01-177-1/+75
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6d6f4b9316 CPack/DragNDrop: Support CPACK_DMG_<component>_FILE_NAME b53230fbee Tests: Add DragNDrop as a testable CPack generator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4227
| * | | | | CPack/DragNDrop: Support CPACK_DMG_<component>_FILE_NAMEAndrew Fuller2020-01-163-1/+7
| | | | | |
| * | | | | Tests: Add DragNDrop as a testable CPack generatorAndrew Fuller2020-01-164-0/+68
| | | | | |
* | | | | | Merge topic 'cpack-nsis-test'Brad King2020-01-161-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b796749ba8 Tests: Fix CPackNSISGenerator test on non-x86 Windows platforms Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Kyle Edwards <kyle.edwards@kitware.com> Merge-request: !4230
| * | | | | | Tests: Fix CPackNSISGenerator test on non-x86 Windows platformsBrad King2020-01-151-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit ebfffc609e (CPack/NSIS: Add option for setting MUI_HEADERIMAGE_BITMAP, 2020-01-02) this test was added but only globbed for 32-bit windows packages. Fix the test to glob for packages of any architecture.
* | | | | | Merge topic 'file_download_filename'Brad King2020-01-163-16/+70
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c0da651c09 file(DOWNLOAD): Don't fail if given just a filename to write to b56d429324 Tests: Fix CMake.FileDownload test failures not reported 3bc73803b4 Tests: Fix CMake.FileDownload test in presence of proxy 1352585301 Tests: Extend CMake.FileDownload test internal timeouts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4159