diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2022-09-12 07:55:58 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2022-09-12 08:34:38 +0200 |
commit | 06c7e10f20cc1f2ade57914c9ec0c28afdc410aa (patch) | |
tree | c14c52d42c1b1beb4386ab6b4aaead9b6f163614 /libbuild2/build/script/parser+command-if.test.testscript | |
parent | 05cba8680738c754d6ca46b8ead5030f633295b2 (diff) |
Do not treat primary ad hoc group member as group for variable lookupadhoc-vars
Note that we started with this semantics but it was changed in a commit on
2021-09-16 for reasons not entirely unclear but most likely due to target-
specific variables specified for the group not being set on all the members.
Which we have now addressed (see the previous commit).
Note also that this new (old) semantics is not without its own drawbacks.
Specifically, there is a bit of waste when the target-specific variable is
really only meant for the recipe and thus setting it on all the members is
unnecessary. For example:
<{hxx ixx cxx}{options}>: cli{options}
{
options = ...
}
{{
# Use options.
}}
But this feels like a quality of implementation rather than conceptual
issue. For example, we could likely one day address it by synthesizing a
separate group target for ad hoc groups.
Diffstat (limited to 'libbuild2/build/script/parser+command-if.test.testscript')
0 files changed, 0 insertions, 0 deletions