summaryrefslogtreecommitdiff
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* bindexplib: Revert support for constants symbolsBrad King2017-07-102-12/+0
| | | | | | | | | | Revert the main logic change of commit v3.9.0-rc1~192^2 (bindexplib: fix constants symbols export, 2017-04-26) and its test case. Unfortunately some constants may be provided by multiple object files with different `@...` suffixes, leading to ambiguous references. Revert support pending further investigation. Fixes: #17045
* Merge branch 'fix-crash-on-non-enabled-language-features' into release-3.9Brad King2017-06-2726-12/+30
|\
| * target_compile_features: Do not crash on non-enabled languageBrad King2017-06-264-0/+8
| | | | | | | | Fixes: #17011
| * Tests: Enable languages explicitly in RunCMake.target_compile_featuresBrad King2017-06-2622-12/+22
| | | | | | | | | | Enable C or CXX (or nothing) as needed in each test case. This will allow us to add test cases that do not enable CXX.
* | Merge branch 'vs-cuda-fix-flags' into release-3.9Brad King2017-06-271-0/+5
|\ \ | |/ |/|
| * VS: Fix support for nvcc flags not in our flag tableBrad King2017-06-271-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | The change in commit v3.9.0-rc4~3^2 (VS: Improve workaround for CUDA -Xcompiler placement bug, 2017-06-21) accidentally appended to the `AdditionalOptions` as if it were a `;`-separated list, but it is actually a command-line string. Append with a space instead. While at it, fix the same problem for the `AdditionalOptions` added to `CudaLink` by commit v3.9.0-rc3~1^2 (CUDA: When linking device code suppress CUDA 8.0+ deprecation warnings, 2017-06-09). Fixes: #17008
* | Merge branch 'vs-rc-flags' into release-3.9Brad King2017-06-221-0/+4
|\ \
| * | VS: Fix support for rc /nologo flag in per-source COMPILE_FLAGSBrad King2017-06-221-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.9.0-rc1~160^2 (VS: Use tool-specific flag table for COMPILE_FLAGS parsing, 2017-05-03) we now correctly use the `rc` flag table to process the COMPILE_FLAGS flags of `.rc` source files instead of incorrectly using the `cl` flag table as before. However, our `rc` flag table is not complete. The `/nologo` flag was working before only by accident because the `cl` flag table entry for it happened to match. Add the proper entry to the `rc` flag table. Fixes: #16991
* | | Merge branch 'vs-cuda-fix-flags' into release-3.9Brad King2017-06-213-0/+9
|\ \ \ | | |/ | |/|
| * | VS: Improve workaround for CUDA -Xcompiler placement bugBrad King2017-06-211-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit v3.9.0-rc1~431^2~6 (VS: Place CUDA host compiler options in proper project file fields, 2017-03-07) we worked around a bug in the CUDA VS integration by dropping `AdditionalCompilerOptions`. However, this silently drops `-Xcompiler=` options given by the user that don't map to one of CudaCompile's dedicated settings. Improve the workaround to instead put the remaining `AdditionalCompilerOptions` into the `AdditionalOptions` field behind `-Xcompiler=` ourselves.
| * | VS: Fix target_compile_options for CUDABrad King2017-06-212-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the VS generator to honor `COMPILE_OPTIONS` for CUDA. The exclusion added by commit v3.9.0-rc1~431^2~7 (VS: Do not pass CUDA compile options to C compiler, 2017-03-07) was correct but we need additional logic to pass the CUDA compile options to the CUDA compiler. Also we should still pass the CXX or C options to MSVC (ClCompile) when those languages are enabled even if the link language is CUDA.
* | | Vs: allow CSharp targets to be linked to CXX targetsMichael Stürmer2017-06-215-0/+54
|/ / | | | | | | Fixes: #16755
* | IPO: Consider support for each language separatelyBrad King2017-06-149-13/+13
| | | | | | | | | | | | | | | | | | We only define `INTERPROCEDURAL_OPTIMIZATION` behavior for C, CXX, and Fortran languages. Do not try to enable support for other languages. Furthermore, each language builds with a different compiler, so check for support by CMake and the compiler for each language independently. Fixes: #16944
* | Merge topic 'GoogleTest-disabled-tests'Brad King2017-06-052-1/+29
|\ \ | | | | | | | | | | | | | | | | | | | | | 92bbb706 GoogleTest: Add support for disabled tests Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Craig Scott <craig.scott@crascit.com> Merge-request: !920
| * | GoogleTest: Add support for disabled testsChuck Atkins2017-06-052-1/+29
| | | | | | | | | | | | Fixes: #10612
* | | Merge topic 'test-GNUInstallDirs-FreeBSD'Brad King2017-06-057-8/+136
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 913736f1 Tests: Fix RunCMake.GNUInstallDirs on BSD platforms Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !921
| * | | Tests: Fix RunCMake.GNUInstallDirs on BSD platformsBrad King2017-06-027-8/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The `GNUInstallDirs` module has different defaults on BSD platforms. Update the test expected output to account for the difference. Fixes: #16887
* | | | Merge topic 'ninja-mingw'Brad King2017-06-011-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 82badfff Ninja: Fix CMP0058 on MinGW c5ff50fc Tests: Fix CustomCommandByproducts regex for phony rules Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !919
| * | | | Tests: Fix CustomCommandByproducts regex for phony rulesBrad King2017-05-311-1/+1
| |/ / / | | | | | | | | | | | | Update the regex to match phony rules that have no inputs.
* | | | Merge topic 'add-common-record-features-macros'Brad King2017-06-017-44/+87
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3c1ecb52 Intel: Fix missing C std default for 12.0 <= ver < 12.1 f70b0bb3 SunPro: Make sure all known versions get CXX98 defaults 220ede74 GNU: Fix language defaults for 3.4 a40e6ba8 Clang: Fix language defaults for 2.1 9b112a84 Compilers: Port to use default cmake_record_lang_compile_features macros 37221529 MSVC: Add empty definitions for std compile options e556f1b9 CompileFeatures: Makes tests work with meta-feature only 20ffa147 Tests: Allow test macro to take no executable arguments ... Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !833
| * | | | CompileFeatures: Makes tests work with meta-feature onlyChuck Atkins2017-05-297-42/+81
| | | | |
| * | | | Tests: Allow test macro to take no executable argumentsChuck Atkins2017-05-291-2/+6
| | | | |
* | | | | Tests: Skip BootstrapTest if testing external CMakeBrad King2017-05-311-0/+3
| |/ / / |/| | | | | | | | | | | | | | | If we are not actually building CMake then we should not run the `BootstrapTest` (which builds CMake).
* | | | Merge topic 'ninja-mingw'Brad King2017-05-311-3/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6a2f8335 Ninja: Fix escaping of path to depfile 781eb380 Tests: Fix VSResource test on MinGW with Ninja and a space in the path Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !911
| * | | | Tests: Fix VSResource test on MinGW with Ninja and a space in the pathBrad King2017-05-301-3/+2
| |/ / /
* | | | Merge topic 'fix-XCTest-build-config'Brad King2017-05-311-0/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | be55f069 Tests: Fix XCTest build configuration Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !907
| * | | | Tests: Fix XCTest build configurationBrad King2017-05-301-0/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | Build the test binaries with the same configuration in which they will be tested instead of the default Debug configuration. This fixes the test when run in a Release configuration.
* | | | Tests: Fix Qt5Autogen test on XcodeBrad King2017-05-302-4/+5
|/ / / | | | | | | | | | | | | | | | | | | Fix the `mocPlugin` project to provide `mocPlugin.xcodeproj` instead of `Project.xcodeproj` so that the `try_compile` project name matches. Also search in the `Debug` subdirectory of the `try_compile` build tree for the resulting binaries, to work with multi-config generators.
* | | Merge topic 'sunpro-std-flags'Brad King2017-05-281-3/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 370d0d25 SunPro: update flags used for CMake itself f1f21e30 SunPro: set -library=stlport as standard compile option for C++98 9316120c SunPro: add standard compile option for C++03 783fbb77 Tests: Compile entire Plugin test with the same language standard Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !879
| * | | Tests: Compile entire Plugin test with the same language standardBrad King2017-05-261-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Set `CMAKE_CXX_STANDARD` early so that both KWSys and our test code build with the same language standard. This is important on compilers that have incompatible standard libraries.
* | | | Autogen: Test adaptionsSebastian Holtermann2017-05-261-2/+8
| | | |
* | | | Autogen: Per-config file suffixes. New AUTOGEN_BUILD_DIR target property.Sebastian Holtermann2017-05-261-2/+2
| | | | | | | | | | | | | | | | | | | | Closes #14760 Closes #14313
* | | | Merge topic 'FindDoxygen-add-docs-function'Brad King2017-05-256-0/+109
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 59ffabfe Improve Doxygen support Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !684
| * | | | Improve Doxygen supportAlex Turbov2017-05-246-0/+109
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Except Graphviz's `dot` Doxygen may use few other utilities like `mscgen` (Message Sequence Chart) and `dia` (Diagram Editor). Now this module allows to manage Doxygen settings from `CMakeLists.txt` and forget about `Doxyfile`s. Also it provides a helper function to add a target to generate documentation: `doxygen_add_docs`. Implement code review notes: - Introduce `COMPONENTS` to find: `dot`, `mscgen` and `dia`; - Deprecate variables `DOXYGEN_SKIP_DOT`, `DOXYGEN_EXECUTABLE`, `DOXYGEN_DOT_EXECUTABLE`, `DOXYGEN_DOT_FOUND` in favour of `doxygen_add_docs ` usage instead; - Properly handle paths to found tools in Windows; - Prevent adding a custom target if Doxygen was not really found; - Introduce exported (executable) targets for found components. Co-Author: Craig Scott <craig.scott@crascit.com>
* | | | | Merge topic 'find_dependency-improvements'Brad King2017-05-2528-50/+71
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ab358d6a Improve find_dependency argument handling Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !872
| * | | | | Improve find_dependency argument handlingMatthew Woehlke2017-05-2228-50/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove highly specialized and totally positional argument handling in find_dependency macro, and instead just pass arguments through to find_package. This gives users access to the full suite of arguments that find_package knows, and is backward compatible with the old arguments. Also, rewrite the unit tests for this, since the old tests are exclusively focused on testing the old argument handling and are no longer applicable, and add some success tests (the old tests did not even set up the CMake state in a way that CMake had any hope of ever finding the test package).
* | | | | | Merge topic 'find_package-shorter-message'Brad King2017-05-259-47/+5
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d0b9d1cc find_package: shorten output for missing package in config mode d9df805f cmFindPackageCommand: Split condition to improve readability Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !851
| * | | | | find_package: shorten output for missing package in config modeChristoph Grüninger2017-05-239-47/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If CONFIG or MODULE is given and the package is not REQUIRED, output a one-liner instead of the full warning. Update Tests/RunCMake/find_package expected output to match.
* | | | | | FindProtobuf: Rename imported targets to match upstream namesBrad King2017-05-221-3/+3
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename our recently added imported targets to match those provided by the upstream's CMake-based build. That way a project using `find_package(Protobuf)` can get the same target names no matter how protobuf is found. Suggested-by: Konstantin Podsvirov <konstantin@podsvirov.pro>
* | | | | Merge topic 'add-findpackage-root-prefix'Brad King2017-05-2217-0/+476
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fe8f08d2 find_*: Add docs for PackageRoot search path group 57744ca9 find_*: Add tests for PackageRoot search path group ef3d360a find_*: Add a new PackageRoot search path group Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !796
| * | | | | find_*: Add tests for PackageRoot search path groupChuck Atkins2017-05-1617-0/+476
| | | | | |
* | | | | | Merge topic 'pgi-fix-windows'Brad King2017-05-221-3/+48
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4eb15824 Windows-PGI: Add platform definitions a94ae96e Windows-PGI: Adapt default compiler flags c2c2d366 ImplicitLinkInfo: Add support for PGI on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !794
| * | | | | | ImplicitLinkInfo: Add support for PGI on WindowsChristian Pfeiffer2017-05-201-3/+48
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since PGI does not write linker directives into objects, the necessary libraries have to be parsed from commandline. PGI does however link the Visual C++ runtime libraries, so they have to be filtered out to ensure no collision with settings of other languages can occur.
* | | | | | Merge topic 'target-depends-per-config'Brad King2017-05-192-2/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 87a37e64 cmComputeTargetDepends: Avoid nested loops over configurations 5a913794 cmComputeTargetDepends: Avoid computing with empty configuration 70c65572 Tests: Fix RunCMake.CMP0022 tll case for Debug configuration Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !865
| * | | | | | Tests: Fix RunCMake.CMP0022 tll case for Debug configurationBrad King2017-05-182-2/+4
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `CMP0022-WARN-tll` case overrides legacy properties set by `target_link_libraries` so that we can verify that the policy warning is emitted. When building in the Debug configuration, the `_DEBUG` variant of the legacy property is set/checked too. Fix the test case to override both variants. Previously it only passed because `cmComputeTargetDepends::AddInterfaceDepends` always evaluated dependencies with the "" configuration.
* | | | | | Merge topic 'solarisEmptyFileFix'Brad King2017-05-192-2/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1bfe3720 Tests: Ensure Solaris linker never sees empty contents Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !860
| * | | | | | Tests: Ensure Solaris linker never sees empty contentsCraig Scott2017-05-182-2/+6
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following tests on Solaris builds: wrapping, qtwrapping and CMakeCommands.target_link_libraries
* | | | | | Merge topic 'solaris_testLibSONAME'Brad King2017-05-191-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 94cd4505 Tests: Make function name not match SONAME Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !868
| * | | | | | Tests: Make function name not match SONAMECraig Scott2017-05-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We look for `SONAME` in the output of `readelf` to check whether the binary contains the field. Do not provide a symbol that may accidentally match. Fixes: #16894
* | | | | | | Merge topic 'FindProtobuf-targets'Brad King2017-05-195-0/+62
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 37627217 Help: Add notes for topic 'FindProtobuf-targets' e4e1d194 FindProtobuf: add tests f29635b6 FindProtobuf: add targets Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !861
View Lorry Controller Status