summaryrefslogtreecommitdiffstats
path: root/Source/cmServerProtocol.cxx
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'gicv-stdstring'Brad King2018-09-121-4/+4
|\ | | | | | | | | | | | | 13d10ee616 cmState::GetInitializedCacheValue: Return as const std::string* Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2365
| * cmState::GetInitializedCacheValue: Return as const std::string*Vitaly Stakhovsky2018-09-101-4/+4
| |
* | genex: Simplify cmGeneratorExpressionInterpreterBrad King2018-09-071-5/+4
|/ | | | | | | | | All callers were constructing with a non-empty target name using the target whose pointer was passed anyway. Drop this argument. Simplify logic accordingly. Re-order constructor arguments to match the cmCompiledGeneratorExpression::Evaluate arguments. Also remove unnecessary getters.
* Merge topic 'cmakeServerSourcesForInterfaceLibraries'Brad King2018-09-041-40/+183
|\ | | | | | | | | | | | | | | | | d74c2282ea cmake-server: Support codemodel filegroups for INTERFACE_SOURCES Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Tobias Hunger <tobias.hunger@gmail.com> Acked-by: Markus Enzenberger <markus.enzenberger@gmail.com> Merge-request: !2282
| * cmake-server: Support codemodel filegroups for INTERFACE_SOURCESJustin Goshi2018-08-301-40/+183
| | | | | | | | | | | | | | | | | | This change returns information for INTERFACE_SOURCES. We add a flag to the filegroup to indicate if the target represents interface sources. Protocol version is updated to 1.3 since this is a change to what was released in cmake version 3.12.
* | Merge topic 'genex-evaluate-stdstring'Brad King2018-08-301-1/+1
|\ \ | | | | | | | | | | | | | | | | | | 65ab9ce938 Genex: Return Evaluate results as const std::string& Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2314
| * | Genex: Return Evaluate results as const std::string&Vitaly Stakhovsky2018-08-281-1/+1
| | | | | | | | | | | | Also remove unused overloads.
* | | cmMakefile: return directories as const std::string&Vitaly Stakhovsky2018-08-271-1/+1
|/ /
* | cmCompiledGeneratorExpression::Evaluate(): return const std::string&Vitaly Stakhovsky2018-08-091-4/+3
|/
* IWYU: Fix workaround mapping for std::hashBrad King2018-08-071-1/+0
| | | | | | | | | It is provided by `functional`, not `utility`. Fix the mapping added by commit 276d3c7afe (IWYU: Add workaround mapping for std::hash, 2018-07-31). Also generalize the workaround from commit v3.12.0-rc1~39^2~1 (IWYU: Define a macro to tell code it is preprocessing for iwyu, 2018-05-25) to allow local builds to configure specific flags. This is needed because iwyu behaves differently in different environments.
* IWYU: Add workaround mapping for std::hashBrad King2018-07-311-0/+1
| | | | | When using GCC 8's standard library IWYU thinks that `<system_error>` must be included to get `std::hash`. Add a mapping for `<utility>`.
* Revise C++ coding style using clang-format-6.0Kitware Robot2018-06-011-13/+23
| | | | | | | | | | | | 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.
* Genex: Fix COMPILE_LANGUAGE in SYSTEM include directoriesBrad King2018-03-121-3/+4
| | | | | | | | | When evaluating `INTERFACE_SYSTEM_INCLUDE_DIRECTORIES`, or evaluating `INTERFACE_INCLUDE_DIRECTORIES` on an imported target, thread the compile language through to the generator expression evaluator so that it can support `$<COMPILE_LANGUAGE:...>`. Fixes: #17811
* Reduce raw string pointers usage.Pavel Solodovnikov2018-01-311-6/+5
| | | | | | | | | | * Change some functions to take `std::string` instead of `const char*` in the following classes: `cmMakeFile`, `cmake`, `cmCoreTryCompile`, `cmSystemTools`, `cmState`, `cmLocalGenerator` and a few others. * Greatly reduce using of `const char*` overloads for `cmSystemTools::MakeDirectory` and `cmSystemTools::RelativePath`. * Remove many redundant `c_str()` conversions throughout the code.
* Reduce allocation of temporary values on heap.Pavel Solodovnikov2018-01-261-3/+3
| | | | | - Use `std::move` while inserting temporary results into vectors. - Change `push_back` to `emplace_back` where appropriate.
* Merge topic 'server-polish-opt-handshake'Brad King2018-01-251-14/+15
|\ | | | | | | | | | | | | | | 239a3ef8 Server-mode: Document protocol version for optional handshake arguments 9b1a3d24 Server-mode: Rename functions to reflect what they do Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1693
| * Server-mode: Rename functions to reflect what they doTobias Hunger2018-01-241-14/+15
| | | | | | | | | | Rename the test* static functions to getOrTest* to better reflect what they do now.
* | sourceFile properties: add property INCLUDE_DIRECTORIESMarc Chevrier2018-01-241-1/+18
| |
* | sourceFile properties: add property COMPILE_OPTIONSMarc Chevrier2018-01-231-0/+5
|/ | | | | | | Add the support of per-source property COMPILE_OPTIONS, including generator expressions support. Related: #17507
* server: fix crash if no min version specifiedJustin Goshi2018-01-191-2/+2
| | | | | | | If a CMakeLists.txt file doesn't contain cmake_minimum_required then the server was crashing. The root cause was the json object model does not support null and was crashing. Add the null check and use an empty string in this case.
* server: Expand generator expressions for test infoJustin Goshi2018-01-041-7/+18
|
* Genex: Per-source $<COMPILE_LANGUAGE:...> supportMarc Chevrier2017-12-131-7/+12
| | | | Fixes: #17542
* Merge branch 'backport-revert-server-target-backtraces' into ↵Brad King2017-12-061-48/+0
|\ | | | | | | revert-server-target-backtraces
| * server: Revert "Report backtraces in codemodel response"Brad King2017-12-061-49/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The backtrace information is very repetitive and hugely increases the size of the codemodel object. We need to remove it until an alternative representation can be developed. Revert commit v3.10.0-rc1~393^2 (server: Report backtraces in codemodel response, 2017-06-20), except for the protocol version number (because it indicates other new things). Unfortunately this is incompatible with clients that expect the "crossReferences" field in targets. However, the regression in memory usage is quite serious, especially on large projects, and therefore breaks even older clients that do not use backtraces. Since the "crossReferences" field was only provided by one release (3.10.0), it is simplest to revert it outright for 3.10.1. Fixes: #17502
* | server: drop "ctestInfo" backtrace informationBrad King2017-12-061-3/+0
| | | | | | | | | | | | | | | | | | Backtrace information was included by commit 35a52bd1b4 (server: add "ctestInfo" request to get test info, 2017-10-25) to match that already provided for targets. However, the backtrace representation uses too much memory and needs to be dropped. Remove it from test information. Issue: #17502
* | Add generator expression support to per-source COMPILE_DEFINITIONSMarc Chevrier2017-12-051-2/+8
| | | | | | | | | | | | This allows users to specify different genex-based compile definitions for each file in a target. Fixes: #17508
* | Refactor per-source generator expression evaluationMarc Chevrier2017-12-041-5/+3
| | | | | | | | | | | | Prepare to add generator expression support to more source properties. Factor out some duplicated code into a helper to avoid further duplication.
* | Merge topic 'server-target-isGeneratorProvided'Brad King2017-11-271-0/+2
|\ \ | | | | | | | | | | | | | | | | | | afd9a339 server: return whether or not a target is generator provided Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1475
| * | server: return whether or not a target is generator providedJustin Goshi2017-11-201-0/+2
| | | | | | | | | | | | | | | | | | Some generators auto-generate targets. For example VS generators create the ALL_BUILD target. Add the ability to mark targets as generator provided and return that info through cmake-server codemodel.
* | | Merge topic 'serverFixTestDiscovery'Brad King2017-11-271-72/+19
|\ \ \ | |/ / |/| | | | | | | | | | | | | | fe2c2b0f server: ctestInfo fix to return all tests Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1479
| * | server: ctestInfo fix to return all testsJustin Goshi2017-11-171-72/+19
| | | | | | | | | | | | | | | Prior to this change we were looking at targets. But tests are associated with directories. This change fixes how we gather all tests.
* | | server: project has install rule bug fixJustin Goshi2017-11-161-1/+14
|/ / | | | | | | | | Need to check all generators associated with the project because any of them may have an install rule.
* | server: return minimum cmake required version for each projectJustin Goshi2017-11-071-0/+2
| |
* | server: add "ctestInfo" request to get test infoJustin Goshi2017-11-031-0/+165
| |
* | cmake-server: Add target install destinations to codemodelJustin Goshi2017-10-241-1/+33
|/ | | | | Protocol version is updated to 1.2 since this is a change to what was released in cmake version 3.10.
* Improve several occurrences of vector::push_back in loopsMatthias Maennich2017-09-281-1/+2
| | | | | | | Fix issues diagnosed by clang-tidy by pre-allocating the vector capacity before the loop [performance-inefficient-vector-operation]. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Meta: modernize old-fashioned loops to range-based `for`.Pavel Solodovnikov2017-09-121-32/+30
| | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* clang-format: format all code as Cpp11Daniel Pfeifer2017-08-301-2/+2
|
* Merge topic 'cstyle-casts'Daniel Pfeifer2017-08-301-1/+1
|\ | | | | | | | | | | | | 190e3825 Replace C-style casts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1176
| * Replace C-style castsDaniel Pfeifer2017-08-271-1/+1
| |
* | server: backport to C++11Daniel Pfeifer2017-08-261-1/+1
|/
* Use C++11 nullptrDaniel Pfeifer2017-08-241-1/+1
|
* Performance: Fix a few more unnecessary vector copies missed in af3fd6fAaron Orenstein2017-08-181-1/+1
|
* cmServerProtocol: Do not move into json::Value::append()Daniel Pfeifer2017-08-151-1/+1
| | | | | | | CMake's copy of json-cpp is not aware of rvalues yet. It is confusing to pass the result of std::move to a function that takes a const& because no move will actually happen. This change may be reverted once CMake upgrades to a new version of json-cpp.
* cmServerProtocol: pass cmBacktraceRange by valueDaniel Pfeifer2017-08-151-1/+1
|
* cmServerProtocol: allow 'cache' request before 'configure'Daniel Pfeifer2017-08-131-4/+0
| | | | Fixes: #16989
* cmServerProtocol: fix test of empty valuesDaniel Pfeifer2017-08-131-7/+8
| | | | | | | If a required value is in the cache, it is not necessary to set it explicitly. Fixes: #16948, #16988
* Merge topic 'server-fix-crash'Brad King2017-08-031-1/+3
|\ | | | | | | | | | | | | 70272f3c server: Fix crash on missing cache entries Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1096
| * server: Fix crash on missing cache entriesLaurent Rineau2017-08-021-1/+3
| | | | | | | | Test for nullptr before constructing std::string.
* | Merge topic 'server-refactor'Brad King2017-07-181-2/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | cf0ae55d server: Add support for connections that aren't event based 5ddfb6a4 server: Add connection as part of a request d4f5d35c server: Refactor to make the event loop owned by server object 5acbf08b Tests: Teach Server test to forward exit code from server process Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !552
generated by cgit v0.12 at 2024-11-12 03:02:45 (GMT)