summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'try_compile-more-consistent-errors'Brad King2022-09-202-2/+2
|\ | | | | | | | | | | | | | | 30a234d275 try_compile: Improve error message consistency Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7696
| * try_compile: Improve error message consistencyMatthew Woehlke2022-09-162-2/+2
| | | | | | | | | | Tweak some error messages from try_compile (and try_run) to be more consistent with each other.
* | Merge topic 'add-try_compile-test-documentation'Brad King2022-09-192-0/+18
|\ \ | | | | | | | | | | | | | | | | | | | | | e933fccc7e Tests: document try_{compile,run} old/new signature testing Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7681
| * | Tests: document try_{compile,run} old/new signature testingMatthew Woehlke2022-09-152-0/+18
| | | | | | | | | | | | | | | Add a brief documentation blurb to the try_compile and try_run test sets on writing tests for both "old" and "new" signatures.
* | | Merge topic 'improve-try_compile-arg-error'Brad King2022-09-193-3/+3
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | f976800577 try_compile: More useful error if insufficient arguments Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !7682
| * | try_compile: More useful error if insufficient argumentsMatthew Woehlke2022-09-153-3/+3
| |/ | | | | | | | | Tweak try_compile and try_run to give a more informative error (rather than "unknown error") when given an insufficient number of arguments.
* | Merge topic 'try_compile-new-project-signature'Brad King2022-09-1620-1/+61
|\ \ | | | | | | | | | | | | | | | | | | | | | 56ae40cc59 try_compile: Add PROJECT keyword-dispatched signature Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !7677
| * | try_compile: Add PROJECT keyword-dispatched signatureMatthew Woehlke2022-09-1420-1/+61
| | | | | | | | | | | | | | | | | | | | | Introduce a new signature for the project flavor of try_compile (and try_run) which removes the `bindir` argument and adds a required PROJECT tag. This is similar to the SOURCES flavor added by commit aa9220d3 (try_compile: Add keyword-dispatched signature, 2022-09-02).
* | | Merge topic 'xcode-lib-dirs'Brad King2022-09-163-0/+25
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | d4cc39842e Xcode: Do not append per-config suffixes to library search paths Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7672
| * | | Xcode: Do not append per-config suffixes to library search pathsMikko Sivulainen2022-09-153-0/+25
| |/ / | | | | | | | | | | | | | | | | | | Add policy `CMP0142` to remove the automatic addition of the `$(CONFIGURATION)$(EFFECTIVE_PLATFORM_NAME)` suffix in a compatible way. Fixes: #21757
* | | Merge topic 'fphsa-no-version-var'Brad King2022-09-163-0/+5
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 07f0a78874 FPHSA: Fix regression when VERSION_VAR is missing Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7680
| * | | FPHSA: Fix regression when VERSION_VAR is missingBrad King2022-09-153-0/+5
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | If a package is found but FPHSA is called by the find module without a `VERSION_VAR`, and the `find_package` call specifies a version, we have previously accepted the package as found. This was accidentally regressed by commit 8f50f135ae (FPHSA: Improve error message when VERSION_VAR is empty or has been unset(), 2022-08-01). Restore it and add a test case.
* | | cmake --list-presets: List package as a valid optionKyle Edwards2022-09-153-0/+5
|/ /
* | Merge topic 'fix-try_compile-tests'Brad King2022-09-153-3/+9
|\ \ | | | | | | | | | | | | | | | | | | | | | 4ecbf1f938 Tests: Fix recently added tests Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7678
| * | Tests: Fix recently added testsMatthew Woehlke2022-09-143-3/+9
| |/ | | | | | | | | | | Fix the tests added by commit 0c141b03 (try_compile: Record output location instead of reverse computing it, 2022-08-31), which were added as old-and-new-signature tests but only used the old signature.
* | Merge topic 'link-framework-with-multi-config-postfix'Brad King2022-09-151-0/+5
|\ \ | |/ |/| | | | | | | | | fc06450ff4 Apple: Fix regression when linking a framework with postfix Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7675
| * Apple: Fix regression when linking a framework with postfixMarc Chevrier2022-09-141-0/+5
| | | | | | | | | | | | | | Fix a regression caused by commit 40178f3c90 (cmGlobalGenerator: Add helper to split framework path, 2022-02-10, v3.24.0-rc1~661^2~1). Fixes: #23961
* | Merge topic 'MsvcDebugInformationFormatAbstraction'Brad King2022-09-1425-0/+275
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | a858466aac MSVC: Add test for debug information format 0e96a20478 MSVC: Add abstraction for debug information format d4c8111da4 Clang/Windows: Clarify name of internal runtime library flags variables Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7606
| * | MSVC: Add test for debug information formatBrad King2022-09-1411-0/+147
| | | | | | | | | | | | | | | Verify that the format's flags are used to compile, both in the main project and in `try_compile` test projects.
| * | MSVC: Add abstraction for debug information formatGlen Chung2022-09-1414-0/+128
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace our hard-coded default for `/Zi` with a first-class abstraction to select the debug information format an enumeration of logical names. We've long hesitated to do this because the idea of "debug information format" touches on related concepts on several platforms. Avoid that scope creep by simply defining an abstraction that applies only when targeting the MSVC ABI on Windows. Removing the old default flag requires a policy because existing projects may rely on string processing to edit them and choose a runtime library under the old behavior. Add policy CMP0141 to provide compatibility. Fixes: #10189
* | | Merge topic 'modules-use-new-try_compile-signature'Brad King2022-09-144-2/+15
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 2edf0fc6d7 Modules: Use new keyword-dispatched try_compile signature Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7656
| * | | Modules: Use new keyword-dispatched try_compile signatureMatthew Woehlke2022-09-144-2/+15
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Modify most of the modules that ship with CMake to use the new try_compile / try_run signatures added by commit aa9220d3a0 (try_compile: Add keyword-dispatched signature, 2022-09-02). This improves debugging by each invocation using its own directory so that the results of multiple invocations can be retained. This does not cover any invocations which provide an entire project, as that flavor of try_compile has not yet been updated.
* | | CMakePresets.json: Fix error message for unsupported package presetsKyle Edwards2022-09-131-1/+1
| | |
* | | Merge topic 'fix-commandline-test-with-symlinks'Brad King2022-09-131-1/+6
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7b62e40a64 Tests: Fix RunCMake.CommandLine test in unusual environments Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7660
| * | | Tests: Fix RunCMake.CommandLine test in unusual environmentsMatthew Woehlke2022-09-121-1/+6
| |/ / | | | | | | | | | | | | | | | Modify `Tests/RunCMake/CommandLine/P_working-dir.cmake` to not spuriously fail in environments when the build directory's nominal path is not canonicalized (i.e. `realpath(BINDIR) != BINDIR`).
* | | Merge topic 'try_compile_correctly_compute_exec_name'Brad King2022-09-129-18/+28
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0c141b0393 try_compile: Record output location instead of reverse computing it Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7626
| * | | try_compile: Record output location instead of reverse computing itRobert Maynard2022-09-079-18/+28
| |/ / | | | | | | | | | Fixes #23887, #23942, #20163, #18288
* | | Merge topic 'fetchcontent-redirect-version-exact'Brad King2022-09-121-0/+4
|\ \ \ | | |/ | |/| | | | | | | | | | | | | 48b380c961 FetchContent: Ignore EXACT for redirected find_package() calls Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7665
| * | FetchContent: Ignore EXACT for redirected find_package() callsCraig Scott2022-09-111-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When FetchContent_MakeAvailable() populates a dependency for which find_package() integration is enabled, all future calls to find_package() MUST succeed using the contents of the redirection directory. The generated config version file was not handling calls where the EXACT keyword was given, resulting in such calls rejecting the redirection directory's contents and continuing its search. It is not allowed to do that. Fix the generated file to also set PACKAGE_VERSION_EXACT to true so that calls with EXACT now accept it, as was originally intended. Fixes: #23950
| * | Merge topic 'truncation_invalid' into release-3.24Brad King2022-09-097-0/+10
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bd30d20bc3 Tests: Add cases covering bad ctest output truncation types dbf840392d ctest: print error for invalid CTEST_CUSTOM_TEST_OUTPUT_TRUNCATION Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7596
* | \ \ Merge topic 'fphsa-error-message-empty-version-var'Brad King2022-09-094-1/+15
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8f50f135ae FPHSA: Improve error message when VERSION_VAR is empty or has been unset() Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7527
| * | | | FPHSA: Improve error message when VERSION_VAR is empty or has been unset()Andrea Pappacoda2022-09-074-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, when `VERSION_VAR` is set but empty, FPHSA emitted a confusing error message, saying that the package was found but of an unsuitable version "". Fix the error message to state that the package was not found. Fixes: #23807
* | | | | Merge topic 'truncation_invalid'Brad King2022-09-097-0/+10
|\ \ \ \ \ | | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bd30d20bc3 Tests: Add cases covering bad ctest output truncation types dbf840392d ctest: print error for invalid CTEST_CUSTOM_TEST_OUTPUT_TRUNCATION Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7596
| * | | | Tests: Add cases covering bad ctest output truncation typesBrad King2022-09-077-0/+10
| | | | | | | | | | | | | | | | | | | | Issue: #23869
* | | | | Merge topic 'check-library-properties-fix-performances-regression'Brad King2022-09-092-5/+5
|\ \ \ \ \ | | |_|/ / | |/| | / | |_|_|/ |/| | | | | | | | | | | | | | | 985b4c82a6 Check link libraries properties: fix performances regression a47eef32a3 renames method FinalizeTargetCompileInfo() in FinalizeTargetConfiguration(). Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7651
| * | | Check link libraries properties: fix performances regressionMarc Chevrier2022-09-072-5/+5
| |/ / | | | | | | | | | Fixes: #23939
* | | Merge topic 'try_compile-unique-bindir'Brad King2022-09-0772-403/+596
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aa9220d3a0 try_compile: Add keyword-dispatched signature d1befe5515 cmSystemTools: Add MakeTempDirectory Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Acked-by: Matthew Woehlke <matthew.woehlke@kitware.com> Merge-request: !7579
| * | | try_compile: Add keyword-dispatched signatureMatthew Woehlke2022-09-0572-403/+596
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce a new signature for try_compile (and try_run) which removes the `bindir` argument and requires the SOURCES tag. This will eventually allow us to add other ways of providing sources, but also allows us to change the behavior without breaking compatibility. The old signature uses a special, but non-unique temporary location inside the specified `bindir`, which conventionally is just the project's build directory. The new signature unconditionally uses the a unique temporary directory which is unconditionally within the project's build directory (which is no longer separately specified). This ensures that successive runs do not overwrite previous runs, will simplify debugging, and should also, eventually, allow us to execute multiple trials in parallel.
* | | | Merge topic 'return-propagate-variables'Brad King2022-09-0714-1/+218
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 838a5fae23 return(): Propagate variables to result scope 8f0e1f2111 cmMakefile: add RaiseScope for list of variables Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Acked-by: Robert Maynard <robertjmaynard@gmail.com> Merge-request: !7634
| * | | | return(): Propagate variables to result scopeMarc Chevrier2022-09-0314-1/+218
| | | | | | | | | | | | | | | | | | | | Fixes: #23871
* | | | | Merge topic 'cpack-reuse-cmake-tools'Brad King2022-09-076-11/+11
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0c638ab597 CPack: re-use paths of tools discovered by CMakeFindBinUtils Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7635
| * | | | | CPack: re-use paths of tools discovered by CMakeFindBinUtilsMatteo Martelli2022-09-066-11/+11
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `CMakeFindBinUtils.cmake` sets `CMAKE_<TOOL>` variables that in cross-compilation environments point to the toolchain specific executables: e.g. `CMAKE_OBJCOPY` might point to `<path_to_toolchain>/arm-poky-linux-gnueabi-objcopy`. Pass this information to CPack.
* | | | | Merge topic 'test-find-boost'Brad King2022-09-071-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 047c98f4a0 FindPackageTest: Replace VTK with Boost Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7638
| * | | | | FindPackageTest: Replace VTK with BoostFeRD (Frank Dana)2022-09-031-1/+1
| |/ / / /
* | | | | Merge topic 'functiontests-haz-fails'Brad King2022-09-071-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4cae5b8f5c FunctionTest: Fix some test failure checks Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7640
| * | | | | FunctionTest: Fix some test failure checksFeRD (Frank Dana)2022-09-031-2/+2
| |/ / / / | | | | | | | | | | | | | | | Fixes: #23584
* | | | | Merge topic 'implicit-link-info-msvc'Brad King2022-09-063-3/+92
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 93f2cd5b7c CMakeParseImplicitLinkInfo: Better detection of msvc Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !7619
| * | | | | CMakeParseImplicitLinkInfo: Better detection of msvcRobert Maynard2022-09-023-3/+92
| | | | | | | | | | | | | | | | | | | | | | | | Use the compiler id information in addition to checking for `cl.exe`.
* | | | | | Merge topic 'truncation_test'Craig Scott2022-09-0611-11/+18
|\ \ \ \ \ \ | |_|/ / / / |/| | | / / | | |_|/ / | |/| | | | | | | | | | | | | | | | | | cb55d5e730 RunCMakeTest: fix Truncation test definition Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !7632
| * | | | RunCMakeTest: fix Truncation test definitionFrank Winklmeier2022-09-0511-11/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `ctest_test` and `CTestCommandLine` truncation tests had multiple problems: - escape expected result string to avoid regex matching - specify the truncation size - pass the truncation mode correctly into the test definition - use unique test names Issue: #23868
generated by cgit v0.12 at 2025-01-23 22:35:59 (GMT)