[m-rev.] for review: include Mmake.workspace and Mmake.params next to each other

Julien Fischer jfischer at opturion.com
Tue Aug 12 14:26:48 AEST 2025


On Tue, 12 Aug 2025 at 06:45, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:

> Include Mmake.params and Mmake.workspace next to each other.
>
> Mmake.common.in:
>     Before this diff, there were a whole bunch of definitions between
>     the inclusion of Mmake.workspace and the inclusion of Mmake.params.
>     Move all those definitions elsewhere. Specifically, move the definitions
>     whose right hand sides are autoconfigured values, and which are NOT
>     also set by Mmake.workspace, to before both inclusions, moving related
>     non-autoconfigure variable definitions with them, while moving
>     other definitions to after both inclusions.
>
>     Document the cause of the "phasing problem" that kept the inclusion
>     of Mmake.params after the inclusion of Mmake.workspace (the definition
>     of MC), and a possible solution.
>
> tools/stdlines_for_sh:
>     Add a version of stdlines for sh scripts.

That looks fine.

Julien.


More information about the reviews mailing list