summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* cmTargetPropCommandBase: Fix typo in error messageDeniz Bahadir2017-10-262-9/+9
|
* Merge topic 'defer-target-source-check'Brad King2017-10-2518-50/+36
|\ | | | | | | | | | | | | | | 4e7f6738 Defer check for sources within a target until generation. 6e4e7c65 Tests: Exclude bad RunCMake.add_executable case on multi-arch Xcode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1242
| * Defer check for sources within a target until generation.Deniz Bahadir2017-10-2417-49/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `add_library` and `add_executable` commands can now be called with no source-files and won't generate a warning or error message, as long as source-files will be added later via the `target_sources` command. If during the generation step still no sources are associated with targets created by such calls a useful error message will be generated and generation fails. Targets of type `INTERFACE_LIBRARY`, `UTILITY` or `GLOBAL_TARGET` are excluded from this check because we do not need sources for these target types during generation. Fixes: #16872
| * Tests: Exclude bad RunCMake.add_executable case on multi-arch XcodeBrad King2017-10-241-1/+3
| | | | | | | | | | | | | | The `NoSourcesButLinkObjects` case would not be expected to work under Xcode with multiple architectures even if the target objects were listed directly as sources. Exclude it. We already exclude similar cases in `RunCMake.add_library`.
* | VS: Select and save a VS 2017 instance persistentlyBrad King2017-10-199-7/+52
| | | | | | | | | | | | | | | | | | | | Visual Studio 2017 supports multiple instances installed on a single machine. We use the Visual Studio Installer tool to enumerate instances and select one. Once we select an instance for a given build tree, save the result in `CMAKE_GENERATOR_INSTANCE` so we can re-configure the tree with the same instance on future re-runs of CMake. Fixes: #17268
* | Add infrastructure for generators to select a build tool instanceBrad King2017-10-1914-0/+60
|/ | | | | | | | | | | | | | Add cache entry `CMAKE_GENERATOR_INSTANCE` to hold the instance location persistently across re-runs of CMake in a given build tree. For now we reject the option by default if explicitly set. It will be implemented on a per-generator basis. Pass the setting into try_compile project generation. Add a RunCMake.GeneratorInstance test to cover basic use cases for the option. Verify that `CMAKE_GENERATOR_INSTANCE` is empty by default, and that it is rejected when the generator does not support a user setting. Issue: #17268
* Merge topic 'curl_netrc_options'Brad King2017-10-199-0/+72
|\ | | | | | | | | | | | | | | | | | | | | d45aa38a Add dev notes for topic 'curl_netrc_options' 60c272b6 ExternalProject: Add support for NETRC and NETRC_FILE suboption 754e39dd Add testcases for file(DOWNLOAD|UPLOAD) netrc options 5d67e902 file(DOWNLOAD|UPLOAD): Add 'NETRC' and 'NETRC_FILE' suboption Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Ruslan Baratov <ruslan_baratov@yahoo.com> Merge-request: !1376
| * Add testcases for file(DOWNLOAD|UPLOAD) netrc optionsShane Parris2017-10-189-0/+72
| |
* | VS,Xcode: Add CMakeLists.txt sources without mutating targetsBrad King2017-10-186-53/+1
| | | | | | | | | | | | | | | | | | | | | | | | Rather than injecting `CMakeLists.txt` files into each target's `SOURCES`, teach the generators to add them during generation using dedicated code. This avoids mutating the original targets, and avoids polluting `$<TARGET_PROPERTY:foo,SOURCES>` with generator-specific content. This also avoids listing the `CMakeLists.txt` sources in the results of `CMAKE_DEBUG_TARGET_PROPERTIES==SOURCES` so the `RunCMake.TargetSources` test no longer needs a separate case for IDEs.
* | Merge topic 'module-fetchcontent'Brad King2017-10-1818-0/+318
|\ \ | | | | | | | | | | | | | | | | | | | | | 1e56634f FetchContent: Add tests 60e74d2f FetchContent: New module for populating content at configure time Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1306
| * | FetchContent: Add testsCraig Scott2017-10-1618-0/+318
| |/
* | Merge topic 'imported-interface-no-system'Brad King2017-10-161-1/+0
|\ \ | | | | | | | | | | | | | | | | | | | | | 22beb07f Help: Clarify documentation of NO_SYSTEM_FROM_IMPORTED 2de0e0fd Do not initialize NO_SYSTEM_FROM_IMPORTED on INTERFACE libraries Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1386
| * | Do not initialize NO_SYSTEM_FROM_IMPORTED on INTERFACE librariesBrad King2017-10-131-1/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | The change in commit v3.8.0-rc1~276^2 (Allow NO_SYSTEM_FROM_IMPORTED on imported INTERFACE libraries, 2016-11-21) was incorrect. The property is not meant to be set on imported targets at all. It is meant to be set on their consumers that compile sources. Since INTERFACE libraries have no sources to compile, the property is not needed on them. Revert most of that change. Unfortunately we must still tolerate project code setting NO_SYSTEM_FROM_IMPORTED on INTERFACE libraries because they were allowed by CMake 3.8 and 3.9. Issue: #17348
* | 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
* | Merge topic 'UseJava_OUTPUT_DIR'Brad King2017-10-112-0/+15
|\ \ | | | | | | | | | | | | | | | | | | dead049e UseJava: OUTPUT_DIR must exists in all cases. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1368
| * | UseJava: OUTPUT_DIR must exists in all cases.Marc Chevrier2017-10-102-0/+15
| |/ | | | | | | Fixes: #17316
* | 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.
* | Merge topic 'xcode-tests-ios-deployment-target'Brad King2017-10-105-27/+20
|\ \ | |/ |/| | | | | | | | | | | 3a49b4eb Xcode 9: Lower iOS deployment version to get armv7 builds 78a560e4 Revert "Xcode: Adjust tests to drop of 32bit iOS architectures" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1365
| * 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.
* | Merge topic 'xcode9-ios-tests'Brad King2017-10-055-4/+22
|\ \ | |/ | | | | | | | | | | d210b281 Xcode: Adjust tests to drop of 32bit iOS architectures Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1347
| * Xcode: Adjust tests to drop of 32bit iOS architecturesGregor Jasny2017-10-055-4/+22
| |
* | Tests: Remove ancient workaround in LoadCommand testsBrad King2017-10-042-14/+0
|/
* Merge topic 'test-macos-updates'Brad King2017-10-042-3/+13
|\ | | | | | | | | | | | | | | 58d9297e Tests: Fix RunCMake.Framework ios arch for Xcode 9 b8dd7a70 Tests: Fix RunCMake.Framework expected output on macOS 10.13 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1338
| * 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 `.*`.
* | Merge topic 'src-flags-genex-target'Brad King2017-10-043-4/+15
|\ \ | |/ |/| | | | | | | | | | | 303cd703 VS,Xcode: Fix TARGET_PROPERTY genex in source COMPILE_FLAGS property Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Nikita Nemkin <nikita@nemkin.ru> Merge-request: !1336
| * VS,Xcode: Fix TARGET_PROPERTY genex in source COMPILE_FLAGS propertyBrad King2017-10-033-4/+15
| | | | | | | | | | | | This already worked in other generators. Also add a test case. Fixes: #17314
* | Merge topic 'FindProtobuf-gen-desc'Brad King2017-10-035-1/+98
|\ \ | |/ |/| | | | | | | | | | | 1299f4cc FindProtobuf: add flag to allow descriptor files to be generated 4e91be95 FindProtobuf: Refactor custom command output listing Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1301
| * FindProtobuf: add flag to allow descriptor files to be generatedPeter Mitrano2017-10-025-1/+98
| | | | | | | | | | | | | | | | | | | | | | - The .desc files will be in the same folder as the generated .cc and .h files. - Paths to generate .desc files are stored in a variable passed in - This is only implemented for C++ - Remove legacy ARGS - Add test that generates and uses C++ protobuf message - Add test that checks that the generated .desc file can be instantiated with DynamicMessageFactory - Add Help rst for new feature
* | Merge topic 'FindBoost-cmake-package'Brad King2017-10-027-0/+19
|\ \ | | | | | | | | | | | | | | | | | | c044b9ae FindBoost: Improve messages when a Boost CMake package is found Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1330
| * | 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.
* | | Tests: Fix CudaOnly.WithDefs test on CUDA 9Brad King2017-09-291-1/+5
|/ / | | | | | | CUDA 9 dropped support for `compute_20`, so use a different alternative.
* | Merge topic 'add-vs-shader-properties'Brad King2017-09-291-2/+4
|\ \ | | | | | | | | | | | | | | | | | | 49dab3eb VS: Add VS_SHADER_OUTPUT_HEADER_FILE and VS_SHADER_VARIABLE_NAME properties. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1319
| * | VS: Add VS_SHADER_OUTPUT_HEADER_FILE and VS_SHADER_VARIABLE_NAME properties.Kevin M. Godby2017-09-271-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The VS_SHADER_OUTPUT_HEADER_FILE property is the name of the generated header file containing the object code of the shader. The VS_SHADER_VARIABLE_NAME property is the name of the variable containing the object code in the above header file. Signed-off-by: Kevin M. Godby <kevin@godby.org>
* | | Autogen: Tests: Update AUTOMOC_MACRO_NAMES testSebastian Holtermann2017-09-2815-49/+139
| | |
* | | 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 'ninja-rc-depfile-quoting'Brad King2017-09-271-0/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 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-251-0/+7
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-272-1/+64
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 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-252-1/+64
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-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 'autogen-generators-refactor'Brad King2017-09-2246-274/+572
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 51fd7b71 Autogen: Tests: Add a change-not test to the mocRerun test c8f92db7 Autogen: Tests: Disable an AUTOMOC_DEPENDS_FILTER test for Ninja 3c77515e Autogen: Tests: Refactor the QtAutogenRebuild rccDepends test 04a0daee Autogen: Tests: Move each rerun test script to a NAME.cmake file e5c6610a Autogen: Tests: Extend CMAKE_AUTOMOC_DEPEND_FILTERS rebuild test 93265652 Autogen: Tests: Add <SUBDIR>/ui_view.h AUTOUIC includes to sameName test 4eb7d817 Autogen: Tests: Add <SUBDIR>/item.moc includes to sameName test 3f223743 Autogen: Read relative paths from rcc output ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1244
| * | Autogen: Tests: Add a change-not test to the mocRerun testSebastian Holtermann2017-09-212-17/+36
| | |