[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