| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
The change in commit 86ad7cc886 (project: Only define non-cache vars if
already defined, 2024-09-15, v3.30.4~2^2) was meant to only check for
non-cache variables when deciding whether to set non-cache project
variables for the current call. However, it erroneously checked for any
variable, including cache variables. This gives the intended result on
the first run, but on subsequent runs a cache variable will exist that
did not on the first run, leading to different behavior between the two
runs. Fix the logic to only check for a pre-existing non-cache
variable, as was originally intended.
Fixes: #26355
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In c1ece78d11 (project: non cache <project> prefix variables are
also created, 2024-08-27), we started explicitly setting the non-cache
variable for <projectName>_SOURCE_DIR, <projectName>_BINARY_DIR,
and <projectName>_IS_TOP_LEVEL in addition to setting them as
cache variables. This changed the behavior when a project name
was used more than once, and the second project call happens in
the same scope or a child scope of the first. Previously, the first
project call would set cache variables, and the second project call
would not overwrite those cache variables. With the change in
c1ece78d11, after the second project call the non-cache variables
would mask the cache variables and the project code would see
a different value to what it did before.
Setting the non-cache variable was added to handle the case where
a call to FetchContent_MakeAvailable() would set some non-cache
variables, and it just so happened those matched the same cache
variables that the project() command would set in the project being
fetched. The fetched project would then see a different set of
project-specific variables compared to when it was built standalone.
This commit here narrows the change from c1ece78d11 such that
the non-cache variable is only set by project() if there was already
a non-cache variable set. This still fixes the motivating problem
c1ece78d11 was intended to solve, but it avoids changing the variable
values seen by a project that re-uses the same project name in related scopes.
Issue: #26243, #25714
Fixes: #26281
|
| |
|
|
| |
Issue: #25714
|
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| | |
2449c04d8d Help: improve docs for the EXCLUDE_FROM_ALL directory property
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9497
|
| | |
| |
| |
| |
| |
| | |
This patch improves the wording of the documentation of the
EXCLUDE_FROM_ALL directory property, and makes the
add_subdirectory()- and fetchcontent-documentation refer to it.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7c516f7e28 file(): TOUCH, TOUCH_NOCREATE and MAKE_DIRECTORY accept empty lists
Acked-by: Kitware Robot <kwrobot@kitware.com>
Tested-by: buildbot <buildbot@kitware.com>
Merge-request: !9570
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Projects may be generating a list of files or directories to pass as arguments
to file(TOUCH), file(TOUCH_NOCREATE), or file(MAKE_DIRECTORY). Those
lists might end up being empty, so rather than requiring at least one item,
allow an empty list.
Fixes: #24897
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
4b55d56259 Help: Start find_package() docs with a new "Typical Usage" section
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9567
|
| | |/ /
| | |
| | |
| | | |
Fixes: #22680
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
4cb5bb014d try_compile: Propagating top level includes into whole-project calls
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Robert Maynard <robertjmaynard@gmail.com>
Merge-request: !9532
|
| | | |
| | |
| | |
| | | |
Fixes: #24151
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
67171a99be Help: Fix inaccurate use of "function" in macro docs
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Juan Ramos <juan.ramos@kitware.com>
Merge-request: !9534
|
| | |/ |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
8d2a503c1e add_custom_command: Allow OUTPUT filenames containing a hash '#' character
b38000d774 cmGlobalXCodeGenerator: Re-implement legacy makefile path escaping
d929089687 cmGlobalXCodeGenerator: Do not use legacy makefile escaping in shell commands
d61fc2c52e cmGlobalXCodeGenerator: Migrate legacy makefile path escaping to local helper
6010e007c7 cmState: Add method to check for the Borland Makefiles generator
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9514
|
| | |/
| |
| |
| |
| |
| |
| | |
Most generators now support this character after escaping and quoting
cleanups over time. Disallow it only on generators that do support it.
Fixes: #25604
|
| |/
|
|
|
|
| |
it wasn't obvious how to pass compiler options. Since the old
add_definitions() is used we can workaround this by passing
compiler options there.
|
| |
|
|
| |
Fixes: #25938
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
263f6b888c Help: Document TARGET_PROPERTY genex handling of transitive properties
f70eb84be7 Help: Clarify role of TARGET_NAME generator expression
36145e2680 Help: Organize target-dependent generator expressions into subsections
50eadd794e Help: Clarify cross-references to target-dependent generator expressions
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9460
|
| | | |
|
| |\ \
| |/
|/|
| |
| |
| |
| |
| |
| | |
d2d2ffe1c1 Help: file: document GET_RUNTIME_DEPENDENCIES separately
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Acked-by: Juan Ramos <juan.ramos@kitware.com>
Merge-request: !9456
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Moved documentation of file(GET_RUNTIME_DEPENDENCIES ...) from
the 'Reading' section to a separate section at the bottom of
the page. Because it is a very long text, and because this
signature is quite different from all the others in the
'Reading' section.
|
| |/
|
|
|
|
|
| |
Re-organize the section to explicitly describe target-specific
commands, build properties, and usage requirement properties.
Add builtin transitive properties that have been created since
the document was first written.
|
| |
|
|
| |
In particular, give an example of matching `]`.
|
| |
|
|
| |
Fixes: #25550
|
| |\
| |
| |
| |
| |
| |
| | |
e722f73e40 Help: Document MANUALLY_ADDED_DEPENDENCIES in add_dependencies
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9388
|
| | |
| |
| |
| |
| | |
This adds a note about the MANUALLY_ADDED_DEPENDENCIES target property
to the add_dependencies documentation page. It was introduced in v3.8.0.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
a61c9afdf8 add_library: Reject shared libraries on platforms that do not support them
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !9347
|
| | |/
| |
| |
| |
| |
| | |
Add policy CMP0164 to provide compatibility.
Fixes: #25759
|
| |/
|
|
|
| |
This was missed by commit 434fe8a34b (file(DOWNLOAD|UPLOAD): Add
CMAKE_TLS_VERSION environment variable, 2024-02-26).
|
| |\
| |
| |
| |
| |
| |
| | |
8cbd6f5754 Help: Fix location of install() note about superseded commands
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9382
|
| | |
| |
| |
| |
| |
| |
| | |
The note regarding commands superseded by install() somehow wound
up nested under the install(EXPORT) signature, when it has nothing
to do with install(EXPORT). Move it to the top level, immediately
before the Examples section.
|
| | |
| |
| |
| | |
Closes: #25796
|
| |\ \
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6ba2bbc0f3 Help: Restructure and improve LINKER_TYPE docs
c39bc6e412 Help: CMAKE_<LANG>_USING_LINKER_<TYPE> can be a list of flags
1c7ccefd83 Help: Add custom linker example for CMAKE_<LANG>_USING_LINKER_<TYPE>
aecea7b28b Help: Grammar, typos and wording improvements for linker variables
5b21897c6e Help: Remove stray formatting for try_compile() LINKER_LANGUAGE keyword
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9362
|
| | | |
|
| | |\
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
eceb368ccc Help: Fix typos in 3.29 docs
31fb693d06 Help: Fix typos and grammar in pre-3.29 docs
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9353
|
| |\ \ \
| | |/
| |/|
| | |
| | |
| | |
| | |
| | | |
eceb368ccc Help: Fix typos in 3.29 docs
31fb693d06 Help: Fix typos and grammar in pre-3.29 docs
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9353
|
| | | | |
|
| |\ \ \
| | |/
| |/|
| | |
| | |
| | |
| | | |
c980745b82 Help: Specify advisable range for exit code in cmake_language(EXIT)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9352
|
| | |/
| |
| |
| |
| | |
The limitations added here are documented in the bash shell's
reference manual.
|
| |\ \
| |/
| |
| |
| |
| |
| |
| | |
dcb894fb00 Help: Update project() code injection for new 3.29 include behavior
09cf39ab07 Help: Improve wording of CMAKE_PROJECT_..._INCLUDE... changes for 3.29
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9343
|
| | |
| |
| |
| |
| |
| | |
Specifically, the various CMAKE_PROJECT_...INCLUDE... variables can
now take a list of things, and those things can be module names
too, not just file names.
|
| |\ \
| |/
| |
| |
| |
| |
| |
| | |
3a739d4dcb EXPORT_PACKAGE_DEPENDENCIES: Add experimental feature gate
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !9336
|
| | |
| |
| |
| |
| |
| |
| | |
Some design concerns have been raised after trying the 3.29 release
candidates. Avoid committing to a stable public interface for now.
Issue: #25767
|
| |\ \
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\ \
| |/
| |
| |
| |
| |
| | |
b03356f954 Help/add_custom_command: fix verb grammo
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9297
|
| | | |
|
| | |\
| | |
| | |
| | |
| | |
| | |
| | | |
189d28d77d Help/file: mention the working directory for `ARCHIVE_EXTRACT`
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9291
|