<div dir="ltr"><div dir="ltr">On Tue, May 28, 2024 at 9:09 AM Magnus Ihse Bursie <<a href="mailto:magnus.ihse.bursie@oracle.com">magnus.ihse.bursie@oracle.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>

  
  <div>
    <p>On 2024-05-07 06:04, Jiangli Zhou wrote:</p>
    <blockquote type="cite">
      <pre><blockquote type="cite"><pre>I did think I correctly changed every dynamic check that you had added
to a compile-time check, so it bewilders me somewhat when you say that
jvm.cfg is not needed in your branch.

Can you verify and confirm that the static launcher actually works in
your branch, if there is no "lib/jvm.cfg" present?
In my <path>/leyden/build/linux-x86_64-server-slowdebug/images/jdk directory:
</pre>
</blockquote>
$ mv lib/jvm.cfg lib/jvm.cfg.no_used
$ find . | grep jvm.cfg
./lib/jvm.cfg.no_used

$ bin/javastatic -cp <my_jar> HelloWorld
HelloWorld
</pre>
    </blockquote>
    <p>I was very much mislead by this. I was sure I hade made some
      mistake when I picked out the changes you have made for static
      builds (and removed all the other changes, e.g. for the hermetic
      jar files), since you said this worked for you. I have been
      scrutinizing the difference between your branch and mine, over and
      over again, without understanding what the difference could be.<br>
    </p>
    <p>Finally I did what I should have done at the very beginning, and
      actually tested building and running your branch.</p>
    <p>It did not work either.</p>
    <p>So why did you claim it worked? I kept digging, and I found out
      the reason. You had indeed implemented a fix for this, but only on
      Linux. I was testing on macOS. (It is also not implemented for
      Windows, but since I'm still struggling to find a way to create
      proper static builds there it is less of a problem for now.)</p>
    <p>My branch worked just as well as your on Linux. I have now fixed
      so it works on macOS too. With this hurdle out of the way, I can
      get back to doing real work on the patch. Unfortunately this
      detour took far too much time. :-(<br></p></div></blockquote><div><br></div><div>Mystery solved! Yes, our prototype was for linux-x64 only (communicated in <a href="https://mail.openjdk.org/pipermail/leyden-dev/2023-February/000106.html">https://mail.openjdk.org/pipermail/leyden-dev/2023-February/000106.html</a> and our discussion meetings). I think I was not clear that you were testing on macOS.</div><div><br></div><div>The supported platform was also discussed in the hermetic Java meetings. My understanding was that we can focus on Linux (or unix-like) initially. Let's get the initial integration point support for Linux. Glad to see that you are making progress! Please let me know if you are running into any other issues.</div><div><br></div><div>Best,</div><div>Jiangli</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><p>
    </p>
    <p>/Magnus<br>
    </p>
  </div>

</blockquote></div></div>