| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |\
| |
| | |
Framework: fixture methods now handle lists for dstdir
|
| |/
|
|
|
|
|
|
|
|
|
|
|
| |
TestCmd.dir_fixture and TestCmd.file_fixture are described as accepting a
list for both srcdir and dstdir. For example, the docstring for `dir_fixture`:
srcdir or dstdir may be a list, in which case the elements are first
joined into a pathname.
However, the implementation only handled this for srcdir/srcfile.
Added the same stanza for dstdir/dstfile..
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\
| |
| | |
Fix running individual test files when ninja is not installed
|
| |/
|
|
|
|
|
|
|
|
|
|
|
| |
The code in testing/framework/TestSCons.py attempted to handle an
ImportError if ninja is not available. However, when running individual
test files from the scons/test/ directory, this directory is included as
the first entry in sys.path. When this happens, the `import ninja`
statement succeeds, finding the scons/test/ninja/ directory and treating
it as a package. This results in an AttributeError being thrown later
attempting to access `ninja.BIN_DIR`, rather than an ImportError.
I have confirmed that this change now allows `./runtest.py test/Help.py`
to succeed, even when ninja is not installed.
|
| |\
| |
| | |
Update doc development overview
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Includes updates to the diagram that shows the doc flow
(although it could still be improved. I'm not a diagramming
expert so settling for what I could accomplish)
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\ \
| |/
|/| |
Fix small problem in a yacc test
|
| |/
|
|
| |
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\
| |
| | |
Some tweaks to project description docs
|
| | |
| |
| |
| |
| |
| |
| | |
The CONTRIBUTING page and the documentation overview page
are updated with additional infomration and some rewording.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\ \
| | |
| | | |
:bug: Allow passing `extra_libs` to `CheckLibWithHeader`
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
DefaultEnvironment(tools=[]) to speed up on windows
|
| | | |
| | |
| | |
| | | |
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| | | |
| | |
| | |
| | | |
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
"Muscle memory" had me add calls to skip tools for DefaultEnvironment,
but the two library builds use it, so that was just plain wrong.
Restored prior state.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Add boilderplate header/footer, fix one place where the wrong
(non-portable) name for a library was used. Still does not
build on Windows, due to mismatch between .dll/.lib construction.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Manpage entries and docstrings updated with version-added specifiers.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| | | | |
|
| | | | |
|
| | |/ |
|
| |\ \
| |/
|/| |
Handle --debug containing memoizer
|
| |/
|
|
|
|
|
|
|
|
|
| |
The memoizer statistics have to be handled specially as they use
conditional decorators, those have to be enabled before any of the
decorated methods are read by Python. This worked if there was exactly
"--debug=memoizer" on the commandline or in SCONSFLAGS, but not if it
was in a multi-value option like "--debug=presub,memoizer".
Handle the up-front-check a little more completely to fix this.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\
| |
| |
| |
| | |
bdbaddog/fix_2281_Aliases_ignore_pre_post_add_actions
Fix 2281 aliases ignore pre post add actions
|
| | | |
|
| | | |
|
| | | |
|
| |/
|
|
| |
Alias() node, if there was no action specified when that node was created
|
| | |
|
| |\
| |
| | |
test framework: bulk reformat
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Insted of really long strings of repeated characters, use f-strings to
compose the "expected" output for the banner function tests. This keeps
the code formatter from breaking things an ugly way, and is really more
"accurate" anyway.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |/
|
|
|
|
| |
Imcludes test snippets contained in docstrings, too.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\
| |
| | |
Update docs on special method evaluation
|
| | |
| |
| |
| | |
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The User Guide uses an example (in the Command chapter) of making a target
name out of the source name, a special attribute, and concatenation
with a new suffix. The special attribute part is not something that has
been introduced. Some searching suggests it's never actually described in
the User Guide, though .file, .abspath and .srcdir are used in examples.
The attribute used, .basename, doesn't actually exist - there's a
.base and a .filebase. Furthermore, the substitution suggested doesn't
work. Expansion of special variables like $SOURCE into nodes is deferred -
see the docstring of SCons.Subst.NLWrapper - so the internal expansion
ends up trying to lookup the attribute on a string, which fails with
an AttributeError. The way the user guide entry is written, it was
not actually evaluated: it was described as an <scons_example>, but
an incomplete one, and since there was no corresponding <scons_output>
the problem was not detected.
The changes fix up the example to have it use an existing attribute
(.base) and do File() on the source, and add a sidebar to provide a
bit of an explanation so this isn't just "magic". A subsequent example
(ex4, which I added) is dropped as it doesn't add enough value by itself,
and the final example (formerly ex5, renamed to ex3) now includes this
substitution so it's actually run by the doc machinery, and can be
seen to be working correctly.
This finally fixes #2905, which was closed in 2018 as having been addressed,
though the non-working example actually remained. The issue is
also mentioned in #4660, which is not resolved by changing the docs -
leaving that open, at least for the time being.
Signed-off-by: Mats Wichmann <mats@linux.com>
|
| |\ \
| | |
| | | |
Fix Help()'s local_only arg to match documents.
|
| | | |
| | |
| | |
| | | |
Now code matches the documentation. Additionally fixturized the Help test
|
| |/ /
| |
| |
| | |
Now code matches the documentation. Additionally fixturized the Help test
|
| |\ \
| | |
| | | |
Set TEMPLATE rule to `install_pool` for the Ninja tool
|
| | | | |
|