Skip to content
This repository was archived by the owner on Sep 3, 2024. It is now read-only.

Commit e38062a

Browse files
author
William McLendon
committed
Update CHANGELOG
1 parent f6dbbb6 commit e38062a

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

CHANGELOG.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
2020
## [0.3.1] <!-- YYYY-MM-DD --> [Unreleased]
2121
#### Added
2222
#### Changed
23+
- Changes to `opt-set-cmake-var` behaviour:
24+
- Trigger a WARNING if `PARENT_SCOPE` and a _TYPE_ is provided since CMake
25+
requires TYPED vars to be CACHE vars and `PARENT_SCOPE` will force the
26+
var to be considered a non CACHE var that would store the parameters
27+
before `PARENT_SCOPE` in the call as _list_ entries.
28+
- Trigger an exception if `PARENT_SCOPE` and `FORCE` are both provided since
29+
`FORCE` is only valid for CACHE variables but `PARENT_SCOPE` is only valid
30+
for non-CACHE vars (and as noted above will _force_ the `set()` operation
31+
to be a non-CACHE var which then triggers a CMake error because `FORCE` is
32+
invalid to that form of `set()`.
33+
- **bash** generated options that have a TYPE and `PARENT_SCOPE` are processed
34+
to generate the same _list_ string that would be generated by the matching
35+
`set()` operation (with the warning).
2336
- Revert(ish) the modifications from MR12 which removed the error when generating
2437
BASH output if there is an unhandled CMake variable left hanging around.
2538
Rather than make this an unavoidable error though, ExpandVarsInText now inherits

0 commit comments

Comments
 (0)