summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* fix copyright and other dates in users guide4.9.0William Deegan2025-03-022-5/+5
|
* updatesWilliam Deegan2025-03-023-5/+5
|
* minor tweaks to RELEASE.txtWilliam Deegan2025-03-021-0/+3
|
* updates for releaseWilliam Deegan2025-03-021-5/+5
|
* Updates for 4.9.0 releaseWilliam Deegan2025-03-024-43/+23
|
* Regenerated docs for 4.9.0 release.William Deegan2025-03-024-28/+28
|
* Update NEXT_RELEASE to 4.9.0William Deegan2025-03-0214-25/+25
|
* Merge pull request #4689 from mwichmann/test/fixture-destWilliam Deegan2025-02-282-0/+7
|\ | | | | Framework: fixture methods now handle lists for dstdir
| * Framework: fixture methods now handle lists for dstdirMats Wichmann2025-02-282-0/+7
|/ | | | | | | | | | | | | 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>
* Merge pull request #4684 from simpkins/fix_testsWilliam Deegan2025-02-241-1/+8
|\ | | | | Fix running individual test files when ninja is not installed
| * Fix running individual test files when ninja is not installedAdam Simpkins2025-02-231-1/+8
|/ | | | | | | | | | | | | 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.
* Merge pull request #4681 from mwichmann/doc/overviewWilliam Deegan2025-02-173-55/+75
|\ | | | | Update doc development overview
| * Update doc development overview [ci skip]Mats Wichmann2025-02-173-55/+75
| | | | | | | | | | | | | | | | 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>
* | Merge pull request #4680 from mwichmann/test/yacc-protoWilliam Deegan2025-02-172-2/+3
|\ \ | |/ |/| Fix small problem in a yacc test
| * Fix small problem in a yacc testMats Wichmann2025-02-172-2/+3
|/ | | | Signed-off-by: Mats Wichmann <mats@linux.com>
* Merge pull request #4675 from mwichmann/project/overviewWilliam Deegan2025-02-152-19/+77
|\ | | | | Some tweaks to project description docs
| * Some tweaks to project description docsMats Wichmann2025-01-262-19/+77
| | | | | | | | | | | | | | The CONTRIBUTING page and the documentation overview page are updated with additional infomration and some rewording. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #4676 from rdbisme/extra-libsWilliam Deegan2025-02-1514-26/+245
|\ \ | | | | | | :bug: Allow passing `extra_libs` to `CheckLibWithHeader`
| * | Add note to docstring that this is a live test per mwichmannWilliam Deegan2025-02-151-0/+2
| | |
| * | SimplifyWilliam Deegan2025-02-151-2/+0
| | |
| * | Simplify use of dir_fixtureWilliam Deegan2025-02-151-1/+1
| | |
| * | Simplify use of dir_fixtureWilliam Deegan2025-02-151-2/+0
| | |
| * | Simplify use of dir_fixture and some test logic. Added ↵William Deegan2025-02-152-11/+4
| | | | | | | | | | | | DefaultEnvironment(tools=[]) to speed up on windows
| * | Per request, "fixture-ize" CheckLibWithHeader_extra_libs testMats Wichmann2025-02-1310-120/+125
| | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * | Windows-ify the new CheckLibWithHeader testMats Wichmann2025-02-091-15/+67
| | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * | Fix CheckLibWithHeader test - undo mistakeMats Wichmann2025-02-081-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | "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>
| * | Minor fixes to CheckLibWithHeader testMats Wichmann2025-02-081-8/+43
| | | | | | | | | | | | | | | | | | | | | | | | 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>
| * | Also enable extra_libs for CheckLibMats Wichmann2025-02-075-32/+70
| | | | | | | | | | | | | | | | | | Manpage entries and docstrings updated with version-added specifiers. Signed-off-by: Mats Wichmann <mats@linux.com>
| * | :memo: Add change summaryRuben Di Battista2025-01-263-0/+6
| | |
| * | :white_check_mark: Add testRuben Di Battista2025-01-261-0/+94
| | |
| * | :bug: Allow passing `extra_libs` to `CheckLibWithHeader`Ruben Di Battista2025-01-261-3/+3
| |/
* | Merge pull request #4677 from mwichmann/issue/debug-memoizerWilliam Deegan2025-02-034-29/+39
|\ \ | |/ |/| Handle --debug containing memoizer
| * Handle --debug containing memoizerMats Wichmann2025-02-034-29/+39
|/ | | | | | | | | | | 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>
* Merge pull request #4674 from ↵William Deegan2025-01-214-2/+28
|\ | | | | | | | | bdbaddog/fix_2281_Aliases_ignore_pre_post_add_actions Fix 2281 aliases ignore pre post add actions
| * [ci skip] Fix typo in commentsWilliam Deegan2025-01-211-1/+1
| |
| * Fix call to SCons.Node.Node.build()William Deegan2025-01-201-1/+1
| |
| * Update CHANGES/RELEASEWilliam Deegan2025-01-202-0/+4
| |
| * Fix Issue #2281, post and pre actions were being ignored when added to an ↵William Deegan2025-01-202-2/+24
|/ | | | Alias() node, if there was no action specified when that node was created
* [ci skip] add test framework reformat to .git-blame-ignore-revsWilliam Deegan2024-12-301-0/+2
|
* Merge pull request #4669 from mwichmann/framework-blackWilliam Deegan2024-12-2912-1288/+1875
|\ | | | | test framework: bulk reformat
| * Rework a TestCommon test to look less ugly [skip appveyor[Mats Wichmann2024-12-291-13/+13
| | | | | | | | | | | | | | | | | | 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>
| * test framework: bulk reformatMats Wichmann2024-12-2912-1280/+1867
|/ | | | | | Imcludes test snippets contained in docstrings, too. Signed-off-by: Mats Wichmann <mats@linux.com>
* Merge pull request #4665 from mwichmann/doc/source-subst-specialWilliam Deegan2024-12-296-31/+58
|\ | | | | Update docs on special method evaluation
| * Fix a review comments on special methods [skip appveyor]Mats Wichmann2024-12-222-5/+11
| | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * Update docs on special method evaluationMats Wichmann2024-12-166-33/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | Merge pull request #4670 from bdbaddog/fix_help_local_onlyWilliam Deegan2024-12-296-66/+50
|\ \ | | | | | | Fix Help()'s local_only arg to match documents.
| * | Fixed mis-match between code and manpage for Help()'s local_only argument. ↵William Deegan2024-12-226-66/+50
| | | | | | | | | | | | Now code matches the documentation. Additionally fixturized the Help test
| * | Fixed mis-match between code and manpage for Help()'s local_only argument. ↵William Deegan2024-12-221-0/+0
|/ / | | | | | | Now code matches the documentation. Additionally fixturized the Help test
* | Merge pull request #4668 from adamscott/template-pool-to-scons_poolWilliam Deegan2024-12-173-1/+10
|\ \ | | | | | | Set TEMPLATE rule to `install_pool` for the Ninja tool
| * | Set TEMPLATE rule to `install_pool` for the Ninja toolAdam Scott2024-12-172-2/+7
| | |