summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | Autogen: Doc: Add examples to the SKIP_AUTO* documentationsSebastian Holtermann2017-09-284-5/+41
| | | | | | | | | | |
| * | | | | | | | | | Autogen: Doc: Update CMP0071 descriptionSebastian Holtermann2017-09-281-12/+20
| | | | | | | | | | |
| * | | | | | | | | | Autogen: Offer solution for CMP0071 in warning messageSebastian Holtermann2017-09-281-28/+37
| |/ / / / / / / / /
* | | | | | | | | | Merge topic 'cxx11-eq-delete'Brad King2017-09-296-36/+3
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b8018135 Use C++11 '= delete' instead of CM_EQ_DELETE 42f4e697 Drop unused CMake_HAVE_CXX_UNIQUE_PTR macro Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1326
| * | | | | | | | | | Use C++11 '= delete' instead of CM_EQ_DELETEBrad King2017-09-286-35/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now require C++11 support including `= delete`. Drop use of the old compatibility macro.
| * | | | | | | | | | Drop unused CMake_HAVE_CXX_UNIQUE_PTR macroBrad King2017-09-281-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now require C++11 `std::unique_ptr` support from the compiler, so none of our code is conditioned on the macro.
* | | | | | | | | | | Merge topic 'clang-tidy'Brad King2017-09-2935-339/+310
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 77f674be Fix some occurrences of readability-braces-around-statements f0bab294 Convert some leftover loops to C++11 range-based loop b5d7f5b0 Fix occurrences of readability-non-const-parameter 9a2da339 Fix some occurrences of readability-avoid-const-params-in-decls 870dd06d Fix left-over occurrences of else-after-return 2033abff Fix minor clang-tidy findings 79b8c380 Improve several occurrences of vector::push_back in loops a45928cd Fix some occurrences of missing override keywords ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1292
| * | | | | | | | | | | Fix some occurrences of readability-braces-around-statementsMatthias Maennich2017-09-285-18/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [readability-braces-around-statements] Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Convert some leftover loops to C++11 range-based loopMatthias Maennich2017-09-289-232/+165
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [modern-loop-convert] Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Fix occurrences of readability-non-const-parameterMatthias Maennich2017-09-281-12/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [readability-non-const-parameter] The patch separates the definitions of the function headers for the two variants (CMAKE_USE_ELF_PARSER or not) and comments out the parameter names to not consider them for any const-ness in the case they are actually not even looked at. Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Fix some occurrences of readability-avoid-const-params-in-declsMatthias Maennich2017-09-282-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [readability-avoid-const-params-in-decls] Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Fix left-over occurrences of else-after-returnMatthias Maennich2017-09-285-31/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [readability-else-after-return] These were mostly only showing up on OSX. Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Fix minor clang-tidy findingsMatthias Maennich2017-09-284-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy - modernize-use-bool-literals - misc-string-integer-assignment - performance-faster-string-find - readability-redundant-string-cstr - readability-delete-null-pointer Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Improve several occurrences of vector::push_back in loopsMatthias Maennich2017-09-2811-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy by pre-allocating the vector capacity before the loop [performance-inefficient-vector-operation]. Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Fix some occurrences of missing override keywordsMatthias Maennich2017-09-284-13/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [modernize-use-override]. These occurrences are only showing up on OSX. Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Clean up some C-Style castsMatthias Maennich2017-09-282-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [google-readability-casting] Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Fix some occurrences using string by value rather than by const&Matthias Maennich2017-09-288-18/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy - performance-unnecessary-value-param - performance-unnecessary-copy-initialization Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | Replace several occurrences of empty string comparisons by string::empty()Matthias Maennich2017-09-282-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix issues diagnosed by clang-tidy [readability-container-size-empty] Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | clang-tidy: Make .clang-tidy a build-time dependencyMatthias Maennich2017-09-282-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently a change to the clang-tidy configuration remains unnoticed for incremental builds in the way that it won't trigger a rebuild. This can be considered a missing dependency that this patch fixes by introducing a compile time definition (-DCLANG_TIDY_SHA1) that triggers a rebuild upon change. Currently this only applies to the target CMakeLib. Signed-off-by: Matthias Maennich <matthias@maennich.net>
| * | | | | | | | | | | clang-tidy: exclude 'modernize-use-noexcept' introduced in 5.0.0Brad King2017-09-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We still support compiling CMake on VS 2013 that does not support `noexcept`, so disable the clang-tidy diagnostic for now.
| * | | | | | | | | | | clang-tidy: exclude 'modernize-return-braced-init-list' introduced in 5.0.0Matthias Maennich2017-09-271-0/+1
| | |_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to many occurrences and as I am not sure about the coding guide lines in regards to this, I blacklist it. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* | | | | | | | | | | CMake Nightly Date StampKitware Robot2017-09-291-1/+1
| |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge topic 'remove-auto_ptr-checks'Brad King2017-09-284-21/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9e0362ca Remove checks for auto_ptr that we no longer need 94e67fa1 cpack: Fix compilation on Cygwin after auto_ptr removal Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1320
| * | | | | | | | | | Remove checks for auto_ptr that we no longer needBrad King2017-09-273-20/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We no longer use `auto_ptr` at all, so drop the checks for it.
| * | | | | | | | | | cpack: Fix compilation on Cygwin after auto_ptr removalBrad King2017-09-271-1/+1
| | |_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix a compilation error on Cygwin introduced by commit f0489856e3 (Retire std::auto_ptr and its macro CM_AUTO_PTR, 2017-09-21).
* | | | | | | | | | CMake Nightly Date StampKitware Robot2017-09-281-1/+1
| |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Merge topic 'document_env_variables'Craig Scott2017-09-2729-0/+261
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e6b77c5f Help: Document CMake's environment variables Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1235
| * | | | | | | | Help: Document CMake's environment variablesRobert Maynard2017-09-2629-0/+261
| | | | | | | | |
* | | | | | | | | Merge topic 'CMakeCPack-source-ignore'Brad King2017-09-272-6/+18
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fa23f66a CMakeCPack: Exclude version-control-specific files from source package Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1314
| * | | | | | | | CMakeCPack: Exclude version-control-specific files from source packageBrad King2017-09-262-6/+18
| | |/ / / / / / | |/| | | | | |
* | | | | | | | Merge topic 'auto_ptr'Brad King2017-09-2754-573/+353
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f0489856 Retire std::auto_ptr and its macro CM_AUTO_PTR Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Pavel Solodovnikov <hellyeahdominate@gmail.com> Merge-request: !1300
| * | | | | | | | Retire std::auto_ptr and its macro CM_AUTO_PTRMatthias Maennich2017-09-2554-573/+353
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthias Maennich <matthias@maennich.net>
* | | | | | | | | Merge topic 'vs-guid-tolerate-no-curly'Brad King2017-09-272-2/+10
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b3e6514c VS: Adapt project parsers to support "ProjectGUID" without curly brackets Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1313
| * | | | | | | | | VS: Adapt project parsers to support "ProjectGUID" without curly bracketsFredrik Orderud2017-09-262-2/+10
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is needed to correctly parse Windows Installer "wiproj" projects, that by default contain "ProjectGUID" tags with GUID values without surrounding curly brackets. Otherwise CMake truncates the first & last character from the GUID value for these projects.
* | | | | | | | | Merge topic 'reduce-strcpy'Brad King2017-09-271-33/+16
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e43cb4c1 CursesDialog: avoid calling strcpy() Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1311
| * | | | | | | | | CursesDialog: avoid calling strcpy()Rolf Eike Beer2017-09-251-33/+16
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also use memset() and a few places where the compiler will collapse the for loop into such a call anyway.
* | | | | | | | | Merge topic 'GetPrerequisites-no-clear-on-missing'Brad King2017-09-275-1/+11
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1a7b8c83 GetPrerequisites: Restore behavior on missing binary of not clearing list Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1312
| * | | | | | | | | GetPrerequisites: Restore behavior on missing binary of not clearing listBrad King2017-09-265-1/+11
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to commit v3.4.0-rc1~264^2~1 (GetPrerequisites: Add error checks for execute_process() calls, 2015-07-29), `get_prerequisites` would simply warn on a missing binary and not update the result list at all. That commit accidentally made the case an error. This was fixed by commit v3.8.0-rc1~110^2 (GetPrerequisites: Do not fail on files we cannot find, 2017-01-10), but the fix also cleared the result list. Clearing the list is incorrect because it is supposed to be able to accumulate results over multiple calls. Remove the list clearing behavior to restore the original behavior on a missing binary. Fixes: #17306
* | | | | | | | | Merge topic 'autogen-per-config-sources'Brad King2017-09-2710-252/+380
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4603d6b0 Autogen: Docs: Add documentation internal links f86ba8ee Autogen: Reintroduce per-config sources support 5d3bca64 Autogen: Rename cmQtAutoGen::GeneratorType to cmQtAutogen::Generator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1307
| * | | | | | | | | Autogen: Docs: Add documentation internal linksSebastian Holtermann2017-09-253-8/+8
| | | | | | | | | |
| * | | | | | | | | Autogen: Reintroduce per-config sources supportSebastian Holtermann2017-09-257-218/+346
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reintroduce per-config sources support in AUTOGEN but disable it by default.
| * | | | | | | | | Autogen: Rename cmQtAutoGen::GeneratorType to cmQtAutogen::GeneratorSebastian Holtermann2017-09-255-29/+29
| | | | | | | | | |
* | | | | | | | | | Merge topic 'ninja-rc-depfile-quoting'Brad King2017-09-272-1/+8
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | df965cb9 Ninja: Fix quoting of RC language depfile in cmcldeps call Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1309
| * | | | | | | | | | Ninja: Fix quoting of RC language depfile in cmcldeps callBrad King2017-09-252-1/+8
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.0.0-rc1~448^2 (Ninja: use deps = gcc/msvc feature, 2013-10-18) the value of the `DEP_FILE` binding already includes the needed quoting to refer to the file. However, that commit forgot to update one of the `$DEP_FILE` references to not be quoted anymore. The offending code path currently only affects cmcldeps for RC. Remove the extra quoting now. Fixes: #17298
* | | | | | | | | | Merge topic 'FindOpenGL-glvnd'Brad King2017-09-275-40/+349
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e2e8a690 FindOpenGL: Add support for GLVND on Linux Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !782
| * | | | | | | | | | FindOpenGL: Add support for GLVND on LinuxTom Fogal2017-09-255-40/+349
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Find GLVND components if available. Add `GLX` and `EGL` options for COMPONENTS that allow requesting these libraries explicitly. Introduce new import targets for these windowing-system-specific libraries. On a GLVND system, populate the legacy `OPENGL_LIBRARIES` variable and the `OpenGL::GL` target using the `OpenGL` and `GLX` components. On non-GLVND systems, continue to use the legacy `GL` library and simply do not provide the GLVND components. Application code can choose to adapt based on the availability of GLVND components as imported targets.
* | | | | | | | | | Merge topic 'revert-perf-source-lookup'Brad King2017-09-274-59/+20
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a7005c98 Tests: Add case for legacy source file property behavior 1604716d Revert "Performance: Improve efficiency of source file lookup in cmMakefile" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1308
| * | | | | | | | | | Tests: Add case for legacy source file property behaviorBrad King2017-09-252-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The change in commit 3b95ab5693 (Performance: Improve efficiency of source file lookup in cmMakefile, 2017-08-17) broke some legacy behavior of source file properties in which the order sources are first resolved with extensions affects how setting properties without extensions works. It has been reverted for now, but the discovery was made after merging because the broken case was not covered by our test suite. Add a test case representing the legacy behavior. Issue: #15208
| * | | | | | | | | | Revert "Performance: Improve efficiency of source file lookup in cmMakefile"Brad King2017-09-252-58/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3b95ab569345028a1a8fe521d5ecd81fa97f2653. It regressed some legacy source file property behavior. Revert pending further investigation.
* | | | | | | | | | | Merge branch 'release-3.9'Brad King2017-09-270-0/+0
|\ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the "ours" merge strategy because we're merging a fixup that is specific to the 3.9 branch and do not need the changes in `master`.