summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* CMake 3.29.3v3.29.3Brad King2024-05-071-1/+1
|
* Merge topic 'cxxmodules-build-export-different-directory' into release-3.29Brad King2024-05-031-2/+4
|\ | | | | | | | | | | | | | | | | 1fba6d3233 Merge branch 'cxxmodules-build-export-different-directory-3.28' into cxxmodules-build-export-different-directory ec35251510 cmDyndepCollation: look at all `export(TARGETS)` calls Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9485
| * Merge branch 'cxxmodules-build-export-different-directory-3.28' into ↵Ben Boeckel2024-05-021-2/+4
| |\ | | | | | | | | | | | | | | | | | | cxxmodules-build-export-different-directory * cxxmodules-build-export-different-directory-3.28: cmDyndepCollation: look at all `export(TARGETS)` calls
| | * cmDyndepCollation: look at all `export(TARGETS)` callsBen Boeckel2024-05-021-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, only `export()` calls in the same directory were noticed. Also add a test that exports in a different directory than the target itself resides in. Fixes: #25813
* | | Merge topic 'vs-scan-module-deps-settings' into release-3.29Brad King2024-05-024-14/+52
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | 3022f0363f VS: set ScanSourceForModuleDependencies at vcxproj level dff511ad28 cmGeneratorTarget: add a target-level query for "needs dyndep" Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: AaronRuizMoraUK <aaronruizmora@gmail.com> Merge-request: !9471
| * | VS: set ScanSourceForModuleDependencies at vcxproj levelBen Boeckel2024-05-012-5/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the target-wide `CXX_SCAN_FOR_MODULES`/`CMP0155` selection with the `.vcxproj`-wide `ScanSourceForModuleDependencies` setting. Set the per-source equivalent only when needed for a per-source `CXX_SCAN_FOR_MODULES` property. This approach enables Intellisense for interfaces imported from modules. It is also more consistent with what a user might expect when investigating the state of module scanning from the VS property panels. Fixes: #25806 Fixes: #25947
| * | cmGeneratorTarget: add a target-level query for "needs dyndep"Ben Boeckel2024-05-012-9/+31
| | | | | | | | | | | | | | | This can still be overridden per-source, but it indicates the state of the target's default behavior.
| * | Merge topic 'export-name-safe-name' into release-3.28Brad King2024-05-015-6/+23
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 142a85f9c1 cxxmodules: use filesystem-safe export names in filenames 4452d41488 cmGeneratorTarget: add method to get a filesystem-safe export name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9474
* | \ \ Merge topic 'export-name-safe-name' into release-3.29Brad King2024-05-015-6/+23
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | 142a85f9c1 cxxmodules: use filesystem-safe export names in filenames 4452d41488 cmGeneratorTarget: add method to get a filesystem-safe export name Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9474
| * | | cxxmodules: use filesystem-safe export names in filenamesBen Boeckel2024-04-303-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | Also add tests. Fixes: #25828
| * | | cmGeneratorTarget: add method to get a filesystem-safe export nameBen Boeckel2024-04-292-0/+11
| | |/ | |/| | | | | | | | | | | | | | | | | | | C++ module support puts the export name in a filename. Export names with `:` in them are not valid filenames on Windows. Add a method to escape names as necessary. See: #25828
* | | Merge topic 'exported-modules-with-headers' into release-3.29Brad King2024-04-301-0/+4
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | | | | 2041f7c9bf cmGeneratorTarget: add the original target as a COMPILE_ONLY link 051c2110c8 Tests/CXXModules: test exporting modules which include headers Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9469
| * | cmGeneratorTarget: add the original target as a COMPILE_ONLY linkBen Boeckel2024-04-271-0/+4
| |/ | | | | | | Fixes: #25909
| * CMake 3.28.4v3.28.4Brad King2024-03-201-1/+1
| |
* | CMake 3.29.2v3.29.2Brad King2024-04-111-1/+1
| |
* | Fix regression on reconfigure with unnormalized -DCMAKE_<LANG>_COMPILER=Brad King2024-04-101-8/+6
| | | | | | | | | | | | | | | | | | | | Since commit 3f2a5971c0 (Modules: CMAKE_*_COMPILER convert path to cmake path, 2023-12-02, v3.29.0-rc1~292^2) we normalize the path to the compiler. Update our logic that checks whether the compiler has changed to account for path normalization. Fixes: #25883 Issue: #25456
* | Swift: Ninja: Remove module dependency for executablesEvan Wilde2024-04-091-14/+27
| | | | | | | | | | | | | | We shouldn't include the swiftmodule in the ninja dependency graph unless that target emits a swiftmodule. Fixes: #25869
* | CMake 3.29.1v3.29.1Brad King2024-04-031-1/+1
| |
* | cmSystemTools: Fix compilation on DragonFly BSDCharlotte Koch2024-04-021-1/+2
| | | | | | | | Define `_XOPEN_SOURCE >= 700` to make `gettimeofday()` et al. available.
* | ctest: Restore enforcement of RESOURCE_LOCK test propertyBrad King2024-04-012-4/+12
| | | | | | | | | | | | | | | | | | Refactoring in commit 5ff0b4ed57 (cmCTestMultiProcessHandler: Consolidate test readiness checks, 2023-10-20, v3.29.0-rc1~378^2~4) accidentally broke `RESOURCE_LOCK`. Fix it and replace the previous test with one that would have caught this. Fixes: #25843
* | try_compile(): Include languages with _ or - when passing linker varsCraig Scott2024-03-221-1/+1
| | | | | | | | Fixes: #25809
* | CMake 3.29.0v3.29.0Brad King2024-03-211-1/+1
| |
* | Merge topic 'ninja-fortran-include' into release-3.29Brad King2024-03-202-9/+26
|\ \ | |/ | | | | | | | | | | | | 36dc8d6d50 Ninja: Fix Fortran module deps in files INCLUDEd by preprocessed sources Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9351
| * Ninja: Fix Fortran module deps in files INCLUDEd by preprocessed sourcesBrad King2024-03-192-9/+27
| | | | | | | | | | | | | | If a preprocessed source also uses the Fortran `INCLUDE` directive, search the original source file's directory. Fixes: #25792
* | CMake 3.29.0-rc4v3.29.0-rc4Brad King2024-03-141-1/+1
| |
* | Merge topic 'Apple-TBD-in-subdirecory' into release-3.29Brad King2024-03-141-0/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | bcc26ce465 Apple Text Based Stubs: should be usage in subdirectories Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9341
| * | Apple Text Based Stubs: should be usage in subdirectoriesMarc Chevrier2024-03-141-0/+3
| | | | | | | | | | | | Fixes: #25765
* | | Merge topic 'autogen-makefile-depfile' into release-3.29Brad King2024-03-141-32/+10
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | d2d1763f88 cmQtAutoGenInitializer: De-duplicate autogen/timestamp target depend logic 6193d15556 Autogen: Restore target-ordering dependencies in Makefiles with DEPFILE Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9340
| * | cmQtAutoGenInitializer: De-duplicate autogen/timestamp target depend logicBrad King2024-03-141-24/+10
| | | | | | | | | | | | | | | | | | We place the same target ordering dependencies on either the `_autogen_timestamp_deps` target or the `_autogen` target. Refactor the logic to avoid duplicating that code.
| * | Autogen: Restore target-ordering dependencies in Makefiles with DEPFILEBrad King2024-03-141-15/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit aebfbcaa46 (AutoGen: Use depfiles for the XXX_autogen ninja targets, 2020-01-14, v3.17.0-rc1~58^2) the `_autogen_timestamp_deps` target was given target ordering dependencies through its custom command rather than direct target dependencies as on the `_autogen` target. Then commit 895fa3433f (cmQtAutoGenInitializer: support IMPLIB-only imported targets, 2021-09-23, v3.22.0-rc1~80^2) converted some target-level dependencies into file-level dependencies on the custom command. This only works with a monolithic build graph like Ninja. Since commit ebc9e448b3 (Autogen: Add depfile support for Makefiles, 2023-09-07, v3.28.0-rc1~101^2~1) we use the `_autogen_timestamp_deps` target in Makefile generators too. This exposed the missing target ordering dependency. Fixes: #25766
* | | Merge topic 'LINKER_TYPE-mold-support' into release-3.29Brad King2024-03-142-9/+34
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | 801ae06952 LINKER_TYPE: Support MOLD only on GCC versions that support it 939ac5287e LINKER_TYPE: fix spelling error in message 922883782b LINKER_TYPE: Document that linker tool should be in the PATH Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9334
| * | LINKER_TYPE: Support MOLD only on GCC versions that support itMarc Chevrier2024-03-132-9/+34
| | | | | | | | | | | | Fixes: #25748
| * | LINKER_TYPE: fix spelling error in messageMarc Chevrier2024-03-122-2/+2
| | |
* | | EXPORT_PACKAGE_DEPENDENCIES: Add experimental feature gateBrad King2024-03-124-5/+30
| | | | | | | | | | | | | | | | | | | | | Some design concerns have been raised after trying the 3.29 release candidates. Avoid committing to a stable public interface for now. Issue: #25767
* | | install(EXPORT): Drop REQUIRED from exported find_dependency callsBrad King2024-03-111-1/+1
|/ / | | | | | | | | | | | | | | | | | | In commit c6e6861e63 (install(EXPORT): Export find_dependency() calls, 2023-11-07, v3.29.0-rc1~439^2~1) we made the calls `REQUIRED`. However, a dependency is only required if the dependent package is required. `find_dependency` already forwards the `REQUIRED` mark, and also already marks the dependent package as not found if the dependency is missing. Fixes: #25756
* | Merge topic 'ctest-j-default' into release-3.29Brad King2024-03-118-69/+156
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5de1e21659 ctest: Allow passing -j without value to choose a contextual default bbcbcff7d9 cmCTestMultiProcessHandler: Modernize member initialization 7457b474a1 Tests: Remove unnecessary parallel suppression from CTestCoverageCollectGCOV ae69801d96 Tests: Convert CTestTestSkipReturnCode to RunCMake.ctest_test case 30dda49416 Tests: Convert CTestTestSerialOrder to RunCMake.ctest_test case Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: scivision <michael@scivision.dev> Merge-request: !9315
| * | ctest: Allow passing -j without value to choose a contextual defaultBrad King2024-03-108-36/+134
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Under job server integration, added by commit 80fe56c481 (ctest: Add support for running under a make job server on POSIX systems, 2023-11-15, v3.29.0-rc1~324^2), use a very high default so that parallelism is effectively limited only by available job server tokens. Otherwise, choose a default limit based on the number of processors. Also allow passing `0` to specify unbounded parallelism. Fixes: #25739
| * | cmCTestMultiProcessHandler: Modernize member initializationBrad King2024-03-103-34/+23
| | |
* | | Merge topic 'ctest-tests-from-file' into release-3.29Brad King2024-03-112-27/+26
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 170ec48601 Help: Improve ctest tests-from-file documentation wording and wrapping 1a4837641e ctest: Remove unnecessary and ambiguous tests-from-file comment syntax d52c66bfb3 ctest: Honor tests-from-file options with empty input 8673264e25 Tests: Make ctest tests-from-file expected output more precise Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9322
| * | | ctest: Remove unnecessary and ambiguous tests-from-file comment syntaxBrad King2024-03-081-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test names can contain `#`. Since we ignore lines that do not match any test names anyway, "commenting" can still work without explicit syntax. Also drop whitespace trimming for similar reasons. Fixes: #25741
| * | | ctest: Honor tests-from-file options with empty inputBrad King2024-03-082-22/+24
| | | | | | | | | | | | | | | | If the `--tests-from-file` input file is empty, no tests should run.
* | | | GET_RUNTIME_DEPENDENCIES: Allow more whitespace before objdump's "DLL Name:"Adrian Pop2024-03-101-1/+1
|/ / / | | | | | | | | | LLVM's `llvm-objdump` has spaces before "DLL Name:" instead of tabs.
* | | ctest: Exit with failure when tests-from-file input is missingBrad King2024-03-062-10/+14
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the options added by * commit 022f20f663 (ctest: add command line option to run the tests listed in a given file, 2023-11-29, v3.29.0-rc1~66^2~2) * commit dbacc1d5a8 (ctest: add command line option to exclude tests listed in a given file, 2023-11-30, v3.29.0-rc1~66^2~1) * commit 701029726f (ctest_test: add options INCLUDE_FROM_FILE and EXCLUDE_FROM_FILE, 2023-12-03, v3.29.0-rc1~66^2) are given a missing file, fail instead of ignoring it. Fixes: #25740
* | CMake 3.29.0-rc3v3.29.0-rc3Brad King2024-03-041-1/+1
| |
* | Merge topic 'cxxmodules-invalid-iterator' into release-3.29Brad King2024-03-042-3/+1
|\ \ | |/ | | | | | | | | | | | | 844f054fe0 cmImportedCxxModuleInfo: remove unused pointer field Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9303
| * cmImportedCxxModuleInfo: remove unused pointer fieldBen Boeckel2024-03-012-3/+1
| | | | | | | | | | Also avoids usage of an invalid `importit` when module names are not available.
* | Merge branch 'backport-3.28-cxxmodules-no-export-basedirs-as-include-paths'Brad King2024-02-281-1/+4
|\ \ | |/
| * cmExportFileGenerator: only export include paths for HEADERS file setsBen Boeckel2024-02-281-1/+4
| |
* | Merge topic 'cxxmodules-fileset-cache-fullpath' into release-3.29Brad King2024-02-271-1/+2
|\ \ | |/ | | | | | | | | | | | | 0a18f9baad cmGeneratorTarget: collapse paths before querying the fileset cache Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9286
| * cmGeneratorTarget: collapse paths before querying the fileset cacheBen Boeckel2024-02-241-1/+2
| | | | | | | | | | | | Observed with paths with `../` when outside of the source tree. I attempted to reproduce a test suite case for it but could not make it happen.