diff options
| author | Mats Wichmann <mats@linux.com> | 2024-09-11 18:12:14 (GMT) |
|---|---|---|
| committer | Mats Wichmann <mats@linux.com> | 2024-09-11 19:09:07 (GMT) |
| commit | 50bee2e7edcfd46dc21f28b131c6f1065ac44dcc (patch) | |
| tree | 63bdb02d079d98dfa28634ef1197e3c72c18c4f2 /RELEASE.txt | |
| parent | aad42dd6eeb9b1fe8756895f2567b25ef8e7d82f (diff) | |
| download | SCons-50bee2e7edcfd46dc21f28b131c6f1065ac44dcc.zip SCons-50bee2e7edcfd46dc21f28b131c6f1065ac44dcc.tar.gz SCons-50bee2e7edcfd46dc21f28b131c6f1065ac44dcc.tar.bz2 | |
Fix some AddOption issues
The optparse add_option method supports an additional calling style
that is not directly described in SCons docs, but is included
by reference ("see the optparse documentation for details"):
it takes a single arg consisting of a premade option object.
Because the optparse code detects that case based on seeing zero
kwargs, and we always add at least one (default=) that would fail
for AddOption. Fix for consistency, but don't advertise it further:
not addewd to manpage synoposis/description.
Signed-off-by: Mats Wichmann <mats@linux.com>
Diffstat (limited to 'RELEASE.txt')
| -rw-r--r-- | RELEASE.txt | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/RELEASE.txt b/RELEASE.txt index 196103c..05bdf7b 100644 --- a/RELEASE.txt +++ b/RELEASE.txt @@ -44,6 +44,11 @@ IMPROVEMENTS documentation: performance improvements (describe the circumstances under which they would be observed), or major code cleanups +- For consistency with the optparse "add_option" method, AddOption accepts + an SConsOption object as a single argument (this failed previouly). + Calling AddOption with the full set of arguments (option names and + attributes) to set up the option is still the recommended approach. + PACKAGING --------- |