| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| | |
35e38ef97a Config File Gen: When requiring >= 2.6, don't check < 2.5
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Ben Boeckel <ben.boeckel@kitware.com>
Merge-request: !6456
|
| |
| |
| |
| |
| |
| | |
This seems to be a clear-cut case.
Signed-off-by: Marcus Müller <marcus@hostalia.de>
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
d1599c3f7f Fix build on Android (host)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6452
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
eae125ace5 Refactor: Get rid of `isTrue` variable in the `while` block execution
4c1cdfd8f0 Refactor: Keep `cmWhileFunctionBlocker` members private
d22f68d019 Refactor: Transform `while` loop into `for`
e97e714f0d Fix: `while()` reports an error the same way as `if()`
880ca66b51 Fix: `while()` can silently ignore incorrect condition
61b33c3f4e Fix: Regression in the `cmConditionEvaluator::HandleLevel0`
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6442
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
Particularly `Args`.
|
| | | |
| | | |
| | | |
| | | | |
And reduce scope for some variables + use some more `auto`.
|
| | | |
| | | |
| | | |
| | | | |
With arguments list expanded.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
When `conditionEvaluator.IsTrue(...)` returns `false` it just
didn't print the error occured.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As reported in the BUG #22524, mismatched parenthesis reported differently
for `while()` and `if()`.
The problem was in the double loop (over "handlers" and the arguments),
where the outer loop didn't check the result of the running handler.
|
| | | | |
|
| |_|/
|/| | |
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
'refactor-cmStandardLevelResolver_ComputeFeaturesAvailable-returns-cmProp'
361b1ea4c7 Refactor: cmStandardLevelResolver::ComputeFeaturesAvailable returns cmProp
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6450
|
| | | | |
|
| |_|/
|/| | |
|
|/ / |
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
97ed474431 Refactor: cmTest::GetProperty returns cmProp
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6438
|
| |/ |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
e542116814 Refactor: cmGeneratorTarget::GetSourcesProperty returns cmProp
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6439
|
| |/ |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
791337f181 Refactor: cmCommonTargetGenerator::GetFeature returns cmProp
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6440
|
| |/ |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
3941b74de4 Refactor: cmGlobalGenerator::GetGlobalSetting returns cmProp
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6441
|
| |/ |
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
58d10cf6f1 Alternative symlink-creating mode for file(INSTALL ...)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: Ben Boeckel <ben.boeckel@kitware.com>
Merge-request: !6396
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
An new environment variable 'CMAKE_INSTALL_MODE' is introduced,
which can be used to ask CMake to create symbolic links
instead of copying files during a file(INSTALL ...).
The operation is at the file level only, directory trees are
still created using actual directories, not links.
Signed-off-by: Felix Lelchuk <felix.lelchuk@gmx.de>
|
| |/
|/| |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
e5cd39ca80 cmProp: refactoring: transform alias in class
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6420
|
| | |
| | |
| | |
| | |
| | |
| | | |
To handle safely the values used by CMake variables and properties,
introduce the class cmProp as a replacement from the simple pointer
to std::string instance.
|
| | | |
|
| | | |
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
54a70b3988 Refactor: Move version compare op table out of the function
f37d6a3170 Refactor: Remove redundant checks for `0` and `1` literals
00961a4782 Refactor: Copy exactly required count of args and avoid `pop_back()`
2a72cad9be Style: Better names of local vars for `IN_LIST` handler
0dd7795706 Refactor: Extract the logic of testing for special variable to func
866b0595f6 Refactor: Introduce `cmArgumentList` container class
51d9194a96 Refactor: Reduce one more condition checking on handling math compare
46810235e3 Refactor: Avoid `if` → `else if` → … for compare operators
...
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !6392
|
| | |
| | |
| | |
| | | |
It's static data and never changed.
|
| | |
| | |
| | |
| | | |
They are gonna be checked by `cmIsOn()` anyway.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
Introduce `looksLikeSpecialVariable`. It makes `if()` blocks readable.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The `cmArgumentList` has been turned into a class (forward declared in
the header). It inherits from the `std::list` (yeah, but we don't intend
to store polymorphic classes in it). In addition to the standard methods,
now it's possible to move `HandlePredicate` (renamed to `ReduceOneArg`)
and `HandleBinaryOp` (renamed to `ReduceTwoArgs`) as its members.
Additionally, iterators managements (`IncrementArguments`) have been
refactored into two separate classes mimicking iterators. This also
allows having a uniform `for` loop and concentrates the logic of
iterators advancing in it instead of the loop's body. The arguments
processing algorithms operate with "windows" over a collection of
arguments. Hence there are two kinds of "iteration windows" -- allowing
to observe 2 or 3 elements per loop iteration. These iteration "windows"
also passed to reducers.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
When trying to match one of compare operator key inside `if()` condition,
remember the index of matched operator. Later this index used to
select the operation to perform instead of strings compare again.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
A handler's loop of all levels gonna restart after calls to
`HandlePredcate` or `HandleBinaryOp`... And the first action in the
loop is to setup iterators. So, no need to move 'em inside the
functions. Also, no need to pass iterators by reference.
Also, reorder parameters of both functions so iterators
followed after a reference to the newArgs container.
|
| | | |
|