Mats Wichmann

Was just poking at the FortranCommon part of this equation the other day, but I'm not sure the Intel variant is plugged in quite the same way. Be lovely if...

We should perhaps look at that. I'd like to add to the initial comment: > If a GCC compile command line has one or more `-include foo` options, no dependency...

Just a note on history: the patch referenced here was applied in 2009. Knowing this, I'm removing the "Has Patch" label (which I added recently), because it doesn't have a...

Finally got around to looking at this. I think some kind of indicator is valuable, but I don't necessarily like the current format. We go from a standard Dump output...

I'll add json-format irritant #3: the `CLVar` class has a `__str__` method that lets it render its contents, but json serialization doesn't do that either, so we get ``` "SHCCFLAGS":...

Here's a testcase diff for the current PR implementation: ```py diff --git a/SCons/EnvironmentTests.py b/SCons/EnvironmentTests.py index c40439ae0..8a385c1bc 100644 --- a/SCons/EnvironmentTests.py +++ b/SCons/EnvironmentTests.py @@ -3177,11 +3177,12 @@ def generate(env): def test_Dump(self) ->...

> That above example just compares `pprint` to this PR's implementation, the _live_ `json` output is even less helpful: > > ```json > "File": "Variable_Method_Caller", > ``` > > But...

Today I Learned: the Python stdlib json encoder is actually designed to be extensible, so we could easily improve the presentation of the two problems I mention above, `BuilderDict` and...

Flagging as doc issue for now, and assigning so I take a look.

I don't think the docs imply this. They're just lists (well, they're `TSList`, which is a subclass of `UserList`), and don't provide any unusual attributes. Individual *nodes* in the lists,...