summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* ci: make rvm version comments consistentBrad King2022-05-242-3/+2
|
* clang-tidy: suppress `google-readability-casting` lintBen Boeckel2022-05-241-1/+0
| | | | It now complains about some things that are very pedantic.
* clang-tidy: suppress new `readability-identifier-length` lintBen Boeckel2022-05-241-0/+1
| | | | | CMake uses short names all over the place; 3 character minimums is excessive.
* cmFileCommand: remove an unnecessary castBen Boeckel2022-05-241-1/+1
| | | | `cmCryptoHash::New` already returns a `unique_ptr`.
* clang-tidy: fix `readability-static-definition-in-anonymous-namespace` lintsBen Boeckel2022-05-241-2/+2
|
* clang-tidy: fix `bugprone-exception-escape` lintsBen Boeckel2022-05-241-3/+9
|
* clang-tidy: fix `performance-unnecessary-copy-initialization` lintsBen Boeckel2022-05-2410-20/+20
|
* clang-tidy: address `google-readability-casting` lintsBen Boeckel2022-05-2430-73/+95
| | | | At least those involving `static_cast`.
* clang-tidy: address `readability-container-data-pointer` lintsBen Boeckel2022-05-2412-50/+50
|
* clang-tidy: address `modernize-use-default-member-init` lintsBen Boeckel2022-05-2454-212/+109
|
* zstd: suppress an analyzer lintBen Boeckel2022-05-241-0/+1
| | | | It is detected since the asserts go to nothing in a non-debug build.
* cmGlobalGenerator: check for `nullptr` in `GetLanguageFromExtension`Ben Boeckel2022-05-211-1/+4
| | | | Found by `clang-analyzer`.
* cmCTestSubmitHandler: avoid double fetching envvarsBen Boeckel2022-05-211-4/+4
| | | | | Technically, they can change between these two calls, so use the verified pointer to assign to strings. Discovered by `clang-analyzer`.
* cmMakefileLibraryTargetGenerator: improve output buildingBen Boeckel2022-05-191-1/+3
| | | | | GCC warns about buffer juggling here. To avoid the warning, improve the code to avoid resizing on the following two pushes.
* style: use `cmStrCat` in some more locationsBen Boeckel2022-05-193-4/+3
|
* Merge topic 'find_item-doc-update'Brad King2022-05-194-51/+68
|\ | | | | | | | | | | | | 98af010b5c Help: Avoid duplicating registry query syntax in multiple find_* commands Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7278
| * Help: Avoid duplicating registry query syntax in multiple find_* commandsMarc Chevrier2022-05-184-51/+68
| | | | | | | | | | | | In commit 8d7e80cf3d (find_* commands: add control over Windows registry views, 2022-04-16) we added documentation that is repeated by multiple find commands. Move it to a dedicated manual section to avoid duplication.
* | Merge topic 'ignore-werror-command-arg'Brad King2022-05-198-6/+41
|\ \ | | | | | | | | | | | | | | | | | | 65f7053d6c COMPILE_WARNING_AS_ERROR: Add command-line option Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7268
| * | COMPILE_WARNING_AS_ERROR: Add command-line optionMartin Duffy2022-05-188-6/+41
| | | | | | | | | | | | | | | | | | | | | Add command-line option `--compile-no-warning-as-error` to ignore value of `COMPILE_WARNING_AS_ERROR`. Issue: #19085
* | | Merge topic 'verify-interface-header-sets'Brad King2022-05-1935-64/+66
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 259c265112 VERIFY_HEADER_SETS: Rename to VERIFY_INTERFACE_HEADER_SETS Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !7279
| * | | VERIFY_HEADER_SETS: Rename to VERIFY_INTERFACE_HEADER_SETSKyle Edwards2022-05-1835-64/+66
| | | | | | | | | | | | | | | | Issue: #23448
* | | | Merge branch 'release-3.23'Brad King2022-05-190-0/+0
|\ \ \ \
| * \ \ \ Merge topic 'cpack-nuget-components-on-windows' into release-3.23Brad King2022-05-191-1/+3
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 066ff258db CPack/NuGet: Restore component packaging on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7280
* | \ \ \ \ Merge topic 'cpack-nuget-components-on-windows'Brad King2022-05-191-1/+3
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 066ff258db CPack/NuGet: Restore component packaging on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7280
| * | | | | CPack/NuGet: Restore component packaging on WindowsAlex Turbov2022-05-181-1/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The change from `\` to `/` in commit dade75509f (CPack/NuGet: Fix nuspec file generation for UNIX-style OS, 2020-12-21, v3.20.0-rc1~243^2) does not work on Windows. Use `cmake_path` to form a path to an image directory with preinstalled files. Fixes: #22489
* | | | | CMake Nightly Date StampKitware Robot2022-05-191-1/+1
| |/ / / |/| | |
* | | | Merge branch 'release-3.23'Brad King2022-05-180-0/+0
|\ \ \ \ | |/ / /
| * | | Merge topic 'ccmake-fix-infinite-loop-search' into release-3.23Brad King2022-05-182-7/+8
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 432a8ab739 ccmake: Fix infinite loop during invalid search Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7269
* | \ \ \ Merge topic 'ccmake-fix-infinite-loop-search'Brad King2022-05-182-7/+8
|\ \ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | 432a8ab739 ccmake: Fix infinite loop during invalid search Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7269
| * | | | ccmake: Fix infinite loop during invalid searchMartin Duffy2022-05-182-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issue introduced in commit 2defe9ff95 (ccmake: Fix crash when deleting all cache entries, 2022-05-12). Leaving the Empty Cache widget always on the Entry list had unforeseen consequences, including that it caused an infinite loop when searching for a string that did not match any variables.
* | | | | Merge branch 'release-3.23'Brad King2022-05-180-0/+0
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Merge topic 'clang-imsvc' into release-3.23Brad King2022-05-181-1/+1
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 23a52561c7 Clang: Use -imsvc flag without space to support Clang tools Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Raul Tambre <raul@tambre.ee> Acked-by: hihig2001 <hihig@bluehole.net> Merge-request: !7274
* | \ \ \ \ Merge topic 'clang-imsvc'Brad King2022-05-181-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | faed1562fb Merge branch 'backport-3.23-clang-imsvc' into clang-imsvc 23a52561c7 Clang: Use -imsvc flag without space to support Clang tools Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Raul Tambre <raul@tambre.ee> Acked-by: hihig2001 <hihig@bluehole.net> Merge-request: !7274
| * \ \ \ \ \ Merge branch 'backport-3.23-clang-imsvc' into clang-imsvcBrad King2022-05-171-1/+1
| |\ \ \ \ \ \ | | | |/ / / / | | |/| | | |
| | * | | | | Clang: Use -imsvc flag without space to support Clang toolsBrad King2022-05-171-1/+1
| | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit bb61c2d024 (Clang: use -imsvc for system include dirs when running on Windows, 2020-09-16, v3.19.0-rc1~162^2) we use the `-imsvc` flag followed by the include directory as a separate argument. Some versions of `clang-tidy` do not support the flag unless the include directory is attached to it, so use that form instead. A similar fix was made by commit 3a8b6653dc (MSVC: Use -external:I flag without space to support Clang tools, 2022-01-11, v3.22.2~8^2). Fixes: #23517
* | | | | | Merge topic 'FindGLUT-remove-undocumented-vars'Brad King2022-05-182-4/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6c053e68df FindGLUT: Remove undocumented variables GLUT_INCLUDE_PATH and GLUT_LIBRARY Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7273
| * | | | | | FindGLUT: Remove undocumented variables GLUT_INCLUDE_PATH and GLUT_LIBRARYLetu Ren2022-05-172-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #23370
* | | | | | | Merge topic 'fetchcontent-subbuild-config-type-fix'Brad King2022-05-181-3/+3
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 82f3ceb26b FetchContent: Use Debug rather than Release for sub-build Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7272
| * | | | | | FetchContent: Use Debug rather than Release for sub-buildCraig Scott2022-05-171-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The config used can be part of the path to the sub-build. On some platforms, there are restrictions on the length of paths or file names. Use Debug rather than Release because the former is shorter. When CMAKE_CONFIGURATION_TYPES is left at its default, this also makes the path the same as the previous default before 10865c8e5f (FetchContent: Hard-code a config for the sub-build, 2022-05-01) .
* | | | | | | CMake Nightly Date StampKitware Robot2022-05-181-1/+1
| |_|/ / / / |/| | | | |
* | | | | | Merge topic 'try_compile-project-platform-vars'Brad King2022-05-1716-2/+125
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a6562ff579 try_compile: Add option to skip passing platform variables 4843a37676 try_compile: Propagate platform variables in project-mode too Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !7265
| * | | | | | try_compile: Add option to skip passing platform variablesBrad King2022-05-1610-3/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a `CMAKE_TRY_COMPILE_NO_PLATFORM_VARIABLES` variable to tell `try_compile` not to pass platform variables to the test project. Issue: #23219
| * | | | | | try_compile: Propagate platform variables in project-mode tooBrad King2022-05-1614-2/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add policy CMP0137 to propagate both our builtin variables and those listed by `CMAKE_TRY_COMPILE_PLATFORM_VARIABLES` to `try_compile` whole-project builds. Inspired-by: Alexander Neumann <Alexander.Neumann@hamburg.de> Fixes: #23219
* | | | | | | Merge topic 'update-curl'Brad King2022-05-1727-159/+291
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 71747a28ea Merge branch 'upstream-curl' into update-curl 9d8f81f4f8 curl 2022-05-11 (462196e6) 02902188ec curl: Update script to get curl 7.83.1 Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7267
| * \ \ \ \ \ \ Merge branch 'upstream-curl' into update-curlBrad King2022-05-1626-158/+290
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream-curl: curl 2022-05-11 (462196e6)
| | * | | | | | | curl 2022-05-11 (462196e6)Curl Upstream2022-05-1626-158/+290
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code extracted from: https://github.com/curl/curl.git at commit 462196e6b4a47f924293a0e26b8e9c23d37ac26f (curl-7_83_1).
| * | | | | | | | curl: Update script to get curl 7.83.1Brad King2022-05-161-1/+1
| | | | | | | | |
* | | | | | | | | Merge topic 'findzlib-static'Brad King2022-05-171-6/+16
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3f0b9663a4 FindZLIB: fix ZLIB_USE_STATIC_LIBS on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7262
| * | | | | | | | | FindZLIB: fix ZLIB_USE_STATIC_LIBS on WindowsPeter Würth2022-05-131-6/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - on Windows, the ZLIB_USE_STATIC_LIBS options requires a reordering of the search names in order to prefer the static library, since in most cases both static and shared libraries are built - add zlibwapi[d] library name for the contrib/vstudio builds - add lib prefix and .dll.a suffix for the win32/Makefile.gcc build Fixes #23140
* | | | | | | | | | Merge topic 'install-all-components-fix'Brad King2022-05-176-10/+31
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b0c8e31b54 install: Don't ignore EXCLUDE_FROM_ALL when used with ALL_COMPONENTS Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !7263