summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'cmake-syntax'Brad King2013-08-154-0/+6
|\ | | | | | | | | 9040ec9 Do not warn about left paren not separated by a space
| * Do not warn about left paren not separated by a spaceBrad King2013-08-154-0/+6
| | | | | | | | | | | | | | | | | | | | Since commit 58e52416 (Warn about arguments not separated by whitespace, 2013-02-16) we warn about arguments not separated by spaces. Loosen the warning to not complain about left parens not separated by spaces from the preceding token. This is common in code like "if(NOT(X))". Teach the RunCMake.Syntax test to cover cases of left parens not separated by spaces and check that no warning appears.
* | Merge topic 'cygwin-RunCMake-tests'Brad King2013-08-1436-36/+36
|\ \ | | | | | | | | | | | | d05a9bd Cygwin: Avoid legacy warnings in RunCMake.* tests
| * | Cygwin: Avoid legacy warnings in RunCMake.* testsBrad King2013-08-1336-36/+36
| | | | | | | | | | | | | | | | | | Set the minimum required version of CMake high enough to avoid the warning for CMAKE_LEGACY_CYGWIN_WIN32. The warning appears on stderr and breaks the expected output matching.
* | | Genex: Fix segfault when parsing ends with parameter expectation.Stephen Kelly2013-08-132-0/+4
|/ / | | | | | | | | | | | | The extendResult method expects a non-empty parameters vector, as assured by the normal case. Avoid calling the method when the parser finds an incomplete generator expression, but has already entered the state of expecting to find parameters.
* | Merge topic 'cmake-syntax'Brad King2013-08-0840-0/+193
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b93982f Merge branch 'dev/fix-variable-watch-crash' into cmake-syntax c50f7ed cmListFileLexer: Modify flex output to avoid Borland warning bf73264 Warn about unquoted arguments that look like long brackets 58e5241 Warn about arguments not separated by whitespace e75b69f cmListFileCache: Convert CMake language parser to class e945949 Add RunCMake.Syntax test cases for command invocation styles 0546484 cmListFileArgument: Generalize 'Quoted' bool to 'Delimeter' enum 28685ad cmListFileLexer: Split normal and legacy unquoted arguments 1eafa3e cmListFileLexer: Fix line number after backslash in string f3155cd Add RunCMake.Syntax test to cover argument parsing
| * \ Merge branch 'dev/fix-variable-watch-crash' into cmake-syntaxBrad King2013-08-083-0/+14
| |\ \ | | |/ | |/| | | | | | | Resolve conflict in Source/cmVariableWatchCommand.cxx by integrating the changes from both sides.
| * | Warn about unquoted arguments that look like long bracketsBrad King2013-08-084-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | In the future CMake will introduce Lua-style long bracket syntax. Warn about unquoted arguments that in the future will be treated as opening long brackets. Teach the RunCMake.Syntax test to cover such cases and ensure that the warning appears.
| * | Warn about arguments not separated by whitespaceBrad King2013-08-081-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Teach the lexer to return tokens for whitespace. Teach the parser to tolerate the space tokens where whitespace is allowed. Also teach the parser to diagnose and warn about cases of quoted arguments followed immediately by another argument. This was accidentally allowed previously, so we only warn. Update the RunCMake.Syntax test case StringNoSpace expected stderr to include the warnings.
| * | Add RunCMake.Syntax test cases for command invocation stylesBrad King2013-08-0816-0/+74
| | | | | | | | | | | | | | | Cover commands with whitespace present in allowed combinations. Also cover command error cases such as two on one line.
| * | cmListFileLexer: Fix line number after backslash in stringBrad King2013-08-084-0/+12
| | | | | | | | | | | | | | | | | | If a line inside a string ends in a backslash count the following newline character as a line increment. Add a test covering this case to verify that subsequent line numbers are correct.
| * | Add RunCMake.Syntax test to cover argument parsingBrad King2013-08-0819-0/+53
| | | | | | | | | | | | | | | | | | | | | Test basic unquoted and quoted argument parsing cases including failure on an unterminated string and an unterminated command invocation. Also cover arguments not separated by any spaces, which is accidentally allowed by the current parser.
* | | Merge topic 'dev/fix-variable-watch-crash'Brad King2013-08-083-0/+14
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6aa0c21 variable_watch: Add test for watching a variable multiple times b86e37c variable_watch: Check newValue for NULL f9bb20f variable_watch: Don't share memory for callbacks 05dad99 variable_watch: Fix a typo in the error message 00ce12a variable_watch: Prevent making extra entries in the watch map 34b397e variable_watch: Allow specifying the data to match in RemoveWatch e43e207 variable_watch: Match client_data when finding duplicates 0d6acb1 variable_watch: Add a deleter for the client data fc7c3b4 variable_watch: Store client data as pointers
| * | variable_watch: Add test for watching a variable multiple timesBen Boeckel2013-08-083-0/+14
| | |
* | | Merge topic 'parallel-memcheck'Brad King2013-08-082-13/+16
|\ \ \ | | | | | | | | | | | | | | | | 6f9aaad CTest: create one output file per memcheck (#14303)
| * | | CTest: create one output file per memcheck (#14303)Rolf Eike Beer2013-08-072-13/+16
| | |/ | |/| | | | | | | | | | | | | The output file used for memory checker runs must be unique for every test run in parallel, so simply make them unique for every test run. Simply use the test index to avoid collisions.
* | | Skip CTestLimitDashJ test on BorlandBrad King2013-08-071-3/+5
|/ / | | | | | | | | | | | | The Borland-built CTest binary has trouble running many instances of itself in parallel, making the test unreliable. No particular recent change appears to be the culprit and the failure is not reliably reproducible. Just silence the failure for now by skipping the test.
* | Add the ALIAS target concept for libraries and executables.Stephen Kelly2013-08-0270-3/+386
|/ | | | | | | | | | | | | | | | | * The ALIAS name must match a validity regex. * Executables and libraries may be aliased. * An ALIAS acts immutable. It can not be used as the lhs of target_link_libraries or other commands. * An ALIAS can be used with add_custom_command, add_custom_target, and add_test in the same way regular targets can. * The target of an ALIAS can be retrieved with the ALIASED_TARGET target property. * An ALIAS does not appear in the generated buildsystem. It is kept separate from cmMakefile::Targets for that reason. * A target may have multiple aliases. * An ALIAS target may not itself have an alias. * An IMPORTED target may not have an alias. * An ALIAS may not be exported or imported.
* Merge topic 'INCLUDES-DESTINATION-no-config'Brad King2013-08-015-12/+29
|\ | | | | | | | | | | 80e652f Export: Process generator expressions from INCLUDES DESTINATION. 4355815 cmTarget: Add NAME property
| * Export: Process generator expressions from INCLUDES DESTINATION.Stephen Kelly2013-07-315-12/+29
| | | | | | | | Configuration sensitive expressions are not permitted.
* | Allow target commands to be invoked with no items (#14325).Stephen Kelly2013-07-313-0/+24
|/ | | | | | | | Code such as target_include_directories(foo PRIVATE ${items}) should not work or break based on whether items is defined or not.
* Merge topic 'static_library_flags'Brad King2013-07-314-2/+43
|\ | | | | | | | | | | | | | | | | | | 33e6e0b VS6: Add handling of CMAKE_*_LINKER_FLAGS_<CONFIG> variables 152dfda Add additonal tests for the linker flags 20ed496 Add documentation for the missing CMAKE_*_LINKER_FLAGS_* variables 54f7019 Add CMAKE_STATIC_LINKER_FLAGS to CMakeCommonLanguageInclude 2a43c30 Add support for CMAKE_STATIC_LINKER_FLAGS 14bbf83 Unify the way the flags of a static library are read
| * Add additonal tests for the linker flagsPatrick Gansterer2013-07-314-2/+43
| | | | | | | | | | Extend Tests/LinkFlags to test libraries of the type MODULE and the content of the CMAKE_*_LINKER_FLAGS_* variables.
* | Merge topic 'ninja_bad_cmcldeps_paths'Brad King2013-07-311-0/+3
|\ \ | | | | | | | | | | | | d12459f VS 6: Tell BuildDepends test to tolerate ninjadep failure
| * | VS 6: Tell BuildDepends test to tolerate ninjadep failureBrad King2013-07-301-0/+3
| | | | | | | | | | | | | | | | | | The VS 6 IDE does not want to recompile a particular source after a particular header it includes is modified, even by hand. For now just silence the failure and document it with a comment.
* | | Merge topic 'install-interface-includes'Brad King2013-07-314-10/+0
|\ \ \ | | | | | | | | | | | | | | | | 72d13ff install: Remove error condition using INCLUDES DESTINATION without EXPORT.
| * | | install: Remove error condition using INCLUDES DESTINATION without EXPORT.Stephen Kelly2013-07-294-10/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 650e61f8 (Add a convenient way to add the includes install dir to the INTERFACE., 2013-01-05) introduced an error case for using the install(TARGETS) command with specified INCLUDES DESTINATION, but no specified EXPORT set. It is convenient to use a variable to set the various destinations for different outputs (as KDE does), and some targets such as executables are installed but not exported. This was triggering the error case, but as it is a common case, remove the error.
* | | | Merge topic 'fix-export-includes-crash'Brad King2013-07-292-0/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | f868e47 Fix crash on export of target with empty INTERFACE_INCLUDE_DIRECTORIES.
| * | | | Fix crash on export of target with empty INTERFACE_INCLUDE_DIRECTORIES.Stephen Kelly2013-07-292-0/+7
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | The new feature of install(TARGETS ... INCLUDES DESTINATION) introduced in commit 650e61f8 (Add a convenient way to add the includes install dir to the INTERFACE., 2013-01-05) introduced this crash. If the new feature is used with a target which has no INTERFACE_INCLUDE_DIRECTORIES, a segfault occurred.
* | | | Merge topic 'memcheck_pre_post_args'Brad King2013-07-291-1/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | c0756b6 Revert "CTest: fix pre and post test commands with spaces" (#13887)
| * | | | Revert "CTest: fix pre and post test commands with spaces" (#13887)Rolf Eike Beer2013-07-271-1/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 6187876dea89618044e200808bcae75a18bd4043. It was actually possible before to have paths with spaces in them, the spaces just need to be quoted. This way spaces will work as argument separators.
* | | | Merge topic 'ninja_bad_cmcldeps_paths'Brad King2013-07-293-0/+58
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | 9275554 Ninja: Update BuildDepends test to verify cmcldeps depfiles. 6fa9d0a Ninja: Make cmcldeps depfile output more consistent with 'ninja -t msvc'
| * | | Ninja: Update BuildDepends test to verify cmcldeps depfiles.Robert Maynard2013-07-263-0/+58
| | | |
* | | | Merge topic 'minor-cleanups'Brad King2013-07-299-4/+33
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b8dc7fa Genex: Disallow LINKER_LANGUAGE only when used on a static library. c8a10ba cmTarget: Fix iface libraries and languages for static libraries. f94bdb3 cmTarget: Remove duplicates when printing traces of tll signatures ff3d5fa Export: Fix typo of LINK_INTERFACE_LIBRARIES. 79a7a81 Docs: Document variables for default visibility values. 6f6391b Docs: Generalize and de-duplicate VISIBILITY_PREFIX docs. d8cb47f Docs: Trim trailing whitespace in generated doc. f10e648 Docs: Document existing target property debugging options. 4f4d69f Qt4Macros: Simplify some variable population. a413a40 Qt4Macros: Remove undefined varible use. b60a29e Qt4Macros: Remove unneeded generate CONDITION. e454cba Docs: Document file(GENERATE) CONDITION as optional.
| * | | Genex: Disallow LINKER_LANGUAGE only when used on a static library.Stephen Kelly2013-07-261-2/+2
| | | | | | | | | | | | | | | | | | | | For shared libraries and executables, the linker_language is indepenedent of the linked libraries.
| * | | cmTarget: Fix iface libraries and languages for static libraries.Stephen Kelly2013-07-266-0/+29
| | | |
| * | | Export: Fix typo of LINK_INTERFACE_LIBRARIES.Stephen Kelly2013-07-262-2/+2
| | | | | | | | | | | | | | | | Refer correctly to the old property, not the new one.
* | | | Merge topic 'install-interface-relative'Brad King2013-07-262-0/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | d777b8e Genex: Allow relative paths in INSTALL_INTERFACE.
| * | | | Genex: Allow relative paths in INSTALL_INTERFACE.Stephen Kelly2013-07-252-0/+8
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | These paths can be prepended with the ${_IMPORT_PREFIX} generated in the export file. Such relative paths were previously an error.
* | | | Merge topic 'install-interface-includes'Brad King2013-07-266-1/+31
|\ \ \ \ | |/ / / | | / / | |/ / |/| | 650e61f Add a convenient way to add the includes install dir to the INTERFACE.
| * | Add a convenient way to add the includes install dir to the INTERFACE.Stephen Kelly2013-07-246-1/+31
| | | | | | | | | | | | | | | | | | | | | Export the INCLUDES DESTINATION without appending to the INTERFACE_INCLUDE_DIRECTORIES of the target itself. That way, a target can be exported multiple times with different INCLUDES DESTINATION without unintended cross-pollution of export sets.
* | | Merge topic 'dev/export-target-without-language'Brad King2013-07-256-0/+16
|\ \ \ | | | | | | | | | | | | | | | | 5837f19 export: Error when exporting a target without a language
| * | | export: Error when exporting a target without a languageBen Boeckel2013-07-256-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First, it prevents a NULL dereference and second it reiterates that targets without languages are not supported by CMake. Add a RunCMake.ExportWithoutLanguage test exporting a library without a languages.
* | | | Merge topic 'doc-RunCMake-strips-expected'Brad King2013-07-251-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | c80a4f9 Tests/RunCMake: Document stripping of expected output
| * | | Tests/RunCMake: Document stripping of expected outputBen Boeckel2013-07-251-1/+1
| | |/ | |/| | | | | | | | | | Document that trailing newlines are stripped from the -stdout.txt and -stderr.txt content before using as a regular expression.
* | | Merge topic 'tll-new-signatures'Brad King2013-07-2524-0/+175
|\ \ \ | | | | | | | | | | | | | | | | b655865 target_link_libraries: Add PUBLIC/PRIVATE/INTERFACE keyword signature
| * | | target_link_libraries: Add PUBLIC/PRIVATE/INTERFACE keyword signatureStephen Kelly2013-07-2424-0/+175
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new signature to help populate INTERFACE_LINK_LIBRARIES and LINK_LIBRARIES cleanly in a single call. Add policy CMP0023 to control whether the keyword signatures can be mixed with uses of the plain signatures on the same target.
* | | | Merge topic 'compile-defs-debugging'Brad King2013-07-251-0/+13
|\ \ \ \ | |/ / / |/| | | | | | | | | | | a43e5e0 Test COMPILE_DEFINITIONS target property get/set/get round-trip
| * | | Test COMPILE_DEFINITIONS target property get/set/get round-tripBrad King2013-07-241-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Teach the Preprocess test to get, set, and then get the same value for the COMPILE_DEFINITIONS target property and verify that the value is not changed. This ensures the internal structured storage of the property value can reproduce the original string value.
* | | | Merge topic 'framework-interface-includes'Brad King2013-07-245-0/+27
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | f5ca872 Use linked frameworks as a source of include directories.