summaryrefslogtreecommitdiffstats
path: root/Tests/RunCMake
Commit message (Collapse)AuthorAgeFilesLines
* Tests: Teach RunCMake.ParseImplicitLinkInfo to match output by regexBrad King2021-06-0330-35/+35
| | | | | As in commit 6bc6fc2a7f (Tests: Teach RunCMake.ParseImplicitIncludeInfo to match output by regex, 2019-04-08, v3.14.2~5^2~1).
* Merge topic 'find_item-NO_CACHE'Brad King2021-06-0312-0/+599
|\ | | | | | | | | | | | | | | 87b71eec62 find_*: Add support for option NO_CACHE Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !6181
| * find_*: Add support for option NO_CACHEMarc Chevrier2021-06-0212-0/+599
| | | | | | | | | | | | | | | | Request that find result is stored in a normal variable rather than a cache entry. Fixes: #20687 Issue: #20743
* | Merge topic 'ctest-coverage-no-dir'Brad King2021-06-032-0/+7
|\ \ | | | | | | | | | | | | | | | | | | 5d5610b04f CTest: Exit coverage handling early if no binary dir Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6144
| * | CTest: Exit coverage handling early if no binary dirJoseph Snyder2021-06-022-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | Check for the existence of the Binary directory string before checking for coverage files. If the directory is not specified, write an error message and do not do any checking. Fixes: #22102
* | | cmake_transform_depfile: Remove unused incomplete vstlog supportBrad King2021-06-021-4/+0
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit b2c14bc774 (cmake -E: Add cmake_transform_depfile internal command, 2020-10-02, v3.20.0-rc1~684^2~2) a partial implementation for converting MSBuild "TLog" format files was added. However, it is unused and incomplete: * Does not enforce absolute paths as TLog docs require. * Does not upper-case paths as TLog docs recommend. * The TLog format semantics are not equivalent to `out: in`. Remove the unused TLog transform infrastructure to avoid confusion. Issue: #20286
* | Merge topic 'command_target'Brad King2021-06-022-0/+27
|\ \ | | | | | | | | | | | | | | | | | | 1cb4f592a0 add_custom_command: Target-dependent generator expression support Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6176
| * | add_custom_command: Target-dependent generator expression supportRaul Tambre2021-05-312-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | OUTPUT variant with a TARGET given to allow resolving target-based generator expressions wouldn't work because OUTPUT is resolved before generator targets are created, i.e. FindGeneratorTargetToUse() returns nullptr. This is a known limitation, see #21364. Implements #21336.
* | | Merge topic 'get-runtime-dependencies-file-filter'Brad King2021-06-027-55/+157
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | a12639e658 file(GET_RUNTIME_DEPENDENCIES): Add POST_{IN,EX}CLUDE_FILES arguments Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6170
| * | file(GET_RUNTIME_DEPENDENCIES): Add POST_{IN,EX}CLUDE_FILES argumentsKyle Edwards2021-06-017-55/+157
| | |
* | | Merge topic 'install-imported-runtime-artifacts'Brad King2021-06-013-3/+71
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f7ba3a0589 FileAPI: Populate information for IMPORTED_RUNTIME_ARTIFACTS df7040a271 install(): Add IMPORTED_RUNTIME_ARTIFACTS mode 60e752ced8 Refactor: Move common methods into cmInstallGenerator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6162
| * | | FileAPI: Populate information for IMPORTED_RUNTIME_ARTIFACTSKyle Edwards2021-05-313-3/+71
| | |/ | |/|
* | | Merge topic 'linker-launcher'Brad King2021-05-2949-1/+123
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ae108418ae Launchers: Support setting linker launchers 92c8b83641 cmNinjaNormalTargetGenerator: Fix "Language" rule variable d93bfd95f8 Tests/CompilerLauncher: Fix file names and clarify comment Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6092
| * | | Launchers: Support setting linker launchersBobby D Reynolds2021-05-2843-0/+122
| | | | | | | | | | | | | | | | Fixes: #18316
| * | | Tests/CompilerLauncher: Fix file names and clarify commentBobby D Reynolds2021-05-286-1/+1
| |/ /
* | | Merge topic 'fix-compile-db-crash'Brad King2021-05-291-1/+2
|\ \ \ | |/ / |/| | | | | | | | | | | | | | 316cbbe8d5 Makefiles: Fix CMAKE_EXPORT_COMPILE_COMMANDS crash with custom compile rule Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6173
| * | Makefiles: Fix CMAKE_EXPORT_COMPILE_COMMANDS crash with custom compile ruleBrad King2021-05-281-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend the fix from commit 67e2130c96 (Makefiles: Fix CMAKE_EXPORT_COMPILE_COMMANDS crash with custom compile rule, 2020-11-20, v3.19.1~8^2) to work for the `<FLAGS>` placeholder too. Also fix the test case to actually enable `EXPORT_COMPILE_COMMANDS` on the test target. Fixes: #22261
* | | Merge topic 'WriteBasicConfigVersion_support_leading_zeroes'Brad King2021-05-285-2/+927
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | e609afeb5f WriteBasicConfigVersion supports version strings such as 21.07.04 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6167
| * | WriteBasicConfigVersion supports version strings such as 21.07.04Robert Maynard2021-05-275-2/+927
| | | | | | | | | | | | Fixes #22207
* | | Merge topic 'xcode-link-TARGET_OBJECTS'Brad King2021-05-272-11/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 2ae72ef74b Xcode: Enable multi-arch TARGET_OBJECTS genex in [INTERFACE_]LINK_LIBRARIES Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6161
| * | | Xcode: Enable multi-arch TARGET_OBJECTS genex in [INTERFACE_]LINK_LIBRARIESBrad King2021-05-262-11/+7
| |/ / | | | | | | | | | This was an oversight in the original implementation.
* | | Merge topic 'cpack-install-opts'Craig Scott2021-05-263-0/+15
|\ \ \ | |/ / |/| | | | | | | | | | | | | | 26e36111d3 CPack: Implement new variable CPACK_CUSTOM_INSTALL_VARIABLES Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6141
| * | CPack: Implement new variable CPACK_CUSTOM_INSTALL_VARIABLESNils Gladitz2021-05-233-0/+15
| | | | | | | | | | | | | | | | | | The new variable allows projects to define custom key=value pairs of variables to be set in CPack cmake_install.cmake script invocations. This allows install(SCRIPT|CODE) to be parameterized at runtime.
* | | Ninja: Always compile sources using absolute pathsBrad King2021-05-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Ninja generator traditionally referenced source files and include directories using paths relative to the build directory if they could be expressed without a `../` sequence that leaves the build and source directories. For example, when using a `build/` directory inside the source tree, sources would be compiled as `-c ../src.c` and include directories would be referenced as `-I ../include`. This approach matches the traditional Ninja convention of using relative paths whenever possible, but has undesirable side effects such as: * Compiler diagnostic messages may not use absolute paths, making it harder for IDEs/editors to find the referenced sources or headers. * Debug symbols may not use absolute paths, making it harder for debuggers to find the referenced sources or headers. * Different results depending on the path to the build tree relative to the source tree. * Inconsistent with the Makefile generators, which use absolute paths. Switch to always using absolute paths to reference source files and include directories on compiler command lines. While alternative solutions for diagnostic messages and debug symbols may exist with specific tooling, this is the simplest and most consistent approach. Note that a previous attempt to do this in commit 955c2a630a (Ninja: Use full path for all source files, 2016-08-05, v3.7.0-rc1~275^2) was reverted by commit 666ad1df2d (Revert "Ninja: Use full path for all source files", 2017-02-24, v3.8.0-rc2~9^2) due to problems hooking up depfile dependencies on generated files. This time, the changes in commit 2725ecff38 (Ninja: Handle depfiles with absolute paths to generated files, 2021-05-19) should avoid those problems. Fixes: #13894, #17450
* | | Merge topic 'set-cache-keep-normal-variable'Brad King2021-05-257-0/+81
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | d96eb55282 set(CACHE): do not remove normal variable Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !6146
| * | | set(CACHE): do not remove normal variableMarc Chevrier2021-05-207-0/+81
| | | | | | | | | | | | | | | | Fixes: #22038
* | | | Merge branch 'master' into cmp0082-exclude-from-allKyle Edwards2021-05-24406-321/+4487
|\ \ \ \ | |_|_|/ |/| | |
| * | | Framework: Fix $<TARGET_SONAME_FILE:...> for imported frameworksKyle Edwards2021-05-203-0/+23
| | |/ | |/|
| * | Merge topic 'cmake-ninja-workdir'Brad King2021-05-204-0/+50
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2725ecff38 Ninja: Handle depfiles with absolute paths to generated files bc40cd7a4e Tests: Add case covering a unity build with a generated source ae927f936d cmGlobalNinjaGenerator: Improve allocation pattern in WriteBuild 68e5f92cad cmGlobalNinjaGenerator: Factor out custom command output collection c5195193d3 cmGlobalNinjaGenerator: Reduce string copies in WriteCustomCommandBuild 8bac527b0c cmGlobalNinjaGenerator: Re-order logic in WriteCustomCommandBuild ddc030f5ca cmGlobalNinjaGenerator: Record implicit outputs as known too ceb82752ef cmLocalNinjaGenerator: Use variable for main custom command output path Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6143
| | * | Ninja: Handle depfiles with absolute paths to generated filesBrad King2021-05-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ninja treats every (normalized) path as its own node. It does not recognize `/abs/path/to/file` in a depfile as matching `path/to/file` even when `build.ninja` and the working directory are in `/abs/`. See Ninja Issue 1251. In cases where we pass absolute paths to the compiler, it will write a depfile containing absolute paths. If those files are generated in the build tree by custom commands, `build.ninja` references them by relative path in build statement outputs, so Ninja does not hook up the dependency and rebuild the project correctly. Add infrastructure to work around this problem by adding implicit outputs to custom command build statements that reference the main outputs by absolute path. Use a `${cmake_ninja_workdir}` placeholder to avoid repeating the base path. For example: build out.txt | ${cmake_ninja_workdir}out.txt: CUSTOM_COMMAND ... Ninja will create two nodes for the output file, one with a relative path and one with an absolute path. A depfile may then mention either form of the path and Ninja will hook up the dependency. Unfortunately Ninja will also stat the file twice. Issue: #13894 Fixes: #21865
| | * | Tests: Add case covering a unity build with a generated sourceBrad King2021-05-194-0/+50
| | |/ | | | | | | | | | | | | | | | Exclude the case on generators where it does not yet work. Issue: #21865
| * | Merge topic 'install-script-all-components'Brad King2021-05-2010-0/+72
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | 99ff75455e install: Implement new install(CODE|SCRIPT) option ALL_COMPONENTS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6140
| | * | install: Implement new install(CODE|SCRIPT) option ALL_COMPONENTSNils Gladitz2021-05-1910-0/+72
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In a per-component installation the generated installation scripts are invoked once for each component. Per default custom installation script code added by install(CODE|SCRIPT) only runs for one specific component in this context. The new ALL_COMPONENTS option allows custom script code to be run once for each component being installed.
| * | cmCommandLineArgument: OneOrMore mode supports `=` separatorRobert Maynard2021-05-191-2/+1
| | | | | | | | | | | | Fixes #22187
| * | cmCommandLineArgument: Understands which commands require partial matchingRobert Maynard2021-05-191-1/+1
| |/ | | | | | | | | Allows us to provide better error messages when commands such as `--target` are passed invalid input.
| * Merge topic 'VS-include_external_msproject-CSharp-targets-references'Brad King2021-05-143-0/+47
| |\ | | | | | | | | | | | | | | | | | | | | | c7aa3bdefc Tests/include_external_msproject: Check C# project reference 65b58b0316 VS Generator: Properly reference included external C# projects Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6123
| | * Tests/include_external_msproject: Check C# project referenceDario Passet2021-05-133-0/+47
| | |
| * | Merge topic 'cmake_build_and_install_command_error_when_given_bad_arguments'Brad King2021-05-1413-0/+30
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f78b167a23 cmCommandLineArgument: Provide more information syntax error messages 5aa0dec6b0 cmake: `--build` and `--install` error out when encountering bad flags 928cdb17c5 cmCommandLineArgument: Correctly record parsing failures Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6119
| * \ \ Merge topic 'find_item-consistent-behavior-cache-variables'Brad King2021-05-1323-0/+397
| |\ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | 08db1341a6 find_*: ensure consistent behavior for cache variables f5fa6d53b0 class cmake: Store working directory at cmake launch b1729200c3 find_*: refactor cache variable handling Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6110
| | * | find_*: ensure consistent behavior for cache variablesMarc Chevrier2021-05-1223-0/+397
| | | | | | | | | | | | | | | | Fixes: #22121
| * | | Merge topic 'xcode-inherited-params'Brad King2021-05-123-0/+58
| |\ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | dfaf55fbfd Xcode: add extra '$(inherited)' entries using InheritBuildSettingAttribute. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6077
| | * | Xcode: add extra '$(inherited)' entries using InheritBuildSettingAttribute.Danny Parker2021-05-053-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These have been added to: GCC_PREPROCESSOR_DEFINITIONS OTHER_CFLAGS OTHER_LDFLAGS This is to allow Cocoapods to work correctly as it uses xcconfig files to alter build settings in Xcode, and requires these build settings to inherit from their parent, not overwrite.
| * | | Merge topic 'codespell'Brad King2021-05-107-8/+8
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fd02f10103 Xcode: Fix typos and spelling in error message 5950e54325 Source: Fix typos and spelling in comments 7072d83772 Help: Fix typos and spelling in documentation 03b1140ddc CONTRIBUTING: Fix typos and spelling 87f8843d8b FindMatlab: Fix spelling in warning and documentation 15cc39ed7f Modules: Fix typos and spelling in documentation f015c36c5a Modules: Fix typos and spelling in comments of generated code dad5b9d845 Modules: Fix typos and spelling in comments ... Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !6099
| | * | | Xcode: Fix typos and spelling in error messageJosef Angstenberger2021-05-071-1/+1
| | | | |
| | * | | Tests: Fix typos and spelling in test stringsJosef Angstenberger2021-05-072-2/+2
| | | | |
| | * | | Tests: Fix typos and spelling in test error messageJosef Angstenberger2021-05-071-2/+2
| | | | |
| | * | | Tests: Fix typos and spelling in test nameJosef Angstenberger2021-05-071-1/+1
| | | | |
| | * | | Tests: Fix typos and spelling in commentsJosef Angstenberger2021-05-072-2/+2
| | | | |
| * | | | Merge topic 'install-symlink-to-directory'Brad King2021-05-105-0/+22
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d71a7cc19d install(FILES): Allow installation of symlinks to directory Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6098
| | * | | | install(FILES): Allow installation of symlinks to directoryJens Carl2021-05-065-0/+22
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | Allow symbolic links to a directory to be installed via install(FILES ...). Fixes: #22159
generated by cgit v0.12 at 2025-01-31 18:33:50 (GMT)