[m-rev.] for post-commit review: store env var maps in options_variables

Julien Fischer jfischer at opturion.com
Tue Aug 31 20:14:40 AEST 2021



On Mon, 30 Aug 2021, Zoltan Somogyi wrote:

> For review by Julien.

> Include environment vars in options_variables.
> 
> compiler/options_file.m:
>     As above. This allows all lookup operations on options_variables
>     to stop requiring access to the I/O state.
> 
> compiler/make.build.m:
> compiler/make.m:
> compiler/make.program_target.m:
>     Stop passing now-unneeded I/O state pairs.
> 
> compiler/mercury_compile_main.m:
>     Stop passing now-unneeded I/O state pairs.
>
>     Do pass the complete environment variable map to the function
>     that initializes options_variables.

That looks fine.

Julien.


More information about the reviews mailing list