| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| | |
e7e25c6cb6 cmVisualStudio10TargetGenerator: Remove empty Elem::EndElement()
726c090297 cmVisualStudio10TargetGenerator: close XML tag in Elem destructor
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2109
|
| |
| |
| |
| | |
The method no longer does anything. Remove it and remove calls to it.
|
| |
| |
| |
| | |
RAII actually implemented; EndElement() still kept to avoid major reformatting
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
42738e5075 Help: Update description of the -D option to CMake
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2105
|
| | |
| | |
| | |
| | | |
Fixes #18021
|
| | | |
|
| |/
|/| |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
45c4a75d27 cmVisualStudio10TargetGenerator: make sure each Elem has right scope
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2096
|
| | |
| | |
| | |
| | | |
Prepare for future RAII
|
| |/
|/| |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
b1a05d6c76 Revise implementation of case-insensitive command names
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2024
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Store both the as-written and lower-case command names and use
the latter to avoid case-insensitive string comparisons.
With this I obtain 2-6% speed increase (on Windows) for the configure
step with no significant changes in memory usage. A case-insensitive
comparison is a lot slower than just calling `==` because the operator
will use things like memcmp, so prefer the latter.
The `cmSystemTools::LowerCase` function allocates a new string each time
it is called, so before this change we were allocating in:
* cmMakefile::Configure two times for each function
(to look for `cmake_minimum_required` and `project`)
* cmMakefile::ExecuteCommand twice by function by calling
cmState::GetCommand and copying the name
Now we are only allocating once by function instead of four.
|
| | | |
|
| | | |
|
| |/
|/| |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
a203fcc63d cmake: Teach '-E tar' to report errors copying data
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2075
|
| |/
| |
| |
| |
| | |
The `copy_data` function checks for errors but the caller ignored
them. Simplify its return type and add a check to the caller.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
fd0da95f63 Eclipse: Add property to include additional contents in .cproject file
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2088
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Similar to ECLIPSE_EXTRA_NATURES which can be used to add a third party
nature to the .project, this property offers the possibility to inject
third party contents into the .cproject.
An example of where this is useful is MCUXpresso. This is an IDE based
on Eclipse. Compiling through CMake generated Eclipse projects works
fine by using a custom toolchain. However, in order to be able to debug
using such a project, an extra storageModule specific to the target
board is needed in the cproject.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
79ca546ed2 Add generator expression support to PDB_OUTPUT_DIRECTORY target property
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2083
|
| | | |
| | | |
| | | |
| | | | |
Fixes: #16365
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
790fc415b1 cmVisualStudio10TargetGenerator: remove BuildFileStream member
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2082
|
| |/ / /
| | | |
| | | |
| | | | |
Stream object is now local to `Generate()`
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
b730097788 Merge branch 'upstream-KWSys' into update-kwsys
1f52cb087a KWSys 2018-05-18 (5357cfc4)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2090
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream-KWSys:
KWSys 2018-05-18 (5357cfc4)
|
| | | | | |
|
| | | | | |
|
|/ / / / |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
b6c49713b3 Makefile: Fix command line limits for static libs
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2078
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
I have been hitting the Windows command-line limit when cross-compiling
static library (POCO) and having more than 8000 characters in the call
to "ar". Calculating exact limits here are tricky, since this particular
limit will only take into account object file strings, which is correct
for response files, but not for the archive rules (link.txt files),
since they also contain the call to "ar" and its arguments.
Also, there can be other additional arguments if "ar" tool is wrapped
into something else, so it is a good idea to leave more space than
trying to exactly fit the limit.
Since response files use half of the limit as a heuristic, we reproduce
it here for consistency.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
2db4945150 file(WRITE): Report errors during write operation
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2076
|
| | |_|_|/
| |/| | |
| | | | |
| | | | |
| | | | | |
We already report an error if the file cannot be opened for writing.
Add another check to report an error if a write operation itself fails.
|
|\ \ \ \ \
| |_|_|_|/
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
3333e2791b Help: Add release note about 'copy_directory' behavior change
24367563d7 Merge branch 'upstream-KWSys' into update-kwsys
1e0a2e9377 KWSys 2018-05-15 (5f757898)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2080
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | | |
* upstream-KWSys:
KWSys 2018-05-15 (5f757898)
Fixes: #16001
|
| |_|/
|/| | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
c9349cc1b9 target_link_libraries: Allow use with targets in other directories
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2040
|
| | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | | |
Previously the command did not allow naming targets on the LHS that
were not created in the calling directory. Lift this restriction to
enable more flexible use by projects.
Fixes: #17943
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
71a033616a added CMAKE_DOTNET_TARGET_FRAMEWORK_VERSION variable
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2032
|
| | | |
| | | |
| | | |
| | | | |
Fixes: #17955
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | | |
b88e138369 cmVisualStudio10TargetGenerator: minor code cleanup
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2063
|
| | | | |
|
| |_|/
|/| | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
e461a136c1 Autogen: Register generated dependency files
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2071
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For CMP0058 record that rcc info files are generated by CMake.
Fixes: #17985
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | | |
6e59491659 add_custom_{command,target}: Fix crash on empty expanded command
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2074
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Our custom command generation logic assumes that all command lines have
at least `argv0`. In `add_custom_{command,target}` we already check
that at least a `COMMAND` was given, but using `COMMAND_EXPAND_LISTS` in
combination with a generator expression that expands to an empty string
may produce an empty command line. In this case simply add an empty
string as a command to maintain our internal invariant.
Fixes: #17993
|
| | | |
|