| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Fixes: #21028
|
|\
| |
| |
| |
| |
| |
| | |
5fc5f4d26e add_test: Revert "Allow special characters in test name"
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5067
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Revert commit f84af8e270 (add_test: Allow special characters in test
name, 2020-05-16, v3.18.0-rc1~142^2). Unfortunately the fix breaks
projects that were working around the limitation with manual escaping.
The fix can be re-introduced with a policy in a future version.
Also add a 3.18.1 release note explaining the change.
Fixes: #21017, #20965
Issue: #19391
|
|\ \
| |/
|/|
| |
| |
| |
| | |
c4109a1bc8 VS: Restore toleration of target-wide -TP flag with MSVC
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5051
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since commit 3b547e2e4b (VS: Simplify logic adding source file C/C++
language flag to MSVC, 2020-05-15, v3.18.0-rc1~139^2~1) we only add a
per-source language selection flag when the source file extension does
not match the compiler's default. This approach breaks when a project
adds a target-wide `-TP` flag.
Although such projects likely did not work with non-VS generators, we
did support them before in Visual Studio generators. Add a special case
to tolerate such flags again.
Fixes: #21005
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
1829220310 Merge branch 'backport-3.17-graphviz-restore-per-target'
f3a6b4a209 Tests: Cover Graphviz support for per-target dependency graph options
93549b9224 Graphviz: Restore support for per-target dependency graph options
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5039
|
| |\ \
| | |/
| |/| |
|
| | |
| | |
| | |
| | | |
Issue: #20928
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since commit 2026915f8f (Swift: Propagate Swift_MODULE_DIRECTORY as include
directory, 2020-02-03, v3.18.0-rc1~547^2) we internally call
`GetAllConfigCompileLanguages` on all directly linked targets without
checking if they are interface libraries that don't compile at all.
That violates an internal assumption and assertion.
Fixes: #20977
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
f6969b917d set_property: Deduplicate source file directory scopes
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5009
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
A user could specify the same directory scope to set_property()
multiple times, which in conjunction with APPEND would append the
property multiple times.
Make sure to deduplicate scopes across both DIRECTORY and
TARGET_DIRECTORY options, so that a property is only appended
once in such a scenario.
Fixes: #20941
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
d9af90504f Tests: Update file(G_R_D) test to make system library optional
c51400033c file: Update GET_RUNTIME_DEPENDENCIES for macOS 11 dylib cache
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Kyle Edwards <kyle.edwards@kitware.com>
Merge-request: !4998
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Since the system library may not exist on the filesystem on macOS
11, skip it if it doesn't exist.
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
7cf2f7d2af cmake: Store lowercase command names in profiling output
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5007
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
This ensures commands can be properly aggregated by tools
processing the profiling output.
Fixes: #20946
|
|/ /
| |
| |
| | |
Fixes: #20942
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Allow to specify both DIRECTORY and TARGET_DIRECTORY at the same time in
`set_source_files_properties()` and `set_property(SOURCE)` commands.
Add test cases and update the documentation.
Fixes: #20932
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Clarify the order in which --resource-spec-file, RESOURCE_SPEC_FILE,
and CTEST_RESOURCE_SPEC_FILE are searched, and add tests to
enforce this.
Fixes: #20914
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In commit dd8365b3f1 (Merge branch 'upstream-KWSys' into update-kwsys,
2020-04-06, v3.18.0-rc1~397^2) we imported KWSys commit `019afb6ea`
(SystemTools: Drop GetCurrentWorkingDirectory 'collapse' argument,
2020-04-03). That caused `GetCurrentWorkingDirectory` to no longer send
paths through the KWSys translation map and broke CMake's detection of
the absolute path to a build directory containing a symbolic link.
Add our own `cmSystemTools::GetCurrentWorkingDirectory` wrapper around
the KWSys method in order to restore that mapping.
Test-case-by: Ben Boeckel <ben.boeckel@kitware.com>
Issue: #16228
Fixes: #20900
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
bbcff21f71 file(ARCHIVE*): Collapse FILES and DIRECTORY options
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4955
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The two options were concatenated internally for both ARCHIVE_CREATE
and ARCHIVE_EXTRACT. The distinction between files and dirs was not
meaningful. Therefore, replace them with PATHS or PATTERNS
to more accurately describe the way the options are used.
Fixes: #20884
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
e5f5eeca2f file: Fix CONFIGURE output relative path
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4951
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In commit a6fee09484 (file: Add CONFIGURE subcommand, 2020-03-06,
v3.18.0-rc1~584^2) we accidentally treated relative path outputs
with respect to the current working directory. Treat them with
respect to the current binary directory instead.
Fixes: #20885
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
f2c903fb9a find_library: Check that library files are readable
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4945
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Refactoring in commit 6b85166920 (ENH: Refactor find_library search
logic, 2008-09-22, v2.8.0~1665) dropped a call to `FileExists` on the
path under the assumption that the presence of a file in a directory
listing means it exists. However, dropping that also dropped a check
that verifies the file is readable. Restore the `FileExists` call to
ensure that we only find readable libraries.
|
| | |
| | |
| | | |
Fixes: #20883
|
| | |
| | |
| | |
| | | |
The test cases only need C or C++, but not both. The CTest module
is also not needed because we are not running a dashboard script.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
6c6b227813 Tests: Drop redundant FindModulesExecuteAll test
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4930
|
| |/ /
| | |
| | |
| | |
| | | |
This test has been superseded by `CMakeOnly.AllFindModules`,
which does much more checking anyway.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
b284a3c764 CudaToolkit: Update for new/removed libraries in CUDA 11
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4917
|
| |/ / |
|
| | |
| | |
| | |
| | |
| | | |
Drop the portion of the case names that repeats the name of the
containing test.
|
|/ /
| |
| |
| | |
Fixes: #20836
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
9ae3382855 Tests: Shorten case names in RunCMake.file-GET_RUNTIME_DEPENDENCIES test
248d0570c9 Tests: Factor out RunCMake.file-GET_RUNTIME_DEPENDENCIES test
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4904
|
| | |
| | |
| | |
| | |
| | | |
Now that these cases are in a dedicated test we can drop the
`file-GET_RUNTIME_DEPENDENCIES-` prefix from their name.
|
| | |
| | |
| | |
| | | |
Move the `file(GET_RUNTIME_DEPENDENCIES)` cases out of `RunCMake.install`.
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
cb9fab49dd Tests: Fix CudaOnly.StaticRuntimePlusToolkit build rpath
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4901
|
| | | |
|
| | |
| | |
| | |
| | | |
Move the `GENEX_EVAL` cases and friends out of `RunCMake.GeneratorExpression`.
|
| | |
| | |
| | |
| | | |
Move the `TARGET_FILE` cases and friends out of `RunCMake.GeneratorExpression`.
|
| | |
| | |
| | |
| | | |
Move the `DEVICE_LINK` cases out of `RunCMake.GeneratorExpression`.
|
| | |
| | |
| | |
| | | |
Move the `HOST_LINK` cases out of `RunCMake.GeneratorExpression`.
|
| | |
| | |
| | |
| | | |
Move the `LINK_LANG_AND_ID` cases out of `RunCMake.GeneratorExpression`.
|
| | |
| | |
| | |
| | | |
Move the `LINK_LANGUAGE` cases out of `RunCMake.GeneratorExpression`.
|
| | |
| | |
| | |
| | | |
Move the `COMPILE_LANG_AND_ID` cases out of `RunCMake.GeneratorExpression`.
|
|/ /
| |
| |
| | |
Move the `COMPILE_LANGUAGE` cases out of `RunCMake.GeneratorExpression`.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
877a92e968 CUDA: Add support for disabling CUDA_ARCHITECTURES
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4886
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The ability to disable adding architectures completely for packaging purposes
and cases requiring passing the architectures flags explicitly has been
requested.
Support a false value for CUDA_ARCHITECTURES and CMAKE_CUDA_ARCHITECTURES
for this purpose.
Implements #20821.
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
cc02ced530 find_program: Revert "Find programs that are executable but not readable"
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !4871
|