summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'fix_2.6_check'Brad King2021-08-191-1/+1
|\ | | | | | | | | | | | | | | 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
| * Config File Gen: When requiring >= 2.6, don't check < 2.5Marcus Müller2021-08-161-1/+1
| | | | | | | | | | | | This seems to be a clear-cut case. Signed-off-by: Marcus Müller <marcus@hostalia.de>
* | Merge topic 'android-build-host'Brad King2021-08-191-2/+4
|\ \ | | | | | | | | | | | | | | | | | | d1599c3f7f Fix build on Android (host) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6452
| * | Fix build on Android (host)leha-bot2021-08-131-2/+4
| | |
* | | CMake Nightly Date StampKitware Robot2021-08-191-1/+1
| | |
* | | Merge topic 'while-regression'Brad King2021-08-182-45/+52
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | | Refactor: Get rid of `isTrue` variable in the `while` block executionAlex Turbov2021-08-111-5/+3
| | | |
| * | | Refactor: Keep `cmWhileFunctionBlocker` members privateAlex Turbov2021-08-111-11/+10
| | | | | | | | | | | | | | | | Particularly `Args`.
| * | | Refactor: Transform `while` loop into `for`Alex Turbov2021-08-111-16/+22
| | | | | | | | | | | | | | | | And reduce scope for some variables + use some more `auto`.
| * | | Fix: `while()` reports an error the same way as `if()`Alex Turbov2021-08-091-7/+5
| | | | | | | | | | | | | | | | With arguments list expanded.
| * | | Fix: `while()` can silently ignore incorrect conditionAlex Turbov2021-08-091-18/+18
| | | | | | | | | | | | | | | | | | | | When `conditionEvaluator.IsTrue(...)` returns `false` it just didn't print the error occured.
| * | | Fix: Regression in the `cmConditionEvaluator::HandleLevel0`Alex Turbov2021-08-091-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | CMake Nightly Date StampKitware Robot2021-08-181-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2021-08-171-1/+1
| |_|/ |/| |
* | | CMake Nightly Date StampKitware Robot2021-08-161-1/+1
| | |
* | | CMake Nightly Date StampKitware Robot2021-08-151-1/+1
| | |
* | | Merge topic ↵Marc Chevrier2021-08-143-6/+6
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'refactor-cmStandardLevelResolver_ComputeFeaturesAvailable-returns-cmProp' 361b1ea4c7 Refactor: cmStandardLevelResolver::ComputeFeaturesAvailable returns cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6450
| * | | Refactor: cmStandardLevelResolver::ComputeFeaturesAvailable returns cmPropMarc Chevrier2021-08-123-6/+6
| | | |
* | | | CMake Nightly Date StampKitware Robot2021-08-141-1/+1
| |_|/ |/| |
* | | CMake Nightly Date StampKitware Robot2021-08-131-1/+1
|/ /
* | CMake Nightly Date StampKitware Robot2021-08-121-1/+1
| |
* | Merge topic 'refactor-cmTest_GetProperty-returns-cmProp'Brad King2021-08-114-14/+19
|\ \ | | | | | | | | | | | | | | | | | | 97ed474431 Refactor: cmTest::GetProperty returns cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6438
| * | Refactor: cmTest::GetProperty returns cmPropMarc Chevrier2021-08-094-14/+19
| |/
* | Merge topic 'refactor-cmGeneratorTarget_GetSourcesProperty-returns-cmProp'Brad King2021-08-112-4/+4
|\ \ | | | | | | | | | | | | | | | | | | e542116814 Refactor: cmGeneratorTarget::GetSourcesProperty returns cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6439
| * | Refactor: cmGeneratorTarget::GetSourcesProperty returns cmPropMarc Chevrier2021-08-092-4/+4
| |/
* | Merge topic 'refactor-cmCommonTargetGenerator_GetFeature-returns-cmProp'Brad King2021-08-112-5/+6
|\ \ | | | | | | | | | | | | | | | | | | 791337f181 Refactor: cmCommonTargetGenerator::GetFeature returns cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6440
| * | Refactor: cmCommonTargetGenerator::GetFeature returns cmPropMarc Chevrier2021-08-092-5/+6
| |/
* | Merge topic 'refactor-cmGlobalGenerator_GetGlobalSetting-returns-cmProp'Brad King2021-08-112-4/+5
|\ \ | | | | | | | | | | | | | | | | | | 3941b74de4 Refactor: cmGlobalGenerator::GetGlobalSetting returns cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6441
| * | Refactor: cmGlobalGenerator::GetGlobalSetting returns cmPropMarc Chevrier2021-08-092-4/+5
| |/
* | CMake Nightly Date StampKitware Robot2021-08-111-1/+1
| |
* | Merge topic 'symlinks-rebase-master'Brad King2021-08-104-3/+142
|\ \ | | | | | | | | | | | | | | | | | | | | | 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
| * | Alternative symlink-creating mode for file(INSTALL ...)Felix Lelchuk2021-08-024-3/+142
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | CMake Nightly Date StampKitware Robot2021-08-101-1/+1
| |/ |/|
* | Merge topic 'cmProp-class'Brad King2021-08-0939-224/+481
|\ \ | | | | | | | | | | | | | | | | | | e5cd39ca80 cmProp: refactoring: transform alias in class Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !6420
| * | cmProp: refactoring: transform alias in classMarc Chevrier2021-08-0839-224/+481
| | | | | | | | | | | | | | | | | | 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.
* | | CMake Nightly Date StampKitware Robot2021-08-091-1/+1
| | |
* | | CMake Nightly Date StampKitware Robot2021-08-081-1/+1
| | |
* | | CMake Nightly Date StampKitware Robot2021-08-071-1/+1
|/ /
* | Merge topic 'refactor-conditions-evaluation'Brad King2021-08-062-474/+520
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | Refactor: Move version compare op table out of the functionAlex Turbov2021-08-031-7/+7
| | | | | | | | | | | | It's static data and never changed.
| * | Refactor: Remove redundant checks for `0` and `1` literalsAlex Turbov2021-08-031-9/+1
| | | | | | | | | | | | They are gonna be checked by `cmIsOn()` anyway.
| * | Refactor: Copy exactly required count of args and avoid `pop_back()`Alex Turbov2021-08-031-6/+7
| | |
| * | Style: Better names of local vars for `IN_LIST` handlerAlex Turbov2021-08-031-4/+6
| | |
| * | Refactor: Extract the logic of testing for special variable to funcAlex Turbov2021-08-031-14/+27
| | | | | | | | | | | | Introduce `looksLikeSpecialVariable`. It makes `if()` blocks readable.
| * | Refactor: Introduce `cmArgumentList` container classAlex Turbov2021-08-032-154/+209
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Refactor: Reduce one more condition checking on handling math compareAlex Turbov2021-08-031-12/+13
| | |
| * | Refactor: Avoid `if` → `else if` → … for compare operatorsAlex Turbov2021-08-032-62/+104
| | | | | | | | | | | | | | | | | | 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.
| * | Refactor: Set `const` to `cmConditionEvaluator::IsKeyword` parametersAlex Turbov2021-08-032-3/+4
| | |
| * | Refactor: No need to move iterators after matchAlex Turbov2021-08-031-40/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Refactor: Replace `std::to_string` w/ more specialized `bool2string`Alex Turbov2021-08-031-5/+9
| | |