<div dir="ltr"><div class="gmail_extra">Hi,</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 20, 2013 at 9:06 PM, Paul Bone <span dir="ltr"><<a href="mailto:paul@bone.id.au" target="_blank">paul@bone.id.au</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"><div class="im">On Fri, Apr 19, 2013 at 06:26:23PM +1000, Julien Fischer wrote:<br>

><br>
> Hi Paul,<br>
><br>
> On Thu, 18 Apr 2013, Paul Bone wrote:<br>
><br>
</div><div><div class="h5">>> --- /dev/null<br>
>> +++ b/mfilterjavac/MFILTERJAVAC_FLAGS.in<br>
>> @@ -0,0 +1,25 @@<br>
>> +@BOOTSTRAP_MC_ARGS@<br>
>> +--no-infer-all<br>
>> +--halt-at-warn<br>
>> +--no-warn-inferred-erroneous<br>
>> +--no-mercury-stdlib-dir<br>
>> +-I../library<br>
>> +-I../browser<br>
>> +-I../ssdb<br>
>> +--c-include-directory ../boehm_gc<br>
>> +--c-include-directory ../boehm_gc/include<br>
>> +--c-include-directory ../runtime<br>
>> +--c-include-directory ../library<br>
>> +--c-include-directory ../library/Mercury/mihs<br>
>> +--c-include-directory ../browser<br>
>> +--c-include-directory ../browser/Mercury/mihs<br>
>> +--c-include-directory ../ssdb<br>
>> +--c-include-directory ../ssdb/Mercury/mihs<br>
>> +--c-include-directory ../trace<br>
>> +--csharp-flag -keyfile:../mercury.snk<br>
>> +--no-java-classpath<br>
>> +--java-classpath ../library/mer_rt.jar<br>
>> +--java-classpath ../library/mer_std.jar<br>
>> +--java-classpath ../browser/mer_browser.jar<br>
>> +--java-classpath ../mdbcomp/mer_mdbcomp.jar<br>
>> +--config-file ../scripts/Mercury.config.bootstrap<br>
><br>
> There should also be:<br>
><br>
>     --erlang-include-directory ../library/Mercury/hrls<br>
><br>
<br>
</div></div>This isn't in some of the other directories, is it an error for it to be<br>
missing from slice/SLICE_FLAGS?<br>
<br>
Do I need more --erlang-include-directory options for each of the other<br>
directories such as browser and mdbcomp?<br></blockquote><div><br></div><div><div>Since you have not yet fixed bug #181, we have presumably never been</div><div>able to compile things that far in Erlang.  I think the stuff for the</div>
<div>Erlang backend only got added to directories where we could actually</div><div>build things.</div></div><div><br></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">

Unfortunatly knowing how to setup a new directory or make other sagnificant<br>
changes to the build system doesn't appear to be documented.</blockquote><div> </div><div><div>Beyond, add a Mmakefile and make sure it implements the usual targets,</div><div>e.g. install, depend, clean, realclean etc, what do you have in mind?</div>
<div>(Bear in mind, most of the existing directories are already treated differently</div></div><div>from each other.)</div><div><br></div><div>...</div><div><br></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">
<div><div class="h5">
<br>
</div></div>There are two types of errors that I'm reporting.<br>
<br>
    1. An IO error where mfilterjavac cannot read javac's output.  In which<br>
       case javac's output would be lost and the programmer would be unaware<br>
       of it if we didn't report the error.<br>
<br>
    2. A problem parsing the markers in the .java file.  I've found this<br>
       useful for debugging.  I'm happy to make this a warning rather than<br>
       an error so that it doesn't fail an otherwise good compilation<br></blockquote><div><br></div><div>I prefer the latter.</div><div><br></div><div>On thing I forgot to mention in my initial comments: you should update</div>
<div>the bootcheck script to handle mfilterjavac.  (It should be built in</div><div>stage 2 as we do with the profilers.)</div><div><br></div><div style>Cheers,</div><div style>Julien. </div></div></div></div>