summaryrefslogtreecommitdiffstats
path: root/Tests/RunCMake
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'cpack-deb-mr-1296-fix'Brad King2017-10-114-0/+21
|\ | | | | | | | | | | | | | | c5db2f9a CPack/Deb: fix for regex passing even if invalid characters are present 529729d6 CPack/Deb: CPACK_DEBIAN_PACKAGE_VERSION regex testing exception Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1375
| * CPack/Deb: CPACK_DEBIAN_PACKAGE_VERSION regex testing exceptionDomen Vrankar2017-10-104-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | CPACK_DEBIAN_PACKAGE_VERSION variable could in the past also contain release and epoch version so regex test should expect the entire versioning if both CPACK_DEBIAN_PACKAGE_RELEASE and CPACK_DEBIAN_PACKAGE_EPOCH are not set. Also since the checks were not performed in the past the regex test of CPACK_DEBIAN_PACKAGE_VERSION variable content should only report author warnings instead of errors in case of the test fail. Fixes: #17339
* | Tests: Teach RunCMake to ignore clang unused sanitizer flag warningsBrad King2017-10-101-0/+1
| | | | | | | | | | These can show up in Release builds of CMake because some tests still build Debug.
* | Tests: Refactor RunCMake output line ignore regex constructionBrad King2017-10-101-1/+12
| | | | | | | | Avoid one giant line.
* | Xcode 9: Lower iOS deployment version to get armv7 buildsGregor Jasny2017-10-101-5/+16
| |
* | Revert "Xcode: Adjust tests to drop of 32bit iOS architectures"Gregor Jasny2017-10-105-22/+4
|/ | | | This reverts commit d210b2813072c874ee13fcc941e41aacacf09874.
* Xcode: Adjust tests to drop of 32bit iOS architecturesGregor Jasny2017-10-055-4/+22
|
* Tests: Fix RunCMake.Framework ios arch for Xcode 9Brad King2017-10-031-1/+11
| | | | | Xcode 9 comes with the iPhoneOS 11.0 SDK that does not support the `armv7` architecture. For this SDK version and newer, use `arm64`.
* Tests: Fix RunCMake.Framework expected output on macOS 10.13Brad King2017-10-031-2/+2
| | | | | | The archiver output in the case of universal binaries has changed slightly. Update our expected output to match. While at it, drop unnecessary leading and trailing `.*`.
* FindBoost: Improve messages when a Boost CMake package is foundBrad King2017-09-297-0/+19
| | | | | | Add a test for this case to verify the messages. This test will also be valuable to cover this code path in which we've had several regressions recently.
* Merge topic 'GetPrerequisites-no-clear-on-missing'Brad King2017-09-274-0/+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-264-0/+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 'revert-perf-source-lookup'Brad King2017-09-272-1/+17
|\ \ | |/ |/| | | | | | | | | | | 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
* | Merge topic 'test-fix-msvc-standard-default'Brad King2017-09-251-13/+31
|\ \ | |/ |/| | | | | | | | | a5dd1599 Tests: Fix RunCMake.try_compile CxxStandard case on MSVC Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1302
| * Tests: Fix RunCMake.try_compile CxxStandard case on MSVCBrad King2017-09-221-13/+31
| | | | | | | | | | | | | | | | In post-3.9 development we've taught CMake to understand C++ language standards for MSVC. The RunCMake.try_compile test needs to know whether to expect support or not. Previously we depended on the host CMake version to know this, but CMake 3.9 and below do not. Add special logic to the test to account for this.
* | Merge topic 'cpack-rpm-deb-version'Brad King2017-09-227-4/+40
|\ \ | | | | | | | | | | | | | | | | | | 74b7457f CPack RPM and DEB: improved package version handling Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1296
| * | CPack RPM and DEB: improved package version handlingDomen Vrankar2017-09-217-4/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | Adds support for epoch version to CPackRPM and CPackDeb packagers. Also adds better version checking to CPackDeb that complies with Debian rules. Fixes: #17057
* | | test: Added additional unit test to variable_watchJustin Berger2017-09-212-0/+18
| |/ |/| | | | | | | This tests adding a variable_watch inside the callback to an existing callback
* | Merge topic 'cpack-different-checksum-file-per-generator'Brad King2017-09-211-3/+2
|\ \ | | | | | | | | | | | | | | | | | | b06870e5 CPack: use a distinct checksum file for each generator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1291
| * | CPack: use a distinct checksum file for each generatorDomen Vrankar2017-09-201-3/+2
| |/ | | | | | | | | | | | | | | | | Different CPack generators could produce checksum files with the same name which were overwritten by each other since only package name without extensions was used for checksum file name generation. This patch adds package extension to checksum files to prevent collisions. Fixes: #16840
* | Merge topic 'cpack-check-install-script'Brad King2017-09-213-0/+15
|\ \ | | | | | | | | | | | | | | | | | | 09166339 CPack: Add missing check for CPACK_INSTALL_SCRIPT variable Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1090
| * | CPack: Add missing check for CPACK_INSTALL_SCRIPT variableAlex Turbov2017-09-203-0/+15
| |/ | | | | | | | | | | | | Also add a test case that uses CPACK_INSTALL_SCRIPT. Co-Author: Domen Vrankar <domen.vrankar@gmail.com> Fixes: #15005
* | Merge topic 'test_for_missing_sources'Brad King2017-09-2159-0/+209
|\ \ | |/ |/| | | | | | | | | 55ed21e7 Tests: Add tests for missing sources in add_executable and add_library. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1269
| * Tests: Add tests for missing sources in add_executable and add_library.Deniz Bahadir2017-09-2059-0/+209
| |
* | Merge topic 'refactor-iwyu-code'Brad King2017-09-1517-8/+30
|\ \ | | | | | | | | | | | | | | | | | | 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-1317-8/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 'fphsa-fix-version-exact'Brad King2017-09-151-4/+4
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | 5dc215e0 FPHSA: fix typo in comment a8f70b32 FPHSA: replace left-over VERSION by _FOUND_VERSION. 549366cb FPHSA: Add (failing) unit test when checking EXACT on full version number Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1259
| * | FPHSA: fix typo in commentYves Frederix2017-09-141-2/+2
| | |
| * | FPHSA: Add (failing) unit test when checking EXACT on full version numberYves Frederix2017-09-141-2/+2
| |/ | | | | | | | | This test detects a regression introduced by commit aec27c152e (FPHSA: use more localized _FOUND_VERSION instead of VERSION, 2017-09-05).
* | Tests: Update RunCMake/README.rst to clarify meaning of -result.txtBrad King2017-09-141-2/+2
| | | | | | | | | | | | | | The content of `-result.txt` files is a regex, but the documentation didn't make this clear. Suggested-by: Deniz Bahadir <dbahadir@benocs.com>
* | get_filename_component: Revise PROGRAM/PROGRAM_ARGS split semanticsBrad King2017-09-131-0/+11
|/ | | | | | | | | | | | | | | | | | | | 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_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 '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
* | | 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 branch 'find-package_root-disable' into release-3.9Brad King2017-08-103-0/+3
| |\ \ | | | | | | | | | | | | Merge-request: !1116
* | \ \ 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>
generated by cgit v0.12 at 2025-03-07 08:33:01 (GMT)