<div dir="ltr">Thanks Erik!<div><br></div><div>For the record, I found this while working on a micro benchmark experiment which was using some code from the test lib. Probably not something people do very often :-)<div><br></div><div>Eirik.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 9, 2023 at 3:50 PM <<a href="mailto:erik.joelsson@oracle.com">erik.joelsson@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello Eirik,<br>
<br>
While this target isn't often used (as can be seen from how far it's bit <br>
rotted without anyone noticing), I think it's worth fixing it now that <br>
you have identified the issues. There is some value in being able to <br>
explicitly build these classes even if they are usually built by jtreg <br>
as part of running tests. If you post a PR with this, I will review it.<br>
<br>
/Erik<br>
<br>
On 3/5/23 09:36, Eirik Bjørsnøs wrote:<br>
> Hi,<br>
><br>
> The 'build-test-lib' target seems to be broken:<br>
><br>
> - make/test/BuildTestLib.gmk references the <br>
> directory $(TEST_LIB_SOURCE_DIR)/sun which does not seem to exist. <br>
> This can be fixed by removing the reference.<br>
> - Some test-lib sources use preview-features which is not enabled by <br>
> make/test/BuildTestLib.gmk. This is fixed by adding a JAVAC_FLAGS with <br>
> --enable-preview and also adding 'preview' to DISABLED_WARNINGS<br>
> - ASN1Formatter.annotate has a possible lossy conversion from long to <br>
> int which can be fixed by adding an explicit cast<br>
><br>
> Let me know if this is worth fixing and I can create a PR from this <br>
> branch of fixes:<br>
><br>
> <a href="https://github.com/openjdk/jdk/compare/master...eirbjo:build-test-libs" rel="noreferrer" target="_blank">https://github.com/openjdk/jdk/compare/master...eirbjo:build-test-libs</a><br>
><br>
> Thanks,<br>
> Eirik.<br>
</blockquote></div>