| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
9040ec9 Do not warn about left paren not separated by a space
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|\ \
| | |
| | |
| | |
| | | |
d05a9bd Cygwin: Avoid legacy warnings in RunCMake.* tests
|
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
|/ /
| |
| |
| |
| |
| |
| | |
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.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
| |\ \
| | |/
| |/|
| | |
| | | |
Resolve conflict in Source/cmVariableWatchCommand.cxx by integrating the
changes from both sides.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | |
| | |
| | |
| | |
| | | |
Cover commands with whitespace present in allowed combinations.
Also cover command error cases such as two on one line.
|
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
|\ \ \
| | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
6f9aaad CTest: create one output file per memcheck (#14303)
|
| | |/
| |/|
| | |
| | |
| | |
| | | |
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.
|
|/ /
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* 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.
|
|\
| |
| |
| |
| |
| | |
80e652f Export: Process generator expressions from INCLUDES DESTINATION.
4355815 cmTarget: Add NAME property
|
| |
| |
| |
| | |
Configuration sensitive expressions are not permitted.
|
|/
|
|
|
|
|
|
| |
Code such as
target_include_directories(foo PRIVATE ${items})
should not work or break based on whether items is defined or not.
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| | |
Extend Tests/LinkFlags to test libraries of the type MODULE and
the content of the CMAKE_*_LINKER_FLAGS_* variables.
|
|\ \
| | |
| | |
| | |
| | | |
d12459f VS 6: Tell BuildDepends test to tolerate ninjadep failure
|
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
72d13ff install: Remove error condition using INCLUDES DESTINATION without EXPORT.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
f868e47 Fix crash on export of target with empty INTERFACE_INCLUDE_DIRECTORIES.
|
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
c0756b6 Revert "CTest: fix pre and post test commands with spaces" (#13887)
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| | |_|/
| |/| |
| | | |
| | | |
| | | | |
9275554 Ninja: Update BuildDepends test to verify cmcldeps depfiles.
6fa9d0a Ninja: Make cmcldeps depfile output more consistent with 'ninja -t msvc'
|
| | | | |
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
For shared libraries and executables, the linker_language is
indepenedent of the linked libraries.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
Refer correctly to the old property, not the new one.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
d777b8e Genex: Allow relative paths in INSTALL_INTERFACE.
|
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | | |
These paths can be prepended with the ${_IMPORT_PREFIX} generated
in the export file.
Such relative paths were previously an error.
|
|\ \ \ \
| |/ / /
| | / /
| |/ /
|/| | |
650e61f Add a convenient way to add the includes install dir to the INTERFACE.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
5837f19 export: Error when exporting a target without a language
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
c80a4f9 Tests/RunCMake: Document stripping of expected output
|
| | |/
| |/|
| | |
| | |
| | | |
Document that trailing newlines are stripped from the -stdout.txt
and -stderr.txt content before using as a regular expression.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
b655865 target_link_libraries: Add PUBLIC/PRIVATE/INTERFACE keyword signature
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
a43e5e0 Test COMPILE_DEFINITIONS target property get/set/get round-trip
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
f5ca872 Use linked frameworks as a source of include directories.
|