summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'fix-custom-target-with-csharp'Brad King2018-11-014-2/+15
|\ | | | | | | | | | | | | | | | | 9040df31e2 Merge branch 'backport-fix-custom-target-with-csharp' 1acd1c2b50 CSharp: Fix regression in VS project type selection for custom target a56edad6d6 CSharp: Fix regression in VS project type selection for custom target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2549
| * CSharp: Fix regression in VS project type selection for custom targetBrad King2018-10-314-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A target created by `add_custom_target` should always be a `.vcxproj` file even if it has `.cs` sources involved in custom commands and such. The latter case was broken by refactoring in commit v3.12.0-rc1~160^2~7 (remove TargetIsCSharpOnly() and use methods from cmGeneratorTarget, 2018-03-19). The reason is that the `HasLanguage` method added by commit v3.12.0-rc1~239^2~6 (cmGeneratorTarget: add HasLanguage() as wrapper for GetLanguages(), 2018-03-19) does not check the target type and so is not a suitable check for deciding the project file extension. The `HasLanguage` method was an attempt at an abstraction that turns out not to work very well. Replace it with a dedicated `IsCSharpOnly` method that considers the target type, sources, and non-transitive `LINKER_LANGUAGE`. Fixes: #18515
| * Merge branch 'vs-csharp-in-custom-target' into release-3.13Brad King2018-10-233-0/+3
| |\ | | | | | | | | | Merge-request: !2515
* | \ Merge topic 'set_directory_properties-script-mode'Brad King2018-10-303-0/+5
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 50572d638b set_directory_properties: Restore in script mode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2544
| * | | set_directory_properties: Restore in script modeBrad King2018-10-293-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.10.0-rc1~391^2~3 (Add directory property 'LABELS' and CMAKE_DIRECTORY_LABELS variable, 2017-06-23) this command was accidentally not allowed in script mode. It was dropped because `ctest -S` mode needs to start with CMake's normal script mode and then replace the `set_directory_properties` implementation. Restore the normal `set_directory_properties` in script mode and then add special logic to replace it in ctest. Also add a test case. Fixes: #18523
* | | | Merge topic 'FindMatlab-2018a-API'Brad King2018-10-309-9/+197
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 42731e94be FindMatlab: Fix linker command under Windows. ee73e733e4 FindMatlab: Fix compilation error in one specific test configuration. 160499296c FindMatlab: added unit tests for new functionality. ee7e97a7d3 FindMatlab: add change log item. d7e19032d5 FindMatlab: `matlab_add_mex` has new options `R2017b` and `R2018a`. 518553d953 FindMatlab: Always find core libraries. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2508
| * | | | FindMatlab: Fix compilation error in one specific test configuration.Cris Luengo2018-10-291-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `matlab_wrapper2.cpp` failed to compile in one specific test configuration. I've copied an odd workaround from `matlab_wrapper1.cpp`, which doesn't fail to compile. I presume this workaround was meant to address this issue, but can't verify this locally.
| * | | | FindMatlab: added unit tests for new functionality.Cris Luengo2018-10-249-9/+191
| | | | | | | | | | | | | | | | | | | | Also allowing a way to select which of multiple installed MATLAB versions to use in the test.
* | | | | Merge topic 'ExternalProject-log-patch'Brad King2018-10-291-0/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a4e9391953 ExternalProject: add LOG_PATCH option to log the patch command Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2534
| * | | | | ExternalProject: add LOG_PATCH option to log the patch commandWouter Klouwen2018-10-261-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most steps support the logging into a file but the patch command is a notable exception. This commit adds the LOG_PATCH options that acts as the other LOG_* options.
* | | | | | Merge topic 'minusCparse'Brad King2018-10-2912-1/+22
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 379e5f93a9 Tests: Add cases for -{C,D,U} without a source tree 5873815fef cmake: distinguish '-Cpath' from '-C path' in source dir parsing Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2517
| * | | | | Tests: Add cases for -{C,D,U} without a source treeBrad King2018-10-2612-1/+22
| | | | | |
* | | | | | RPATH: Add option for using $ORIGIN in build treePeter Wu2018-10-264-1/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes binaries independent of the build directory by not embedding the build directory via RPATH. The tests are partially based on the existing RuntimePath test, but with the check moved into a POST_BUILD command such that it can be skipped when the platform lacks support. Fixes: #18413
* | | | | | Merge topic 'cuda-thread-flags'Brad King2018-10-257-16/+73
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2cc050b53b CUDA: Add test for device linking when host linking uses threads 83c13ca44f FindThreads: Pass -pthread to CUDA compiler through -Xcompiler cf92fd9ae9 Merge branch 'cuda-filter-device-link-items' into cuda-thread-flags e768d96c74 CUDA: Filter out host link flags during device linking Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Kelly (KT) Thompson <kgt@lanl.gov> Merge-request: !2512
| * | | | | | CUDA: Add test for device linking when host linking uses threadsRobert Maynard2018-10-247-16/+73
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Convert the `CudaOnly.LinkSystemDeviceLibraries` test to a new `Cuda.ProperDeviceLibraries` test. The former covered only the `cublas_device` library which is removed by CUDA 10. Extend the new test to also cover various cases of using threads. Issue: #18008
| * | | | | Merge branch 'GNUInstallDirs-kfreebsd-man' into release-3.13Brad King2018-10-221-1/+1
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Merge-request: !2511
* | \ \ \ \ \ Merge topic 'vs-csharp-in-custom-target'Brad King2018-10-243-0/+3
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | / | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | d004d8c59a VS: Fix crash on CSharp sources in a custom target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2515
| * | | | | VS: Fix crash on CSharp sources in a custom targetBrad King2018-10-233-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The target generator does not compute ClOptions for custom targets, so we should not use them either. Fixes: #18377, #18485
* | | | | | Merge topic 'GNUInstallDirs-kfreebsd-man'Brad King2018-10-231-1/+1
|\ \ \ \ \ \ | | |_|/ / / | |/| | / / | |_|_|/ / |/| | | | | | | | | | | | | | b2d7ab8bd1 GNUInstallDirs: Don't use BSD info and man paths on GNU/kFreeBSD Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2511
| * | | | GNUInstallDirs: Don't use BSD info and man paths on GNU/kFreeBSDJames Clarke2018-10-221-1/+1
| |/ / /
* | | | ExternalProject: option LOG_MERGED_STDOUTERR to combine stdout and stderrWouter Klouwen2018-10-191-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some circumstances the user of ExternalProject may not desire the split log files for stdout and stderr. In particular with a project has many errors it can be difficult to correlate the output error with the command that it resulted from. This commit adds the LOG_MERGED_STDOUTERR option that when enabled outputs into a unified <name>-<step>.log for each step. If disabled it will default to the previous behaviour of <name>-<step>-out.log and <name>-<step>-err.log.
* | | | ExternalProject: add LOG_DIR option that allows overriding of log locationWouter Klouwen2018-10-191-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some situations it can be helpful to separate out the location of the log files from the location of the stamp files. For instance if you have a continuous integration that exposes the location where log files are stored. This commit adds an option that allows a user to override the default behaviour of putting the log files in STAMP_DIR called LOG_DIR. The previous behaviour of putting the log files into the STAMP_DIR applies if LOG_DIR is not specified.
* | | | Merge topic 'deployqt4-cmp0080-fix'Brad King2018-10-191-0/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 31b6825bd2 Merge branch 'master' into deployqt4-cmp0080-fix 15bbff0581 DeployQt4: Do not include BundleUtilities at configure time 65bea5b90b DeployQt4: Convert to 2-space indentation 9bfb63bd42 Help: Convert DeployQt4 to block-style comment Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2494
| * \ \ \ Merge branch 'master' into deployqt4-cmp0080-fixKyle Edwards2018-10-19170-222/+422
| |\ \ \ \ | | |_|/ / | |/| | |
| * | | | Merge branch 'document-dollar-in-varnames' into release-3.13Brad King2018-10-166-0/+41
| |\ \ \ \ | | | | | | | | | | | | | | | | | | Merge-request: !2479
| * \ \ \ \ Merge branch 'backport-revert-install-code-script-genex' into release-3.13Brad King2018-10-116-19/+0
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Merge-request: !2474
| * \ \ \ \ \ Merge branch 'vs-cuda-pdb' into release-3.13Brad King2018-10-1014-14/+14
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Merge-request: !2473
| * \ \ \ \ \ \ Merge branch 'project-always-set-desc-url' into release-3.13Brad King2018-10-082-0/+27
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge-request: !2457
| * \ \ \ \ \ \ \ Merge branch 'pkgc-op-lt-gt' into release-3.13Brad King2018-10-042-0/+84
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge-request: !2435
* | \ \ \ \ \ \ \ \ Merge topic 'find_fontconfig'Brad King2018-10-194-0/+47
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 84e7920b3a FindFontconfig: Add module to find Fontconfig Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2487
| * | | | | | | | | FindFontconfig: Add module to find FontconfigFrederik Gladhorn2018-10-184-0/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This module is inspired by one from KDE's KWin.
* | | | | | | | | | Merge topic 'better-empty-list-behavior'Brad King2018-10-1721-26/+38
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 121a036f73 cmListCommand: handle empty lists for list(REMOVE_AT) acfe53c588 cmListCommand: make list(ACTION not_a_list) succeed when idempotent bf572ac952 cmListCommand: check list(FILTER) operation before the list Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2478
| * | | | | | | | | | cmListCommand: handle empty lists for list(REMOVE_AT)Ben Boeckel2018-10-167-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Treat an empty list as a list with no valid bounds and return an error message indicating that any given indices are out-of-bounds.
| * | | | | | | | | | cmListCommand: make list(ACTION not_a_list) succeed when idempotentBen Boeckel2018-10-1615-25/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The operations changed here all are no-ops on empty lists anyways, so just have them succeed when given non-extant lists. - `list(REMOVE_ITEM)` - `list(REMOVE_DUPLICATES)` - `list(SORT)` - `list(FILTER)` - `list(REVERSE)`
* | | | | | | | | | | Merge topic 'ctest-coverage-gtm-percent'Brad King2018-10-174-32/+41
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f7d92deff4 CTest: Fix GTM coverage handling of entry point named "%" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2489
| * | | | | | | | | | | CTest: Fix GTM coverage handling of entry point named "%"Joseph Snyder2018-10-164-32/+41
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removing the "%" character from the name of the routine in the line parser causes CTest to be unable to find a routine entry point that is only named "%". Instead leave it during line parsing and handle routine names ending in "%" explicitly when loading files.
* | | | | | | | | | | Merge topic 'document-dollar-in-varnames'Brad King2018-10-166-0/+41
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | / / / | | |_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 82a4822610 CMP0053: document that `$` is a valid literal variable character Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2479
| * | | | | | | | | CMP0053: document that `$` is a valid literal variable characterBen Boeckel2018-10-156-0/+41
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was overlooked in the initial implementation of CMP0053. However, an additional policy to reject it again is not worth it. Instead, add tests and document the behavior. Fixes: #17883
* | | | | | | | | Merge topic 'cppcheck-exit-code'Brad King2018-10-126-9/+18
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3b80cd77fb Fail the build if cppcheck returns a non-zero exit code Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2459
| * | | | | | | | | Fail the build if cppcheck returns a non-zero exit codeHarald Brinkmann2018-10-116-9/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows the build failure to be tuned with cppcheck's options --error-exitcode=<n> and --exitcode-suppressions=<file>.
* | | | | | | | | | Merge topic 'find_libinput'Brad King2018-10-124-0/+41
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f76047f34a FindLibinput: Add module to find libinput Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2420
| * | | | | | | | | | FindLibinput: Add module to find libinputFrederik Gladhorn2018-10-114-0/+41
| | |_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This module is inspired by one from KDE's KWin.
* | | | | | | | | | Merge branch 'backport-revert-install-code-script-genex' into ↵Brad King2018-10-116-19/+0
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | / / | | |_|_|_|_|_|/ / | |/| | | | | | | revert-install-code-script-genex
| * | | | | | | | install: Revert CODE,SCRIPT support for generator expressionsBrad King2018-10-116-19/+0
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert commit v3.13.0-rc1~441^2 (install: Teach CODE,SCRIPT modes to evaluate generator expressions, 2018-05-29). Unfortunately it has been found to break existing code in a real project, e.g. install(CODE [[ message("$<FOOBAR>") ]]) Address this regression by reverting support for the 3.13 release series. Support can be restored later with a policy for compatibility. Issue: #15785 Fixes: #18435
* | | | | | | | Merge topic 'vs-cuda-pdb'Brad King2018-10-1114-14/+14
|\ \ \ \ \ \ \ \ | | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | faf3d7d224 VS: Add workaround for CUDA compiler PDB location with space 592064e026 VS: Drop workaround for CUDA compiler PDB location on CUDA 9.2+ fb378fc4d7 Tests: Fix Cuda test project names Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2473
| * | | | | | | Tests: Fix Cuda test project namesBrad King2018-10-1014-14/+14
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make them match what `ADD_TEST_MACRO` gives to ctest as the project name to build so that the `.sln` file will be found.
* | | | | | | Merge topic 'cmake_policy-get_warning'Brad King2018-10-112-10/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0d988f98e5 cmake_policy: Add undocumented GET_WARNING command f9f96598df Help: Convert FindOpenGL documentation to block comment Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2472
| * | | | | | | cmake_policy: Add undocumented GET_WARNING commandKyle Edwards2018-10-102-10/+16
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This command is intended for modules that issue policy warnings so they can get the warning string from CMake in a uniform manner, rather than duplicating the string. Several modules been updated to include an example of the usage of this new command.
* | | | | | | Merge topic 'install-subdirectory-order'Brad King2018-10-11131-145/+262
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b56f2db87a Testing: Add test for CMP0082 fc8955e889 add_subdirectory: Run subdirectory install rules in correct order 514f0b572e Testing: Update hard-coded line numbers to [0-9]+ in some tests Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2434
| * | | | | | | Testing: Add test for CMP0082Kyle Edwards2018-10-1035-2/+119
| | | | | | | |