build-test-lib target seems broken

[erik.joelsson at oracle.com]

Hello Eirik,

While this target isn't often used (as can be seen from how far it's bit 
rotted without anyone noticing), I think it's worth fixing it now that 
you have identified the issues. There is some value in being able to 
explicitly build these classes even if they are usually built by jtreg 
as part of running tests. If you post a PR with this, I will review it.

/Erik

On 3/5/23 09:36, Eirik Bjørsnøs wrote:
> Hi,
>
> The 'build-test-lib' target seems to be broken:
>
> - make/test/BuildTestLib.gmk references the 
> directory $(TEST_LIB_SOURCE_DIR)/sun which does not seem to exist. 
> This can be fixed by removing the reference.
> - Some test-lib sources use preview-features which is not enabled by 
> make/test/BuildTestLib.gmk. This is fixed by adding a JAVAC_FLAGS with 
> --enable-preview and also adding 'preview' to DISABLED_WARNINGS
> - ASN1Formatter.annotate has a possible lossy conversion from long to 
> int which can be fixed by adding an explicit cast
>
> Let me know if this is worth fixing and I can create a PR from this 
> branch of fixes:
>
> https://github.com/openjdk/jdk/compare/master...eirbjo:build-test-libs
>
> Thanks,
> Eirik.