RFR: 8043118: update test/java/net/URLPermission/nstest/lookup.sh to use explicit @build target

Michael McMahon michael.x.mcmahon at oracle.com
Wed May 14 11:11:06 UTC 2014


On 14/05/14 11:39, Alan Bateman wrote:
> On 14/05/2014 11:35, Michael McMahon wrote:
>> Could I get the following small test case change reviewed please?
>>
>> This fixes a sporadic test case failure due to incorrect use of 
>> @build tag
>> in the script
>>
>> --- a/test/java/net/URLPermission/nstest/lookup.sh
>> +++ b/test/java/net/URLPermission/nstest/lookup.sh
>> @@ -26,7 +26,7 @@
>>  # @library /lib/testlibrary
>>  # @compile -XDignore.symbol.file=true SimpleNameService.java
>>  #            LookupTest.java SimpleNameServiceDescriptor.java
>> -# @build jdk.testlibrary.*
>> +# @build jdk.testlibrary.Utils
>>  # @run shell/timeout=50 lookup.sh
>>  #
>>
>> http://cr.openjdk.java.net/~michaelm/8043118/webrev.1/
> I think Katja changed this test very recently to use the wildcard. Is 
> your jtreg up to date? The wildcard support is recent, b09 I think. If 
> you are b09 then maybe there is something else doing on here.
>
> -Alan.

Okay. I just updated my jtreg and the test passes with the wildcard. I'm 
not sure why
it was changed to use the wildcard, but there doesn't seem to be any 
point in me changing it back.

Thanks,
Michael


More information about the net-dev mailing list