On Wed, 30 Oct 2024 23:27:12 GMT, Anthony Scarpino <ascarp...@openjdk.org> wrote:
>> Fernando Guallini has updated the pull request incrementally with one >> additional commit since the last revision: >> >> add more algos to TestExplicitKeyLength.java > > test/jdk/com/sun/crypto/provider/KeyAgreement/SupportedDHParamGensLongKey.java > line 27: > >> 25: * @test >> 26: * @bug 8072452 8163498 >> 27: * @library /test/lib > > It looks like you can remove this change & file from the PR The reason for this, is that this test is calling another : **SupportedDHParamGens** which uses the SecurityUtils, it does need to build /test/lib to compile ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21578#discussion_r1825029645
