diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2021-02-24 10:05:30 +0100 |
|---|---|---|
| committer | Rémi Verschelde <rverschelde@gmail.com> | 2021-02-24 10:23:00 +0100 |
| commit | b97ef35585733474faf1b2de2f3b38c7eed26123 (patch) | |
| tree | bf6fcbb2a6a615d8814726efc049454422a36625 /tests/test_macros.h | |
| parent | e254715e796f108d6d3fffa24cad9be90d2926e6 (diff) | |
SCons: Properly handle overriding default values to bool options
The `dev=yes` and `production=yes` options work as aliases to set a number of
options, while still aiming to allow overriding specific options if the user
wishes so. (E.g. `production=yes use_lto=no` should work to enable production
defaults *but* disable LTO.)
That wasn't working as `ARGUMENTS.get()` returns a string and not a boolean as
expected by `BoolVariable`, and this wasn't flagged as a bug... So added a
helper method using SCons' `BoolVariable._text2bool` to do the conversion
manually.
Diffstat (limited to 'tests/test_macros.h')
0 files changed, 0 insertions, 0 deletions