summaryrefslogtreecommitdiffstats
path: root/src/engine/SCons
Commit message (Collapse)AuthorAgeFilesLines
...
* | LaTeX scanner: Find > 1 includes per lineLukas Schrangl2019-07-022-1/+16
|/ | | | | | | | "^[^%\n]*" at the beginning of the reg ex would match all but the last include so that they were lost. There is no point checking for "%" since comments are stripped anyways, so just remove that part. Also add test case for multiple includes per line.
* Fix sider warningWilliam Deegan2019-06-201-1/+1
|
* Allow MSVCUnsupportedTargetArch and MSVCUnsupportedHostArch exceptions to ↵William Deegan2019-06-202-1/+4
| | | | propagate and cause SCons to exit
* Switch back to None with proper comparison using is instead of notWilliam Deegan2019-06-191-4/+4
|
* Switch from testing for directory value being false to -1 as "" evaluates to ↵William Deegan2019-06-191-4/+9
| | | | false
* Fix conflictWilliam Deegan2019-06-181-5/+0
|
* Added tests to logic changes by mhqtronic PR #3375William Deegan2019-06-182-17/+205
|
* Merge pull request #3383 from chasinglogic/action-hides-exceptionsWilliam Deegan2019-06-132-3/+21
|\ | | | | Don't hide exceptions when command does not exist
| * Don't hide exceptions when command does not existMathew Robinson2019-06-102-3/+21
| |
* | Use more idiomatic PythonMats Wichmann2019-06-061-3/+3
|/ | | | | | | | Minor: replace an instance of "for i in range(len(foo))" with the now preferred "for i in foo". This change didn't naturally fit with any of the other instances of this usage, so is submitted by itself. Signed-off-by: Mats Wichmann <mats@linux.com>
* PY38: Fix invalid escape chars in compare stringWilliam Deegan2019-05-301-26/+27
|
* PY38: Fix invalid escape chars in compare stringWilliam Deegan2019-05-301-4/+4
|
* PY38: Resolve duplicate creation of 'work' subdir which yielded warnings. ↵William Deegan2019-05-301-1/+3
| | | | Also improve exception handling message in test.subdir()
* PEP8 fileWilliam Deegan2019-05-301-326/+374
|
* Merge pull request #3367 from mwichmann/docs-parse_flagsWilliam Deegan2019-05-281-10/+12
|\ | | | | Update docs for parse_flags keywords
| * Update docs for parse_flags keywordsMats Wichmann2019-05-141-10/+12
| | | | | | | | | | | | | | | | | | The somewhat poorly named parse_flags keyword args actually work like the MergeFlags method - don't just split like the ParseFlags method, but do the merging as well. Tweak the docs and add a reference to env.MergeFlags. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Add cache hit rate to cache debuggingMathew Robinson2019-05-171-0/+14
|/
* Fix some regexes for Python 3.8 complaintsMats Wichmann2019-05-133-3/+3
| | | | | | | | | | | Regexes that contained unescaped backslashes and were not listed in raw string form caused one more test failure when Python 3.8 was experimentally turned on in the Travis CI build. Also one utility script had the same, not affecting tests - found through inspection. Signed-off-by: Mats Wichmann <mats@linux.com>
* [PR #3363] doc update for added flagsMats Wichmann2019-05-041-0/+3
| | | | | | Needed a doc regen to pick up the change. Signed-off-by: Mats Wichmann <mats@linux.com>
* Add -iquote and -idirafter for CCFLAGSMats Wichmann2019-05-022-8/+22
| | | | | | | | | | | Recognize two additional GNU compiler header directory search options: -iquote and -idirafter. Each takes a following arg, which scons now recognizes and adds together with the option to CCFLAGS. Note that (similar to -isystem which was added in git commit f8614aa2), this does not tell scons anything special, it only recognizes the flag + argument so it can be passed on to the compiler. Signed-off-by: Mats Wichmann <mats@linux.com>
* Fix a small error in docbook toolMats Wichmann2019-04-301-1/+1
| | | | | | | The routine used in the builder if lxml is being used had a doubled write command likely to produce unexpected results. Signed-off-by: Mats Wichmann <mats@linux.com>
* Fix misplaced file closingMats Wichmann2019-04-281-2/+7
| | | | | | | | | One file close ended up in the wrong method, meaning it was closing a file which needed to stay open for further reading, while another method of the same name in a different class was missing one. (this didn't trigger any test fails, hmmm, just visual inspection) Signed-off-by: Mats Wichmann <mats@linux.com>
* Merge pull request #3353 from mwichmann/cacheDirWilliam Deegan2019-04-281-13/+75
|\ | | | | [WIP] Avoid cachedir races
| * [PR #3353] fix sider nitpickeryMats Wichmann2019-04-281-4/+4
| | | | | | | | | | | | | | | | sider complained about spelling in a comment after I updated the comment so it got to look at those lines as part of the change. Fix those. Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PR #3353] tweaks per reviewMats Wichmann2019-04-281-10/+9
| | | | | | | | | | | | | | | | | | Simplify the Py2 check for an existing-but-old cachedir by using any and a generator expression. Move an import to module scope. Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PR #3353] quiet sider complaint about bare except:Mats Wichmann2019-04-281-2/+2
| | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * Avoid cachedir racesMats Wichmann2019-04-281-6/+69
| | | | | | | | | | | | | | | | | | 1. Add a Py3-only version of the cachedir config read, using exclusive open to avoid races. 2. In the Py2-only version, add the hack from issue #3351 to dodge one source of races. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3345 from mwichmann/py38warns4-testsWilliam Deegan2019-04-2825-199/+250
|\ \ | |/ |/| [wip] Py38warns4 tests
| * [#PR 3345] clean up some sider complaintsMats Wichmann2019-04-272-3/+1
| | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PR #3345] fix more PY3.8 failsMats Wichmann2019-04-264-16/+21
| | | | | | | | | | | | | | | | | | | | | | File closes in msvs tool Context manager in rpm tool Dummy compiler, assembler scripts can be called in unexpected ways (namely by gcc tool init), which passes --version flag. don't take exception on the getopt call. Try again to undo my breakage of _subproc, which was failing on Win+PY27 Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PR #3345] fix a few more Py3.8 problemsMats Wichmann2019-04-252-5/+12
| | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PR #3345] fix flake8 compliants on the PRMats Wichmann2019-04-252-2/+2
| | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PY 3.8] test fixes for file closings, rawstringsMats Wichmann2019-04-258-101/+126
| | | | | | | | | | | | | | On a linux host (missing some things that may be on the Travis CI setup), Py3.8a3 now shows 19 fails, 1048 pass, with 84 Warning: messages. Signed-off-by: Mats Wichmann <mats@linux.com>
| * [PY 3.8] file closes in tools and othersMats Wichmann2019-04-2516-92/+108
| | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3360 from mwichmann/isnotWilliam Deegan2019-04-2833-154/+158
|\ \ | | | | | | Some more lint-derived cleanups
| * | Some more lint-derived cleanupsMats Wichmann2019-04-2733-154/+158
| |/ | | | | | | | | | | | | | | | | | | | | | | Consistently use "not is" and "not in", many instances used the form "not x is y" instead, which pylint objected to. A couple of bare except clauses got a qualifier. Files otherwise touched had trailing whitespace cleaned up as well. These are all things that sider would complain about if a change happened nearby, so this is pre-emptive. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3341 from bdbaddog/enhanced_debug_explainWilliam Deegan2019-04-281-4/+15
|\ \ | | | | | | Enhanced debug explain
| * \ Merge branch 'master' into enhanced_debug_explainWilliam Deegan2019-04-272-0/+27
| |\ \
| * | | switch to using zip_longest and handle py27 naming it izip_longest from ↵William Deegan2019-04-271-3/+8
| | | | | | | | | | | | | | | | ampping None to allow for explain to handel old and new dependency lists of different lengths
| * | | Merge branch 'master' into enhanced_debug_explainWilliam Deegan2019-04-2643-416/+596
| |\ \ \ | | | |/ | | |/|
| * | | Enhanced --debug=explain output. Breaks out sources, depends, and implicit ↵William Deegan2019-04-021-4/+10
| | | | | | | | | | | | | | | | components to dependency list. Updated test. TODO: better formatting
* | | | Merge pull request #3359 from bdbaddog/fortran_issue_3135William Deegan2019-04-274-83/+216
|\ \ \ \ | |_|_|/ |/| | | Fix Issue #3135 - Type Bound procedures in Fortran submodules
| * | | Also ignore PURE and ELEMENTAL after MODULE in Scanner and Emitter.Peter Diener2019-04-262-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add subroutines that are declared pure and elemental to the test of the Emitter. Note that in test_1.f90, the interface is repeated in the submodule, whereas in test_2.f90 the interface is taken from the module. Also note that the regex does not check whether "module pure" or "module elemental" is actually followed by "subroutine" or "function". This would be a syntax error and should trigger a compile time error.
| * | | Fix Issue #3135 - Also add tests to check that emitter is properly ↵William Deegan2019-04-252-3/+129
| | | | | | | | | | | | | | | | processing interface module declarations
| * | | PEP8 plus switch to unittest.main() when running directlyWilliam Deegan2019-04-251-71/+69
| | | |
| * | | Use raw strings in regexes.Peter Diener2019-03-062-4/+4
| | | |
| * | | Fix issue #3135.Peter Diener2019-03-042-13/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes issue #3135 regarding the issue with using type bound procedures in Fortran submodules. The fix consists of changing the regex used in the scanner and the emitter to ignore lines starting with: module subroutine and module function as these are used to define type bound procedures instead of modules named 'subroutine' or 'function'. The regex is case insensitive.
* | | | Merge branch 'master' into ↵William Deegan2019-04-2643-416/+596
|\ \ \ \ | | |_|/ | |/| | | | | | fix_mongo_bug_33111_multiple_targets_via_emitter_spurious_rebuilds
| * | | [PR #3337] fix two minor Sider complaintsMats Wichmann2019-04-241-2/+1
| | | | | | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * | | [PR #3337] test case now uses mocked envMats Wichmann2019-04-241-24/+42
| | | | | | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>