summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Continue manpage cleanups [ci skip]Mats Wichmann2020-03-124-1052/+1032
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Eliminate some "note" wording in manpage. This is a "tone" thing, trying to avoid so many "asides", like "Note that blah blah", by trying to reword into a more regular flow, and occasionally dropping the aside if it doesn't seem quite suited to the manpage. More harmonization of docbook entity usage, cleanup of comments around generated sections, and other fiddling. <literallayout> around a code fragment was changed to <programlisting> <literallayout> around output was changed to <screen> If a <screen> section contained use input, that was marked with the <userinput> entity. These changes ought to also facilitate better conversion to other document markup formats, something which has been discussed. In the Description section, the remaining command-line discussion at the end was merged into the earlier wording about command-line processing, to improve the flow a bit. Some options entries were adjusted for appearance - indents, etc. Mention of Python 2.7 as a requirement is dropped. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3571 from mwichmann/test-fixtureWilliam Deegan2020-03-152-284/+358
|\ \ | | | | | | Fix some test fixture-handling nits.
| * | [PR #3571] more work on testing document [ci skip]Mats Wichmann2020-02-291-175/+174
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apply some of the ideas of the Python documentation style guide - headings, indents, etc. Fixed some lingering format problems. Add a table of contents (this will work on the wiki as long as we save the copy there in rest format instead of markdown - that is currently the case). Signed-off-by: Mats Wichmann <mats@linux.com>
| * | Add comments on skipping tests to framework doc [ci skip]Mats Wichmann2020-02-271-4/+27
| | | | | | | | | | | | | | | | | | | | | | | | Build up the skipping tests section by adding a bit of discussion on why you'd want to skip, and on not skipping too much by mocking parts of tests and organizing test code. Signed-off-by: Mats Wichmann <mats@linux.com>
| * | [PR #3571] fix error and improve docsMats Wichmann2020-02-272-50/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | sider spotted a cut-n-paste error introduced in the PR (srcdir set instead of srcfile) framework doc now uses directory consistently (over folder, which is a desktop concept rather than a filesystem concept) tweaked wording a bit more Signed-off-by: Mats Wichmann <mats@linux.com>
| * | Fix some test fixture-handling nits.Mats Wichmann2020-02-262-107/+156
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dir-fixture() did not concatenate a source list into a path as the testing doc says it should. fix-fixture() did not either, and the doc didn't say so; now it does (docstring and external testing doc). There was a way to slip through both dir_fixtures and file_fixture with the path invalid - if you have fixture dirs defined, and the dir/file is not found in them, you'll come out of the loop set to the last fixture dir and not try the current dir. This doesn't break anything, just leads to a somewhat misleading message if the fixture really isn't found - the traceback indicates the fixture was not found in whatever the last passed-in fixture dir was. Now it looks like it was not found in the source testing dir. The message can still be improved to be more descriptive. A couple of minor Py2 removals. Testing doc didn't mention FIXTURE_DIRS, so this was added. A bunch of other doc fiddling. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | Merge pull request #3574 from mwichmann/more-py2-cleanupsWilliam Deegan2020-03-156-96/+21
|\ \ \ | | | | | | | | Clean up some more Python 2 code
| * | | [PR #3574] fix sider complaintsMats Wichmann2020-03-032-2/+1
| | | | | | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * | | Clean up some more Python 2 codeMats Wichmann2020-03-036-96/+22
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | * _subproc is a little simpler as subprocess.DEVNULL is sure to br defined. Adjusted the docstring to better show the purpose. * CacheDir didn't need the Py2 branch of getting the config. * MSCommon can use a more specific exception. * packaging can now unconditionally use inspect.getfullargspec. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | Merge pull request #3577 from bdbaddog/add_msys2_mingw_pathsWilliam Deegan2020-03-152-0/+3
|\ \ \ | | | | | | | | Add msys2 installed mingw64 to PATH for mingw tool
| * | | [ci skip] update CHANGES.txt to have actual path addedWilliam Deegan2020-03-151-1/+2
| | | |
| * | | Add msys2 installed mingw64 to PATH for mingw toolWilliam Deegan2020-03-092-0/+2
| |/ /
* | | Merge pull request #3581 from mwichmann/more-uguideWilliam Deegan2020-03-153-65/+97
|\ \ \ | | | | | | | | Some user guide tweaks
| * | | Some user guide tweaks [ci skip]Mats Wichmann2020-03-123-65/+97
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | troubleshooting and caching sections got some improvements to the xml entities used, mainly for options. A few wording tweaks as well. added a blurb to troublseshooting to urge looking for simpler answers. scons.mod - options section changes to use <option> markup. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | Purge some py 2.7 codeWilliam Deegan2020-03-132-14/+1
| | |
* | | [ci skip] Update versionsWilliam Deegan2020-03-131-2/+4
| | |
* | | Merge branch 'master' of github.com:SCons/sconsWilliam Deegan2020-03-131-3/+3
|\ \ \ | |/ /
| * | Merge pull request #3578 from mwichmann/fix-docbuildWilliam Deegan2020-03-101-3/+3
| |\ \ | | |/ | |/| Fix syntax problem in Action doc [ci skip]
| | * Fix syntax problem in Action doc [ci skip]Mats Wichmann2020-03-091-3/+3
| |/ | | | | | | | | | | | | Recent change introduced an xml problem which prevents the docs from validating or building - transforming so it builds now. Signed-off-by: Mats Wichmann <mats@linux.com>
* | [ci skip] Updating SCons-local READMEWilliam Deegan2020-03-131-2/+3
|/
* Merge pull request #3563 from mwichmann/runtest-devmodeWilliam Deegan2020-02-251-2/+9
|\ | | | | Let test runner use new Dev. Mode
| * Merge branch 'master' into runtest-devmodeWilliam Deegan2020-02-2516-365/+645
| |\ | |/ |/|
* | Merge pull request #3570 from mwichmann/runtest-upgradeWilliam Deegan2020-02-251-148/+162
|\ \ | | | | | | runtest cleanups and subprocess update
| * | rutest cleanups and subprocess updateMats Wichmann2020-02-241-148/+162
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Clean up some things suggested by checkers. - Add a few docsstrings. - Make RuntestBase an abstract class - Eliminate Py2 remnants. - Use subprocess.run for a slightly cleaner interface - don't have to separately do wait or communicate, don't need to set up context manager, and everything returns cleanly in a subprocess.CompletedProcess. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | Merge pull request #3568 from datalogics-robb/masterWilliam Deegan2020-02-253-1/+29
|\ \ \ | |/ / |/| | Specify encoding for Java files, as the default varies
| * | Add a blurb about specifining encoding when opening Java source files as text.Robert Boehne2020-02-241-0/+4
| | |
| * | Specifiy encoding for Java files, as the default varies across platforms.Rob Boehne2020-02-242-1/+25
|/ /
* | Merge pull request #3566 from mwichmann/runtest-noexecWilliam Deegan2020-02-241-1/+4
|\ \ | | | | | | Fix runtest no-exec mode
| * | Fix runtest no-exec modeMats Wichmann2020-02-241-1/+4
|/ / | | | | | | | | | | | | | | Failure to default some values in class initializer caused a downstream problem if running no-exec mode, as one field was added only if running in execute mode. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3565 from bdbaddog/fix_IMPLICIT_COMMAND_DEPENDENCIES_TESTSWilliam Deegan2020-02-233-5/+9
|\ \ | | | | | | Fix implicit command dependencies tests
| * | Update description of IMPLICIT_COMMAND_DEPENDENCIES's functional changeWilliam Deegan2020-02-231-3/+4
| | |
| * | Reduce IMPLICIT_COMMAND_DEPENDENCIES's tried in test. No need to try all ↵William Deegan2020-02-232-2/+5
|/ / | | | | | | values, only new functionality. Full range is covered in other test
* | Merge pull request #3556 from mwichmann/substfilesuffixWilliam Deegan2020-02-217-6/+84
|\ \ | | | | | | Change errant TEXTFILESUFFIX to SUBSTFILESUFFIX
| * \ Merge branch 'master' into substfilesuffixWilliam Deegan2020-02-21129-1057/+831
| |\ \ | |/ / |/| |
* | | Merge pull request #3483 from grossag/topic/grossag/compoundactionscanWilliam Deegan2020-02-217-211/+362
|\ \ \ | | | | | | | | Add support for scanning multiple entries in an action string
| * \ \ Merge branch 'master' into topic/grossag/compoundactionscanWilliam Deegan2020-02-21295-2846/+3855
| |\ \ \ | |/ / / |/| | |
| * | | Change IMPLICIT_COMMAND_DEPENDENCIES meaning as requested in PRAdam Gross2020-01-135-26/+84
| | | |
| * | | Address review feedbackAdam Gross2020-01-126-20/+27
| | | |
| * | | Merge branch 'master' into topic/grossag/compoundactionscanAdam Gross2020-01-10320-2572/+3212
| |\ \ \
| * | | | Update scanning code to get it closer to desired stateAdam Gross2020-01-101-19/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes: 1. Remove repository code. 2. Only use env.WhereIs() if it's the first entry in a group. 3. Avoid duplicate sources if they are in another variable (e.g. LIBS).
| * | | | Add repository-related test coverage of the new codeAdam Gross2019-12-042-180/+180
| | | | | | | | | | | | | | | | | | | | That was missing in the codecov report.
| * | | | Fix some more testsAdam Gross2019-12-032-27/+15
| | | | | | | | | | | | | | | | | | | | Reverts some of the changes I made previously in my diff.
| * | | | Switch heavyweight scanning to be opt-inAdam Gross2019-12-032-14/+35
| | | | |
| * | | | Fix some testsAdam Gross2019-12-031-5/+14
| | | | |
| * | | | Various improvements to the changeAdam Gross2019-12-027-43/+79
| | | | |
| * | | | Add support for scanning multiple entries in an action stringAdam Gross2019-12-023-35/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds support for scanning multiple entries in an action string in order to better support the following use cases: 1. A file is provided in an action string and should be taken as a dependency. For example, an action string "$PERL somefile.pl". 2. An action string actually has two actions separated by &&. For example, "cd <some_dir> && $ZIP <args>". Adding support for #1 actually allows us to fix the test IMPLICIT_COMMAND_DEPENDENCIES.py on Windows, which was previously treating a Python file as executable even on Windows. This was causing tests to repeatedly open the default handler of Python files, which if set to Visual Studio causes DDE hangs. This test is fixed because now we can have the action string specify python as the first command and still take an implicit dependency on the script, which is now the second command.
| | | * | Merge pull request #3 from bdbaddog/substfilesuffixMats Wichmann2020-02-195-5/+82
| | | |\ \ | | | | | | | | | | | | Add testcase for GitHub Issue #3540. …
| | | | * | Add testcase for GitHub Issue #3540. Update tests for 3550 to allow 3540 to ↵William Deegan2020-02-165-5/+82
| | | |/ / | | | | | | | | | | | | | | | use some of it's test files
| | | * | Change errant TEXFTILESUFFIX to SUBSTFILESUFFIXMats Wichmann2020-02-152-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In one location a copy-paste apparently didn't get fixed up, and the wrong construction var was used. Fixes #3540 Signed-off-by: Mats Wichmann <mats@linux.com>
| | | | * Change devmode check to not exclude 3.7.0Mats Wichmann2020-02-201-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>