<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Aug 19, 2018 at 1:32 PM, David T. Lewis <span dir="ltr"><<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br>
On Sun, Aug 19, 2018 at 10:31:15AM -0700, Fabio Niephaus wrote:<br>
>  <br>
> @akgrant43 also thinks a command line flag would be the way to go here (see <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/279#issuecomment-414133977" rel="noreferrer" target="_blank">https://github.com/<wbr>OpenSmalltalk/opensmalltalk-<wbr>vm/pull/279#issuecomment-<wbr>414133977</a>).<br>
<span class="">> <br>
> Here's one last argument against it that I can think of: the option parser fails on unknown options and that means that CI scripts can only use, e.g., a `-suppress*` option iff supported by the vm, otherwise it will fail. The only way to make that work is to check the version of the vm before opening an image, which in turn adds complexity to CI scripts.<br>
> <br>
<br>
</span>For the unix VM, command line arguments are (or should be) supported by<br>
environment variables. See function vm_parseEnvironment() for environment<br>
variables that match with various corresponding command line arguments.<br>
<br>
If you follow this pattern, then you woould define a new command line parameter<br>
as previously suggested, and also a new environment variable to do the same<br>
thing. With that approach, you can use the environment variable that does<br>
what you want, rather than adding special-case handling for ${CI}.<br></blockquote><div><br></div><div>Good suggestion David.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Dav<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
</div></div>