JDK 10 RFR of JDK-8181393: Refactor shell test Selector/lots_of_updates.sh and SocketChannel/Open.sh to java
Amy Lu
amy.lu at oracle.com
Mon Jun 5 06:22:03 UTC 2017
On 6/3/17 4:21 AM, Paul Sandoz wrote:
>> webrev: http://cr.openjdk.java.net/~amlu/8181393/webrev.00/
>> <http://cr.openjdk.java.net/%7Eamlu/8181393/webrev.00/>
>>
>
> +1
>
> Out of curiosity is jtreg compilation of a test smart enough to know
> of at least some of its dependencies.
>
> 24 /*
> 25 * @test
> 26 * @bug 6824477
> 27 * @summary Selector.select can fail with IOException "Invalid argument" on
> 28 * Solaris if maximum number of file descriptors is less than 10000
> 29 * @requires (os.family != "windows")
> 30 * @library /test/lib
> 31 * @build LotsOfUpdates
> 32 * @run main LotsOfUpdatesTest
> 33 */
> 34
> 35 import jdk.test.lib.process.ProcessTools;
> 36
> 37 public class LotsOfUpdatesTest {
> Do you require line 31 since LotsOfUpdatesTest refers directly to
> LotsOfUpdates?
Actually test works fine without line 31 because it refers as
LotsOfUpdates.class.getName() (instead of a String "LotsOfUpdates").
I still want the at build in this case just to indicate explicitly that the
test has dependency other than @library
Thank you Paul for reviewing. Pushed.
Thanks,
Amy
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20170605/54746eb8/attachment-0001.html>
More information about the nio-dev
mailing list