summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* FindGTest: Improve test to catch link errorMatthew Woehlke2017-09-181-0/+2
| | | | | | | Add a reference to one of Google Test's command-line flags to the FindGTest test. This will ensure that we are using the correct compile definitions on Windows, as the test will otherwise fail to link. (IOW, this tests the changes made by the previous commit.)
* Merge topic 'vs_improve_custom_command'Brad King2017-09-113-0/+89
|\ | | | | | | | | | | | | | | 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-043-0/+89
| |
* | Merge topic 'timestamp'Brad King2017-09-112-2/+2
|\ \ | | | | | | | | | | | | | | | | | | 014ad298 Timestamp: support %A and %B Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1228
| * | Timestamp: support %A and %BBernhard M. Wiedemann2017-09-072-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | These are defined both by [1] and [2] to give full names of a weekday and month. [1] http://pubs.opengroup.org/onlinepubs/009695399/functions/strftime.html [2] https://msdn.microsoft.com/de-de/library/fe06s4ak.aspx
* | | CTest: Weight reported test times by PROCESSORS in summariesBill Hoffman2017-09-088-15/+15
| | | | | | | | | | | | | | | | | | This commit changes the times reported by labels and subprojects to be weighted by the PROCESSORS property. It is reported with `sec*proc` instead of just `sec`.
* | | CTest: Split out labels and subproject labels in summary outputBill Hoffman2017-09-085-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | This commit splits out the reporting of labels and labels used for sub projects. If a label is a sub project label it will not be included in the label summary. To implement this the commit creates PrintLabelOrSubprojectSummary which is able to do the work of both PrintLabelSummary and PrintSubprojectSummary avoiding code duplication.
* | | Merge topic 'fphsa-version-cleanup'Brad King2017-09-086-0/+23
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 91935d9e FPHSA: lower-case for cmake_parse_arguments + move config-mode related code aec27c15 FPHSA: use more localized _FOUND_VERSION instead of VERSION. 0019bfeb FPHSA: Add (failing) unit test in case a cache variable VERSION is present 81844334 FPHSA: add (failing) unit test for found version being equal to 0. Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Rolf Eike Beer <eike@sf-mail.de> Merge-request: !1224
| * | | FPHSA: Add (failing) unit test in case a cache variable VERSION is presentYves Frederix2017-09-054-1/+15
| | | |
| * | | FPHSA: add (failing) unit test for found version being equal to 0.Yves Frederix2017-09-053-0/+9
| | | |
* | | | Merge topic 'findfreetype-importedtargets'Brad King2017-09-084-0/+64
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b1d1c56c Help: Add notes for topic 'FindFreetype-importedtargets' 457c2021 FindFreetype: Add imported target, docs, and test Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1219
| * | | | FindFreetype: Add imported target, docs, and testBen Morgan2017-09-074-0/+64
| | |/ / | |/| |
* | | | Autogen: Tests: Add AUTOGEN_TARGET_DEPENDS test to mocDepends testsSebastian Holtermann2017-09-0710-54/+128
|/ / /
* | | Merge topic 'findexpat-importedtargets'Brad King2017-09-074-0/+51
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3a4f82e9 Help: Add notes for topic 'FindEXPAT-importedtargets' bfe51369 FindEXPAT: Add unit test of target and variables 78f166f8 FindEXPAT: Add imported target and documentation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1207
| * | | FindEXPAT: Add unit test of target and variablesBen Morgan2017-09-014-0/+51
| | | |
* | | | Merge topic 'vs-csharp-ref-no-asm'Brad King2017-09-063-0/+25
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 94e70e53 Merge branch 'backport-vs-csharp-ref-no-asm' into vs-csharp-ref-no-asm 7e57e6ae VS: Do not reference output assemblies if not possible for CSharp target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1203
| * \ \ \ Merge branch 'backport-vs-csharp-ref-no-asm' into vs-csharp-ref-no-asmBrad King2017-09-053-0/+25
| |\ \ \ \ | | |_|_|/ | |/| | |
| | * | | VS: Do not reference output assemblies if not possible for CSharp targetMichael Stürmer2017-09-053-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.9.0-rc4~4^2 (Vs: allow CSharp targets to be linked to CXX targets, 2017-06-20) CSharp targets get `ProjectReference` entries to their dependencies. This causes VS to also reference the dependency's output assembly by default, which is incorrect for non-managed targets. Fix this by setting `ReferenceOutputAssembly` to `false` for targets that can't provide output assemblies. Unmanaged C++ targets (shared libs & executables) can still be referenced and a warning will be shown in the IDE but the build will not break anymore. Fixes: #17172
| | * | | Merge branch 'fix-find-package-mode' into release-3.9Brad King2017-08-291-0/+2
| | |\ \ \ | | | | | | | | | | | | | | | | | | Merge-request: !1208
* | | \ \ \ Merge topic 'ExternalProject-command'Brad King2017-09-053-0/+55
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 15617484 ExternalProject: Prevent COMMAND from being treated as a true keyword Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1178
| * | | | | | ExternalProject: Prevent COMMAND from being treated as a true keywordCraig Scott2017-09-023-0/+55
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The known keywords for each function are obtained by scraping the documentation for lines matching a particular regular expression. In commit 8842a027 (ExternalProject: Improve documentation, 2017-07-09), the docs were overhauled and the COMMAND docs subsequently matched the regular expression when they shouldn't have. This made COMMAND appear as a true keyword, which thwarted the special handling logic elsewhere for the intended use of COMMAND arguments. This commit contains a workaround for issue #17229 to force a dependency of the patch step on the update step to ensure a predictable step order. Fixes: #17198
* | | | | | FindPatch: Add module to find 'patch' command-line toolJean-Christophe Fillion-Robin2017-09-013-0/+89
| |/ / / / |/| | | |
* | | | | Merge topic 'server-refactor'Brad King2017-09-013-0/+107
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0e4d5033 server: Added thread check to protect writedata 124424e9 server: Protect several fields from potentially pointing to bad memory 693fa0a9 server: Added assert to monitor uv_run status 882dcef8 server: Made connections in a server have a mutex to avoid use after frees 7ef28843 server: Moved buffer formatting into bufferstrategy dc7a18d8 server: test buffer parsing f8fd5a97 server: Made stdio connection accept different types of streams 6afc7f88 server: Remove unused fields / functions ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1111
| * | | | | server: test buffer parsingJustin Berger2017-08-313-0/+107
| | |_|_|/ | |/| | |
* | | | | Merge topic 'tll-global-unknown-lib'Brad King2017-09-012-0/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a47a8533 target_link_libraries: Allow linking to UNKNOWN IMPORTED GLOBAL libs Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1220
| * | | | | target_link_libraries: Allow linking to UNKNOWN IMPORTED GLOBAL libsBrad King2017-08-312-0/+5
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | This combination was accidentally rejected. Allow it and add a test. Fixes: #17245
* | | | | Merge topic 'fix-genex-SOURCES'Brad King2017-09-015-0/+15
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | 068cc545 Genex: Fix TARGET_PROPERTY value of SOURCES Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1218
| * | | | Genex: Fix TARGET_PROPERTY value of SOURCESBrad King2017-09-015-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactoring in commit v3.8.0-rc1~445^2~2 (cmTarget: Move sanity checks and computed property access to callers, 2016-10-13) exposed a typo in commit v3.8.0-rc1~445^2~3 (cmGeneratorTarget: Implement cmTargetPropertyComputer interface, 2016-10-13). Together they broke the `$<TARGET_PROPERTY:mytgt,SOURCES>` generator expression in the case that the `SOURCES` target property is populated in part by the `target_sources` command. Add the missing `;`-separator. Fixes: #17243
* | | | | Merge topic 'cxx11-clang-format'Brad King2017-08-315-29/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4022b286 clang-tidy: blacklist "modernize" checks 2b4c32c9 clang-format: format all code as Cpp11 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1191
| * | | | | clang-format: format all code as Cpp11Daniel Pfeifer2017-08-305-29/+12
| | | | | |
* | | | | | Merge topic 'test-server-result'Brad King2017-08-312-1/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ea359bc5 Tests: Teach Server test to print server return code Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1212
| * | | | | | Tests: Teach Server test to print server return codeBrad King2017-08-302-1/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | This will aid in diagnosing failures.
* | | | | | Merge topic 'lexer-null'Brad King2017-08-314-0/+7
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 14d9a11b ListFileLexer: fix heap-buffer-overflow on malicious input Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1153
| * | | | | ListFileLexer: fix heap-buffer-overflow on malicious inputMatthias Maennich2017-08-304-0/+7
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In case a list file contains a null terminated string that is continued until a later space, the lexer token information got inconsistent: e.g. an argument "TEST\0FOOBAR" is passed by the lexer as a token char* = "TEST\0FOOBAR" and length 11 ^^ note: ascii 0x00 Using strdup in cmListFileLexer leads lexer->token.text to be allocated with size 5 and lexer->token.length to be set to 11 A subsequent call to this function with an argument of 5 < length <= 11 wrongly assumed a sufficiently sized buffer and therefore corrupted the heap buffer. The program might crash due to this corruption. The case "NullTerminatedArgument" is intentionally using a quite large 'rest' to increase the chance to actually hit the issue. It will reliably crash with address sanitizer enabled though. This fix addresses all rules where arbitrary characters are matched to ignore \0 in order to fall through to the rule that matches an arbitrary character as BadCharacter. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* | | | | Merge topic 'add-cppcheck'Brad King2017-08-3015-1/+74
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 311b7b1a Add properties to run cppcheck along with the compiler Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1147
| * | | | | Add properties to run cppcheck along with the compilerBill Hoffman2017-08-3015-1/+74
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Create a `<LANG>_CPPCHECK` target property (initialized by a `CMAKE_<LANG>_CPPCHECK` variable) to specify a `cppcheck` command line to be run along with the compiler.
* | | | | Merge topic 'fix-find-package-mode'Brad King2017-08-301-0/+2
|\ \ \ \ \ | |/ / / / |/| | | / | | |_|/ | |/| | | | | | | | | | 18454ea4 cmake: Fix --find-package mode with imported targets Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1208
| * | | cmake: Fix --find-package mode with imported targetsBrad King2017-08-291-0/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | The change in commit v3.9.0-rc1~116^2~6 (cmakemain: use script role for -P, 2017-05-11) accidentally left project commands out of find-package mode, causing packages that provide imported targets to break. Fixes: #17124
* | | Merge topic 'no-crlf-blobs'Brad King2017-08-3020-1043/+1043
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ebc91a44 Avoid CRLF newlines in Git repo blobs c69b4c8d bzip2: Drop unused .dsp files Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !1211
| * | | Avoid CRLF newlines in Git repo blobsBrad King2017-08-3020-1043/+1043
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit 8ed03baa76 (gitattributes: prefer `eol=crlf` to `-crlf`, 2017-08-23) we left a few CRLF blobs in the repository. Some Git versions get confused by text files with CRLF blobs. Convert them to LF blobs. Use the `eol=crlf` attribute to tell Git to use CRLF on checkout.
* | | | Merge topic 'cstyle-casts'Daniel Pfeifer2017-08-301-2/+3
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | 190e3825 Replace C-style casts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1176
| * | | Replace C-style castsDaniel Pfeifer2017-08-271-2/+3
| | | |
* | | | Merge topic 'test-fix-vs15-toolset-v90'Brad King2017-08-281-1/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ce602feb Tests: Fix RunCMake.BuildDepends on VS 2017 with v90 toolset Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1186
| * | | | Tests: Fix RunCMake.BuildDepends on VS 2017 with v90 toolsetBrad King2017-08-251-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | The MSVC toolsets v100 and below forget to re-link when a manifest changes. Exclude this part of the test case for them.
* | | | | Merge topic 'server-cxx11'Daniel Pfeifer2017-08-271-7/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 27d87fbd CTestCustom: Suppress exception loosening warning 7f29bbe6 server: always enable server 4614a3b2 server: backport to C++11 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1149
| * | | | | server: always enable serverDaniel Pfeifer2017-08-261-7/+5
| | |/ / / | |/| | |
* | | | | IWYU: Mark cmConfigure.h with pragma: keepDaniel Pfeifer2017-08-262-2/+2
|/ / / / | | | | | | | | | | | | Also remove `#include "cmConfigure.h"` from most source files.
* | | | Merge topic 'cxx11-nullptr'Brad King2017-08-252-3/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5962db43 Use C++11 nullptr Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1175
| * | | | Use C++11 nullptrDaniel Pfeifer2017-08-242-3/+2
| | | | |
* | | | | Merge topic 'modernize-git-whitespace-attributes'Brad King2017-08-2512-17/+10
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8ed03baa gitattributes: prefer `eol=crlf` to `-crlf` 40470031 MumpsCoverage: remove unnecessary attribute d20485d8 gitattributes: prefer `eol=lf` to `crlf=input` b85c2589 gitattributes: prefer `-text` for binary files 377f3685 gitattributes: remove unused attributes Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1173