summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Use C++11 nullptr (cont.)Matthias Maennich2017-09-1914-67/+68
| | | | | | | Fix remaining occurrences of the issue addressed in commit 5962db4389 (Use C++11 nullptr, 2017-08-22) that are only showing up on macOS. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Merge topic 'ctest-runcommand'Brad King2017-09-183-26/+42
|\ | | | | | | | | | | | | | | 67529aab Doc: document that CoverageExtraFlags will come first 69fac3c3 pass arguments as vector to cmCTest::RunCommand() Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1261
| * pass arguments as vector to cmCTest::RunCommand()Rolf Eike Beer2017-09-153-26/+42
| | | | | | | | | | | | | | The only 2 callers took care to construct a properly escaped string, but not using the documented way, and that string was passed only to be immediately split into tokens again. Start with a vector and join it only for logging, avoiding needless quotes during that.
* | Merge topic 'cmake-gui-high-dpi'Brad King2017-09-181-0/+4
|\ \ | | | | | | | | | | | | | | | | | | 6fb36572 cmake-gui: Enable High DPI scaling Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1271
| * | cmake-gui: Enable High DPI scalingLuis Caro2017-09-151-0/+4
| |/ | | | | | | | | | | | | | | | | | | This Qt flag enables UI scaling based on scale factor provided by the operating system. Only supported on versions of Qt >= 5.6. The flag has no effect on macOS, where high DPI support is already enabled as a setting in the bundle's info.plist. Fixes: #17026
* | Merge topic 'libuv-bootstrap'Brad King2017-09-181-3/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | 714ce728 bootstrap: Make libuv available during bootstrap 6a2d967d bootstrap: Require compiler mode aware of C99 on Solaris Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1131
| * | bootstrap: Make libuv available during bootstrapBrad King2017-09-171-3/+4
| | | | | | | | | | | | | | | | | | | | | On UNIX, build only the parts of libuv we need for the filesystem, process, and poll abstractions using the POSIX poll() backend. This avoids many platform-specific conditions. On Windows, build all of libuv; there are no conditional alternatives anyway.
* | | Merge topic 'cxx11-override'Brad King2017-09-18254-1164/+1090
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 98530d3c cm_codecvt: Tell IWYU to keep cmConfigure.h 4e14498f Drop now-unused definition of CM_OVERRIDE 0b33aee4 Use C++11 override instead of CM_OVERRIDE Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1265
| * | | cm_codecvt: Tell IWYU to keep cmConfigure.hBrad King2017-09-151-1/+1
| | | | | | | | | | | | | | | | | | | | We no longer need it explicitly for `CM_OVERRIDE`, but we still need to include it first everywhere.
| * | | Drop now-unused definition of CM_OVERRIDEBrad King2017-09-153-32/+0
| | | |
| * | | Use C++11 override instead of CM_OVERRIDEBrad King2017-09-15251-1131/+1089
| | |/ | |/| | | | | | | | | | | | | | | | | | | We now require C++11 support including `override`. Drop use of the old compatibility macro. Convert references as follows: git grep -l CM_OVERRIDE -- '*.h' '*.hxx' '*.cxx' | xargs sed -i 's/CM_OVERRIDE/override/g'
* | | CMake Nightly Date StampKitware Robot2017-09-181-1/+1
| | |
* | | CMake Nightly Date StampKitware Robot2017-09-171-1/+1
| | |
* | | CMake Nightly Date StampKitware Robot2017-09-161-1/+1
|/ /
* | Merge topic 'refactor-iwyu-code'Brad King2017-09-157-233/+303
|\ \ | | | | | | | | | | | | | | | | | | 3bbe95f5 Clean up iwyu code to not be one big if statement. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1247
| * | Clean up iwyu code to not be one big if statement.Bill Hoffman2017-09-137-233/+303
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes the internal -E__run_iwyu to be -E__run_co_compile. This is used for co-compile commands. These are tools that want to mirror the compiler. For each compiler invocation the tool will be invoked first. This started as a way to implement include what you use (iwyu), but has expanded to include cpplint, cppcheck and others. Likely there will be more in the future as well. This commit implements each one in its own function and provides a way to add additional ones in the future with less work.
* | | Merge topic 'codeblocks-exclude-external'Brad King2017-09-151-0/+20
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | fb19b778 CodeBlocks: add option to exclude external files Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1159
| * | | CodeBlocks: add option to exclude external filesAlexandr (Sagrer) Gridnev2017-09-141-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add variable `CMAKE_CODEBLOCKS_EXCLUDE_EXTERNAL_FILES` to optionally exclude files from outside the project root from the project file written by the CodeBlocks extra generator. This optionally restores logic that had been removed by commit v2.8.3~40^2 (CodeBlocks Generator: Do not omit files in the project file listing, 2010-10-05) in response to QTCREATORBUG-2250. Issue: #12110 Fixes: #17188
* | | | Merge topic 'ranged-for'Brad King2017-09-1533-869/+606
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 63f6fd14 Meta: modernize old-fashioned loops to range-based `for` (CTest). Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1262
| * | | | Meta: modernize old-fashioned loops to range-based `for` (CTest).Pavel Solodovnikov2017-09-1433-869/+606
| | |_|/ | |/| | | | | | | | | | | | | | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* | | | Merge topic 'update-kwsys'Brad King2017-09-152-66/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f02eced5 Merge branch 'upstream-KWSys' into update-kwsys 38b8017f KWSys 2017-09-14 (d85b17e7) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1263
| * | | | Merge branch 'upstream-KWSys' into update-kwsysBrad King2017-09-142-66/+0
| | | | | | | | | | | | | | | | | | | | | | | | | * upstream-KWSys: KWSys 2017-09-14 (d85b17e7)
* | | | | CMake Nightly Date StampKitware Robot2017-09-151-1/+1
|/ / / /
* | | | Merge topic 'get_filename_component-fix-program-split'Brad King2017-09-143-1/+79
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 31f73eb1 get_filename_component: Revise PROGRAM/PROGRAM_ARGS split semantics Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1251
| * | | | get_filename_component: Revise PROGRAM/PROGRAM_ARGS split semanticsBrad King2017-09-133-1/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The KWSys `SystemTools::SplitProgramFromArgs` implementation goes into an infinite loop when the value is just " " (a space). Since the "program path with unquoted spaces plus command-line arguments" operation it is trying to provide is poorly defined (string parsing should not depend on filesystem content), just stop using it. Instead consider the main two use cases the old approach tried to handle: * The value is the name or absolute path of a program with no quoting or escaping, but also no command-line arguments. In this case we can use the value as given with no parsing, and assume no arguments. * The value is a command-line string containing the program name/path plus arguments. In this case we now assume that the command line is properly quoted or escaped. Fixes: #17262
* | | | | Merge topic 'vs-fix-config-map'Brad King2017-09-142-8/+20
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 64e973e9 Merge branch 'backport-vs-fix-config-map' into vs-fix-config-map c5b5bb27 VS: Do not consider MAP_IMPORTED_CONFIG_<CONFIG> on non-imported targets Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1256
| * | | | Merge branch 'backport-vs-fix-config-map' into vs-fix-config-mapBrad King2017-09-132-8/+20
| |\ \ \ \ | | |_|/ / | |/| | |
| | * | | VS: Do not consider MAP_IMPORTED_CONFIG_<CONFIG> on non-imported targetsBrad King2017-09-132-8/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.9.0-rc1~309^2 (include_external_msproject: Honor MAP_IMPORTED_CONFIG_<CONFIG>, 2017-04-04) we accidentally honor `MAP_IMPORTED_CONFIG_<CONFIG>` while generating the `.sln` file entries for normal targets. This causes `devenv.com`-driven builds to use the mapping incorrectly for normal targets. Check that a target really comes from `include_external_msproject` before considering the map. Furthermore, when we do use the map, we should only take the first entry if more than one configuration is specified. Otherwise we end up giving VS a configuration name with a `;` in it. Fixes: #17276
| | * | | CMake 3.9.2v3.9.2Brad King2017-09-071-1/+1
| | | | |
| | * | | Merge branch 'backport-vs-csharp-ref-no-asm' into release-3.9Brad King2017-09-053-0/+23
| | |\ \ \ | | | | | | | | | | | | | | | | | | Merge-request: !1203
| | * \ \ \ Merge branch 'fix-genex-SOURCES' into release-3.9Brad King2017-09-011-1/+1
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | Merge-request: !1218
* | | | | | | CMake Nightly Date StampKitware Robot2017-09-141-1/+1
|/ / / / / /
* | | | | | Merge topic 'MsvcArm64'Brad King2017-09-135-1/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bc7c94fe MSVC: Add support for ARM64 architecture Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1215
| * | | | | | MSVC: Add support for ARM64 architectureMinmin Gong2017-09-125-1/+13
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Visual Studio 15.4 adds support for this architecture. Fixes: #17213
* | | | | | Merge topic 'ranged-for'Brad King2017-09-13133-3592/+2456
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7d509579 Meta: modernize old-fashioned loops to range-based `for`. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1249
| * | | | | | Meta: modernize old-fashioned loops to range-based `for`.Pavel Solodovnikov2017-09-12133-3592/+2456
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* | | | | | | CMake Nightly Date StampKitware Robot2017-09-131-1/+1
| |_|_|_|_|/ |/| | | | |
* | | | | | Merge topic 'vs-clang-llvm-support'Brad King2017-09-121-2/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8a4755ca VS: Update support for LLVM-vs* toolsets from LLVM 5.0 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1231
| * | | | | | VS: Update support for LLVM-vs* toolsets from LLVM 5.0Konstantin Ivlev2017-09-111-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert commit v3.7.0-rc1~25^2 (VS: Recognize VS/LLVM toolset names as Clang, 2016-09-28). Since at least LLVM 5.0 the VS integration of the LLVM toolchain now mimics cl and accepts MSVC-style command-line arguments (unlike Microsoft Clang/C2). Fixes: #17193, #17235
* | | | | | | Merge topic 'get-or-create-source-group'Brad King2017-09-124-103/+85
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 95b17c89 Use cmMakefile::GetOrCreateSourceGroup in cmQtAutogeneratorsInitializer a451995f Use cmMakefile::GetOrCreateSourceGroup in cmSourceGroupCommand 1e6569c9 cmMakefile: Add GetOrCreateSourceGroup methods 3e8b3e94 cmMakefile: Collect source group methods in one place Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1243
| * | | | | | Use cmMakefile::GetOrCreateSourceGroup in cmQtAutogeneratorsInitializerSebastian Holtermann2017-09-091-14/+1
| | | | | | |
| * | | | | | Use cmMakefile::GetOrCreateSourceGroup in cmSourceGroupCommandSebastian Holtermann2017-09-091-32/+2
| | | | | | |
| * | | | | | cmMakefile: Add GetOrCreateSourceGroup methodsSebastian Holtermann2017-09-092-0/+33
| | | | | | |
| * | | | | | cmMakefile: Collect source group methods in one placeSebastian Holtermann2017-09-092-57/+49
| | | | | | |
* | | | | | | Merge topic 'iwyu-update'Brad King2017-09-121-0/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ea7177b9 cmCustomCommandGenerator: Fix include-what-you-use diagnostic Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1252
| * | | | | | | cmCustomCommandGenerator: Fix include-what-you-use diagnosticBrad King2017-09-111-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For `size_t` we should include `stddef.h`.
* | | | | | | | CMake Nightly Date StampKitware Robot2017-09-121-1/+1
|/ / / / / / /
* | | | | | | Merge topic 'vs_improve_custom_command'Brad King2017-09-114-8/+27
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9ed24280 VS: only add custom command line if it is not empty 34c4108b add HasOnlyEmptyCommandLines() method to cmCustomCommandGenerator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1050
| * | | | | | | VS: only add custom command line if it is not emptyMichael Stürmer2017-09-042-8/+14
| | | | | | | |
| * | | | | | | add HasOnlyEmptyCommandLines() method to cmCustomCommandGeneratorMichael Stürmer2017-09-042-0/+13
| | | | | | | |