On 2020/4/20 下午9:27, Alan Bateman wrote:
On 20/04/2020 11:32, sundararajan.athijegannat...@oracle.com wrote:
Hi Alan,

I don't remember it now. Likely a mistake. The changeset

http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/a066fe7b1b42

has that file. Perhaps it may be useful to restore & check if the test passes.
Yes. I think it was the only test for this jlink option so let's bring it back or replace it.


I think the test should be brought back. I restored the test and it didn't pass. Here is the diff against the original test[1]: http://cr.openjdk.java.net/~aoqi/8242846/diff.01, and the new webrev is http://cr.openjdk.java.net/~aoqi/8242846/webrev.01/ . I'm not familiar with this field, so I am not sure I fix this rightly.


Cheers,
Ao Qi

[1] http://hg.openjdk.java.net/jdk9/jdk9/jdk/raw-file/decf1acca62d/test/tools/jlink/plugins/OrderResourcesPluginTest.java


-Alan.

Reply via email to