summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* VS: Properly quote arguments in nasm.xmlDavid Benjamin2018-07-024-3/+27
| | | | | | | | | | | | | | | | | | Most arguments were quoted, but some weren't, causing problems if the arguments contained whitespace. In particular, the _STL_EXTRA_DISABLED_WARNINGS value takes spaces and CMake's NASM support applies all add_definitions lines to NASM. The -D flag is missing quotes, so projects using NASM and setting _STL_EXTRA_DISABLED_WARNINGS break in the Visual Studio generator. Likewise, the -o flag is missing quotes, which means filenames with spaces do not work. (The -U flag is unlikely to need quotes, but include them for consistency.) Extend the existing VSNASM test to cover these cases.
* Merge topic 'ctest_submit-https'Brad King2018-06-292-6/+9
|\ | | | | | | | | | | | | | | 71f27c04b0 ctest_submit: show headers as debug info 4464ef21da ctest_submit: Check CA file during CDASH_UPLOAD Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2171
| * ctest_submit: show headers as debug infoZack Galbreath2018-06-272-6/+9
| | | | | | | | | | | | | | Hide any headers specified by `ctest_submit(HTTPHEADER ...)` unless ctest is run with --debug. This option is used to present bearer tokens to CDash. We do not want to inadvertently expose this secret data in the console output log.
* | Merge topic 'lexer-null'Brad King2018-06-294-0/+7
|\ \ | | | | | | | | | | | | | | | | | | b29842a818 ListFileLexer: Do not match null bytes in input Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2174
| * | ListFileLexer: Do not match null bytes in inputBrad King2018-06-264-0/+7
| |/ | | | | | | | | | | | | | | | | | | | | Extend the fix from commit v3.10.0-rc1~188^2 (ListFileLexer: fix heap-buffer-overflow on malicious input, 2017-08-26) to apply to all lexer token matches. Replace all `.` with `[^\0\n]`. Update all `[^...]` match expressions to not match `\0`. We cannot safely process null bytes in strings. Fixes: #18124
* | Merge topic 'test-wizard'Brad King2018-06-254-4/+3
|\ \ | | | | | | | | | | | | | | | | | | 7bffd71c3b Tests: Adopt wizard test in RunCMake.CommandLine Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2166
| * | Tests: Adopt wizard test in RunCMake.CommandLineBrad King2018-06-224-4/+3
| | | | | | | | | | | | Avoid a whole separate test for one removed command-line option.
* | | Merge topic 'file-GLOB-remove-new-errors'Brad King2018-06-256-11/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 142a625729 file: Drop error cases added by CMake 3.12.0-rc1 to avoid regressions Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Shane Parris <shane.lee.parris@gmail.com> Merge-request: !2164
| * | file: Drop error cases added by CMake 3.12.0-rc1 to avoid regressionsBrad King2018-06-226-11/+1
| |/ | | | | | | | | | | | | | | | | Refactoring in commit v3.12.0-rc1~418^2~3 (Refactor HandleGlobCommand, 2018-02-13) introduced error diagnostics for argument combinations that were previously accepted. Restore acceptance to avoid regressing projects that used those combinations even if they do not make sense. Fixes: #18097
* | Merge topic 'cpack-generator-documentation'Brad King2018-06-222-0/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 57e4c51e8a Help: Update old version release notes for CPack generator docs 4181830881 Help: Add release notes for CPack generator documentation and module updates 48bc8b2b82 CPack: Move internal implementation modules into Internal/CPack directory 2a2829cc75 Help: Add new section for CPack generators be6267808a CPackIFW: Turn documentation into a block comment 0180524c7a Help: Move legacy CPack modules into separate section Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2149
| * | Help: Add new section for CPack generatorsKyle Edwards2018-06-212-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The documentation for CPack generators previously lived in their respective internal CMake modules. This setup was misleading, because it implied that you should include the modules in your own code, which is not the case. Moving the documentation into a separate section does a better job of hiding the internal modules, which are just an implementation detail. The generator documentation has also been modified to remove any references to the module name. The CPackIFW module is a special exception: since it has user-facing macros, the documentation for these macros has been kept in the module page, while all other documentation related to the IFW generator has been moved into the new section. To make it easier to find the new documentation, the old help pages for the CPack*.cmake modules have not been deleted, but have been replaced with a link to their respective help page in the new documentation section.
* | | Merge topic 'cmake_autogen_verbose'Brad King2018-06-225-0/+5
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5b85ef5cd0 Autogen: Add release notes for CMAKE_AUTOGEN_VERBOSE 6651aab2ab Autogen: Add documentation for CMAKE_AUTOGEN_VERBOSE aa7d8a092c Autogen: Enable CMAKE_AUTOGEN_VERBOSE in all tests e28dc3b1d8 Autogen: Add CMAKE_AUTOGEN_VERBOSE variable support Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2157
| * | | Autogen: Enable CMAKE_AUTOGEN_VERBOSE in all testsSebastian Holtermann2018-06-215-0/+5
| |/ /
* | | Merge topic 'vs_debugger'Brad King2018-06-229-4/+60
|\ \ \ | |/ / |/| | | | | | | | | | | | | | 797de7a6f6 VS10Project: Expand VS_DEBUGGER_* capabilities Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2150
| * | VS10Project: Expand VS_DEBUGGER_* capabilitiesJon Chronopoulos2018-06-179-4/+60
| | | | | | | | | | | | | | | This adds VS_DEBUGGER_COMMAND_ARGUMENTS and VS_DEBUGGER_ENVIRONMENT as well as allowing VS_DEBUGGER_* to use generator expressions.
* | | Merge topic 'xcode-10-legacy-build-system'Brad King2018-06-203-17/+22
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ca69206893 Tests: Do not use i386 architecture with Xcode 10 and above 71fa78ff7a Tests: Teach RunCMake to ignore Xcode missing file type warnings 057ecb8f6f C++ feature checks: Ignore Xcode warnings 985d3a162c Xcode: Use legacy build system Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2151
| * | | Tests: Do not use i386 architecture with Xcode 10 and aboveBrad King2018-06-192-17/+21
| | | | | | | | | | | | | | | | Xcode 10 dropped support for i386 builds.
| * | | Tests: Teach RunCMake to ignore Xcode missing file type warningsBrad King2018-06-191-0/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Xcode 10 beta warns xcodebuild... warning: file type '::com.apple.instruments.instrdst' is based on missing file type 'default::com.apple.package' Teach RunCMake to drop such lines before matching against expected output.
* | | Merge topic 'install_from_another_directory'Brad King2018-06-194-0/+8
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | e89ad0f94e install: Allow installing targets created in another directory Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2152
| * | | install: Allow installing targets created in another directoryZack Galbreath2018-06-184-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, `install(TARGETS)` would only accept targets created in the same directory scope. Relax this restriction by searching the global scope when determining whether or not a target exists. Fixes: #14444
* | | | Merge topic 'FindCURL-target-name'Brad King2018-06-191-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1e356734a7 FindCURL: Rename imported target to match upstream CURL Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Rolf Eike Beer <eike@sf-mail.de> Merge-request: !2154
| * | | | FindCURL: Rename imported target to match upstream CURLBrad King2018-06-181-1/+1
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upstream CURL provides imported target `CURL::libcurl`. Rename the target added by `FindCURL` to match. We don't need compatibility with the old name because it has never been in a CMake release (except a 3.12 release candidate). Suggested-by: Jakub Zakrzewski <slither.jz@gmail.com> Acked-by: Rolf Eike Beer <eike@sf-mail.de> Fixes: #18091
* | | | Merge topic 'subdir_target_sources'Craig Scott2018-06-1939-0/+298
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | 316815e1f4 target_sources: Interpret relative paths as relative to the calling directory Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2128
| * | | target_sources: Interpret relative paths as relative to the calling directoryPatrick Stotko2018-06-1839-0/+298
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the command considered non-absolute source file paths relative to the associated target on the LHS. This causes problems in incremental builds where files are added from subdirectories and forces users to workaround by manually converting to absolute paths. Change this to enable more intuitive usage by projects. Fixes #17981
* | | | Merge topic 'find_program-conditional-cwd'Brad King2018-06-184-0/+71
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c76c1ea208 find_program: Consider CWD only for paths with separator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2120
| * | | | find_program: Consider CWD only for paths with separatorSam Yates2018-06-144-0/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | find_program() incorrectly prepended search path components to absolute file paths, and incorrectly searched the current working directory for files that contained no directory separators. * Replace calls cmFindProgramHelper::CheckDirectory(std::string()) with call of new method cmFindProgramHelper::CheckCompoundNames() that checks for the presence of a directory separator in the file name. * Use cmSystemTools::CollapseCombinedPath rather than string concatenation to properly combine absolute file names with search path components. * Add unit tests to verify corrections. Fixes: #18044
* | | | | Merge topic 'UseSWIG-bugfixes'Brad King2018-06-183-0/+65
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | 08f2a2408d UseSWIG: add support of target property INCLUDE_DIRECTORIES consumption Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2141
| * | | | UseSWIG: add support of target property INCLUDE_DIRECTORIES consumptionMarc Chevrier2018-06-133-0/+65
| | | | | | | | | | | | | | | | | | | | Fixes: #18003
* | | | | Merge topic 'list_sort'Brad King2018-06-1427-5/+178
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | 49a51a61d7 list: Add options to control the SORT comparison operation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2065
| * | | | list: Add options to control the SORT comparison operationDaniel Franke2018-06-1327-5/+178
| |/ / /
* | | | Merge topic 'UseSWIG-csharp-variant'Brad King2018-06-134-0/+93
|\ \ \ \ | | |/ / | |/| / | |_|/ |/| | | | | | | | 0c2fb4c896 UseSWIG: Add CSHARP variant for wrapper files Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2137
| * | UseSWIG: Add CSHARP variant for wrapper filesRishi Theivendran2018-06-124-0/+93
| |/ | | | | | | | | Issue: #18065 Fixes: #18066
* | Merge topic 'LINK_OPTIONS'Brad King2018-06-0859-2/+508
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 66ea1a3795 LINK_OPTIONS: Add support of "LINKER:" prefix c1f5a44b28 LINK_OPTIONS: Add new family of properties 8e28d2630a Makefile generator: link flags management refactoring Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Alex Turbov <i.zaufi@gmail.com> Merge-request: !2033
| * | LINK_OPTIONS: Add support of "LINKER:" prefixMarc Chevrier2018-06-0620-29/+194
| | |
| * | LINK_OPTIONS: Add new family of propertiesMarc Chevrier2018-06-0641-2/+343
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | This family enable to manage link flags Three new properties: * directory property: LINK_OPTIONS * target properties: LINK_OPTIONS and INTERFACE_LINK_OPTIONS Two new commands * add_link_options(): to populate directory property * target_link_options(): to populate target properties Fixes: #16543
* | install: Teach CODE,SCRIPT modes to evaluate generator expressionsJon Chronopoulos2018-06-076-0/+19
|/ | | | Fixes: #15785
* Merge topic 'more-misc-typos'Brad King2018-06-053-3/+3
|\ | | | | | | | | | | | | | | be28106880 Fix misc. typos Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Alex Turbov <i.zaufi@gmail.com> Merge-request: !2130
| * Fix misc. typosluz.paz2018-06-043-3/+3
| | | | | | | | Found via `codespell` and `grep`
* | Merge topic 'ep-support-passing-var-ending-with-notfound'Brad King2018-06-054-2/+22
|\ \ | | | | | | | | | | | | | | | | | | 7ad5165c67 ExternalProject: Fix cache generation when args end with "-NOTFOUND" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2127
| * | ExternalProject: Fix cache generation when args end with "-NOTFOUND"Jean-Christophe Fillion-Robin2018-06-044-2/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Generalize the fix in commit v3.11.0-rc4~8^2 (ExternalProject: Fix cache generation when last args ends with "-NOTFOUND", 2018-03-10) to work for any argument rather than just the last one. ExternalProject can now successfully generate the cache file when any (not only the last one) cache variable associated with either `CMAKE_CACHE_ARGS` or `CMAKE_DEFAULT_CACHE_ARGS` configure step option is set to a `<value>` ending with `-NOTFOUND`.
| * | Merge branch 'custom-command-expand-empty' into release-3.11Brad King2018-05-142-0/+2
| |\ \ | | | | | | | | | | | | Merge-request: !2074
* | \ \ Merge topic 'WriteBasicConfigFile_tests'Brad King2018-06-041-239/+894
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | 56f45bb79d Refactor write_basic_package_version_file tests Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2122
| * | | Refactor write_basic_package_version_file testsDaniele E. Domenichelli2018-06-011-239/+894
| | | |
* | | | Revise C++ coding style using clang-format-6.0Kitware Robot2018-06-01214-1126/+1182
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Run the `clang-format.bash` script to update all our C and C++ code to a new style defined by `.clang-format`. Use `clang-format` version 6.0. * If you reached this commit for a line in `git blame`, re-run the blame operation starting at the parent of this commit to see older history for the content. * See the parent commit for instructions to rebase a change across this style transition commit.
* | | | clang-format.bash: update to clang-format-6.0Brad King2018-06-016-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update `.clang-format` with configuration to make the 6.0 format as close as possible to what 3.8 produced before. Then revise the style: * Indent preprocessor directives (a feature new since 3.8) * Add a newline and indentation before inheritance `:` and `,` Rename the Git attribute identifying the format to include the clang-format version number: `format.clang-format-6.0`. This will aid external infrastructure in knowing what version of the tool to run.
* | | | Tests/VSWinStorePhone: Do not format managed sourceBrad King2018-06-011-0/+1
| | | |
* | | | Merge topic 'autogen_autorcc_multi_config'Brad King2018-06-0111-0/+94
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5852acffa4 Autogen: Add AUTORCC configuration change test 41685c8ba0 Autogen: Let AUTORCC generate output for all configurations 2930a198f6 Autogen: Add lock file to AUTORCC commands Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2117
| * | | | Autogen: Add AUTORCC configuration change testSebastian Holtermann2018-05-3011-0/+94
| | | | |
* | | | | FindALSA: add testcaseRolf Eike Beer2018-05-314-0/+40
| |/ / / |/| | |
* | | | Merge topic 'vs-toolset-version'Brad King2018-05-3011-0/+44
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 5f13168419 VS: Add option to select the version of the toolset used by VS 2017 Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Francisco Facioni <fran6co@gmail.com> Merge-request: !2093