summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Tests: Fix FindPatch test with VS generatorsBrad King2023-02-131-0/+1
| | | | Tell CTest the project used to name the `.sln` file.
* Merge topic 'deprecate-pre-3.5'Brad King2023-02-13328-532/+455
|\ | | | | | | | | | | | | | | | | | | | | | | | | 3a4791548d Deprecate compatibility with CMake versions older than 3.5 1edf138506 Tests/RunCMake: Update cmake_minimum_required versions 65a1c379ee Tests/CTestTest*: Update cmake_minimum_required versions 589e6fdcd1 Tests: Update cmake_minimum_required versions in a few tests 3092b14ed0 Tests: Teach RunCMake to optionally ignore policy version deprecation warnings Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: scivision <michael@scivision.dev> Merge-request: !8192
| * Deprecate compatibility with CMake versions older than 3.5Brad King2023-02-1150-66/+198
| | | | | | | | | | | | | | | | Issue a deprecation warning on calls to `cmake_minimum_required` or `cmake_policy` that set policies based on versions older than 3.5. Note that the effective policy version includes `...<max>` treatment. Update the check from commit 5845c218d7 (Deprecate compatibility with CMake versions older than 2.8.12, 2020-06-12, v3.19.0-rc1~629^2).
| * Tests/RunCMake: Update cmake_minimum_required versionsBrad King2023-02-11240-367/+214
| | | | | | | | | | | | | | | | For policy-specific tests, use the version before the policy was introduced. Otherwise, use 3.5 where possible. Also, remove `cmake_minimum_required()` and `project()` calls from individual cases where they are handled by `CMakeLists.txt`.
| * Tests/CTestTest*: Update cmake_minimum_required versionsBrad King2023-02-1024-24/+24
| |
| * Tests: Update cmake_minimum_required versions in a few testsBrad King2023-02-103-3/+3
| |
| * Tests: Teach RunCMake to optionally ignore policy version deprecation warningsBrad King2023-02-0914-75/+19
| | | | | | | | | | Avoid requiring tests for deprecated behavior to match the warnings explicitly.
* | Merge topic 'test-https-connections'Brad King2023-02-116-1/+28
|\ \ | |/ |/| | | | | | | | | | | | | 3350c17451 ci: Enable file(DOWNLOAD) TLS_VERIFY test case in CI jobs 17b47da3d8 Tests: Add test for file(DOWNLOAD) with TLS_VERIFY Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8194
| * Tests: Add test for file(DOWNLOAD) with TLS_VERIFYscivision2023-02-106-1/+28
| | | | | | | | | | | | | | | | | | | | | | Occasionally curl updates introduce errors in https verification. Add an explicit test for this capability, activated by an undocumented option that we can use in CI to specify a URL to test. Co-authored-by: Brad King <brad.king@kitware.com> Fixes: #24405 Issue: #24147 Issue: #24398
| * Merge topic 'stop_parsing_after_first_script_arg' into release-3.26Brad King2023-02-092-0/+11
| |\ | | | | | | | | | | | | | | | | | | f9e8a067c2 cmake: Stop parsing after `--` when detecting script mode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8186
* | \ Merge topic 'vs9-deprecate'Brad King2023-02-096-4/+18
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | d7f440c5db Deprecate Visual Studio 9 2008 generator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8181
| * | | Deprecate Visual Studio 9 2008 generatorBrad King2023-02-086-4/+18
| | | | | | | | | | | | | | | | | | | | Update documentation to mark the generator deprecated. Add a warning at the end of generation plus an option to turn off the warning.
* | | | Merge topic 'stop_parsing_after_first_script_arg'Brad King2023-02-092-0/+11
|\ \ \ \ | |/ / / |/| | / | | |/ | |/| | | | | | | f9e8a067c2 cmake: Stop parsing after `--` when detecting script mode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8186
| * | cmake: Stop parsing after `--` when detecting script modeRobert Maynard2023-02-082-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | The fix in commit 08aa516880 (cmake: Stop parsing after `--` when detecting script mode, 2022-12-06, v3.26.0-rc1~216^2) only corrected the case where `-P -- -P <arg>` occurred and not `-P -- -P -<other>`. Fixes: #24220
* | | Merge topic 'ctest-remove-cmake-intdir'Brad King2023-02-081-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | db95afce19 Tests/OutDir: Remove stale cache entry on multi-config generators c6134ca6c7 CTest: Remove usages of CMAKE_INTDIR Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8176
| * | | Tests/OutDir: Remove stale cache entry on multi-config generatorsKyle Edwards2023-02-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This cache entry is left over from the incorrect behavior prior to the previous commit and was causing the test to fail with the new, correct behavior. Unset the cache variable when a multi-config generator is in use.
* | | | Tests: Remove references to dropped KWSys SharedForward componentBrad King2023-02-072-17/+0
| | | |
* | | | Merge topic 'avoid-cmake-cfg-intdir'Brad King2023-02-072-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2b17b6da6a cmGlobalGenerator: Avoid referencing CMAKE_CFG_INTDIR f34876561f Tests: Remove or silence instances of ${CMAKE_CFG_INTDIR} 78cf427157 RULE_LAUNCH_*: Add support for generator expressions cabad8a37f ExternalProject: Always use $<CONFIG> for source files 62e8884d3f CTestTargets: Use $<CONFIG> instead of ${CMAKE_CFG_INTDIR} Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8169
| * | | | Tests: Remove or silence instances of ${CMAKE_CFG_INTDIR}Kyle Edwards2023-02-032-2/+2
| | | | |
* | | | | Merge topic 'cxxmodules-proper-mangling'Brad King2023-02-076-0/+12
|\ \ \ \ \ | |_|/ / / |/| | | / | | |_|/ | |/| | | | | | | | | | | | | | 3f1e005067 Tests/RunCMake/CXXModules: declare `forwarding` as `extern "C++"` Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8159
| * | | Tests/RunCMake/CXXModules: declare `forwarding` as `extern "C++"`Ben Boeckel2023-02-036-0/+12
| | |/ | |/| | | | | | | It is not part of any module, so it needs to be declared as such.
* | | Tests: Drop redundant string(ASCII) case from COnly testBrad King2023-02-031-2/+0
| | | | | | | | | | | | | | | | | | Drop the case added by commit 8714396927 (Test conversion from ascii to string, 2003-01-01, v2.4.0~4658). Its results were not checked, and the command is tested elsewhere anyway.
* | | Tests: Move LINK_FLAGS case from COnly to LinkFlags testBrad King2023-02-032-4/+5
| |/ |/| | | | | | | | | Move the case added by commit 246e964180 (BUG: fix for bug 5455, handle nodefaultlib with more than one lib, 2007-12-06, v2.6.0~751) to a more sensible test.
* | Merge topic 'lang-linker-launcher-genex'Brad King2023-02-0329-26/+27
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 33e27f6ca6 <LANG>_LINKER_LAUNCHER: Allow generator expressions 84ada0b0c9 <LANG>_COMPILER_LAUNCHER: Expand subset of genexes that can be evaluated Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8152
| * | <LANG>_LINKER_LAUNCHER: Allow generator expressionsKyle Edwards2023-02-029-9/+10
| | |
| * | <LANG>_COMPILER_LAUNCHER: Expand subset of genexes that can be evaluatedKyle Edwards2023-02-0220-17/+17
| |/
* | Add deprecation warnings for policies CMP0114 and belowBrad King2023-02-0119-1/+193
|/ | | | | | | The OLD behaviors of all policies are deprecated, but only by documentation. Add an explicit deprecation diagnostic for policies introduced in CMake 3.19 and below to encourage projects to port away from setting policies to OLD.
* try_compile: Record propagated CMake variables in configure logBrad King2023-02-014-0/+47
| | | | | | | These provide more detailed information about how the test project was configured. Issue: #23200
* Merge topic 'msvc-showIncludes'Brad King2023-01-3034-18/+146
|\ | | | | | | | | | | | | | | | | | | | | 8f82e755f3 Ninja: Fix detection of MSVC showIncludes prefix in Italian d6e7e4d4a1 Tests: Extend RunCMake.Ninja ShowIncludes cases to cover more languages 9596305c0b Tests: Generalize RunCMake.Ninja ShowIncludes test infrastructure c6dd4fa21d Tests: Extend RunCMake.Ninja ShowIncludes case with sample path a9d97492fd Ninja: Record showIncludes detection in configure log Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8129
| * Ninja: Fix detection of MSVC showIncludes prefix in ItalianBrad King2023-01-285-0/+15
| | | | | | | | | | | | The prefix does not have two colons. Update our regex. Fixes: #24357
| * Tests: Extend RunCMake.Ninja ShowIncludes cases to cover more languagesBrad King2023-01-2821-0/+93
| | | | | | | | Add cases for English, French, German, and Japanese.
| * Tests: Generalize RunCMake.Ninja ShowIncludes test infrastructureBrad King2023-01-2810-15/+35
| | | | | | | | Prepare to add support for more languages.
| * Tests: Extend RunCMake.Ninja ShowIncludes case with sample pathBrad King2023-01-275-6/+6
| | | | | | | | Verify that the prefix is separated from the path.
* | Merge topic 'cxxmodules-export-iface-names-test'Brad King2023-01-3017-14/+251
|\ \ | | | | | | | | | | | | | | | | | | | | | f755296f9d Tests/RunCMake/CXXModules: add tests which don't export C++ module properties Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8126
| * | Tests/RunCMake/CXXModules: add tests which don't export C++ module propertiesBen Boeckel2023-01-2717-14/+251
| | | | | | | | | | | | | | | | | | | | | The existing `export-interfaces-{build,install}` tests were actually doing this, but make those test exporting the interfaces and copy the existing tests to tests which explicitly test the "no properties" condition.
* | | FindImageMagick: Define targets for specific componentshstejas2023-01-275-0/+42
| |/ |/| | | | | | | | | | | - With this change we can use e.g. ImageMagick::Magick++ directly in targt_link_libraries. - This change also adds CFLAGS which was missing before. - Also adds example on how to use the targets.
* | VS: Honor compile options for ASM_NASMBrad King2023-01-262-1/+7
| | | | | | | | | | | | | | | | The Ninja and Makefile generators honor `target_compile_options` and friends for ASM_NASM `.asm` sources. Teach the VS generator to honor them too for consistency. Issue: #24289
* | VS: Honor compile options for ASM_MARMASMBrad King2023-01-262-1/+4
| | | | | | | | | | | | | | | | The Ninja and Makefile generators honor `target_compile_options` and friends for ASM_MARMMASM `.asm` sources. Teach the VS generator to honor them too for consistency. Issue: #24289
* | Xcode: Inherit Swift flags and compilation conditionsRoss Kilgariff2023-01-254-2/+32
| | | | | | | | | | | | | | | | Extend the change from commit dfaf55fbfd (Xcode: add extra '$(inherited)' entries using InheritBuildSettingAttribute, 2021-05-03, v3.21.0-rc1~182^2) to cover Swift flags and compilation conditions, allowing CocoaPods and CMake to interoperate when used in the same project.
* | Merge topic 'test-try_compile-matching'Brad King2023-01-253-20/+20
|\ \ | | | | | | | | | | | | | | | | | | | | | 7db4df2e8c Tests: Match configure log try_compile directories more precisely Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8114
| * | Tests: Match configure log try_compile directories more preciselyBrad King2023-01-243-20/+20
| |/ | | | | | | | | Do not accept a double-quote in the `try_compile` work directory to ensure we do not match more than the field value.
* | Merge topic 'isdir_empty'Brad King2023-01-253-0/+7
|\ \ | | | | | | | | | | | | | | | | | | a1dc38a567 Help: Document behavior of if(EXISTS,IS_DIRECTORY) for empty paths Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8109
| * | Help: Document behavior of if(EXISTS,IS_DIRECTORY) for empty pathsscivision2023-01-243-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Document and add explicit tests for empty string input: `if(EXISTS "")` and `if(IS_DIRECTORY "")` are always false. This avoids need for users to do extra checks due to CMake non-short-circuit logic as below: if("${p}") if(EXISTS "${p}")
* | | Merge topic 'swiftmodule-dependency-tracking'Brad King2023-01-255-0/+49
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | 1730d208b5 Add incremental Swift static lib build test bf3a8ef6d5 Ninja: Swift: Add dependency edge to swiftmodule file d0b469b7e0 Ninja: NFC: refactor swift module name computations Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8084
| * | Add incremental Swift static lib build testEvan Wilde2023-01-215-0/+49
| |/ | | | | | | | | | | | | | | | | Ensure that we're actually trying to rebuild libB when the public interface for libA changes. Without handling the swiftmodule dependency edge correctly, we would only get a linker error because libA didn't have the symbol that libB depended on. With the fix, we get a proper compiler error because ninja knows to rebuild the intermediate libB when the public interface of libA changes. This is more actionable.
* | Ninja: require Ninja 1.11 for C++ module supportBen Boeckel2023-01-232-2/+2
| | | | | | | | | | See: https://gitlab.kitware.com/cmake/cmake/-/issues/18355#note_1296721 See: https://github.com/ninja-build/ninja/pull/1937
* | Merge topic 'genex-CONFIG-validate-all'Brad King2023-01-233-1/+10
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 910ada1a88 Genex: $<CONFIG:> syntax of all entries checked 42e417ad12 GeneratorExpression Tests: Remove duplicate test entry Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8082
| * | Genex: $<CONFIG:> syntax of all entries checkedRobert Maynard2023-01-202-0/+10
| | | | | | | | | | | | Fixes #24327
| * | GeneratorExpression Tests: Remove duplicate test entryRobert Maynard2023-01-171-1/+0
| | |
* | | Merge topic 'try_compile-verbose'Brad King2023-01-234-1/+25
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | faa950a155 try_compile: Run native build tool with verbose output Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8095