<div dir="ltr">[CC vm-dev]<br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 6, 2017 at 7:04 PM, K K Subbu <span dir="ltr"><<a href="mailto:kksubbu.ml@gmail.com" target="_blank">kksubbu.ml@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On Friday 05 May 2017 02:11 PM, Andrei Chis wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
...<span class="gmail-"><br>
./pharo -help<br>
unknown option: -help<br>
Common <option>s:<br>
  --help                 print this help message, then exit<br>
<br>
Then `./pharo -help` returns an exit code of 1 on mac/linux while<br>
`./pharo --help` returns a exit code of 0 on mac and 1 on linux.<br>
</span></blockquote>
<br>
It is --help on Pharo and -help on other compiles (see below). Can anyone shed light on the legacy compatibility comment in :<br>
<br>
---- platforms/unix/vm/sqUnixMain.c ----<br>
1421   /* legacy compatibility */            /*** XXX to be removed at some time ***/<br>
1422<br></blockquote><div><br></div><div>Looking just at just the file, it appears this is comment is strongly associated with the code below,</div><div>but git blame [1] indicates the comment is 7 years old, while the code below is 7 months old.  ??? </div><div><br></div><div>[1] <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/blame/995e7aedc3b5aeb76904f92abbd8a7a1bcf8138f/platforms/unix/vm/sqUnixMain.c">https://github.com/OpenSmalltalk/opensmalltalk-vm/blame/995e7aedc3b5aeb76904f92abbd8a7a1bcf8138f/platforms/unix/vm/sqUnixMain.c</a> </div><div><br></div><div>cheers -ben</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
1423 #ifdef PharoVM<br>
1424 # define VMOPTION(arg) "--"arg<br>
1425 #else<br>
1426 # define VMOPTION(arg) "-"arg<br>
1427 #endif<span class="gmail-"><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
It's still not clear why running the vm to print the help of the version<br>
should exit with an error code.<br>
</blockquote>
<br></span>
Agreed. I think --help and --version should return 0. I think the 1 was to catch incorrect/unknown arguments.<br>
<br>
If I learn the rationale behind the change, I can propose a patch to fix this.<br>
<br>
Regards .. Subbu<br>
<br>
<br>
</blockquote></div><br></div></div>