Thank you, Serguei and Alex, for reviewing this change.

Best regards,
Daniil

On 4/1/20, 2:19 PM, "serguei.spit...@oracle.com" <serguei.spit...@oracle.com> 
wrote:

    Hi Daniil,
    
    LGTM++
    
    Thanks,
    Serguei
    
    
    On 3/30/20 13:06, Alex Menkov wrote:
    > Looks good.
    >
    > --alex
    >
    > On 03/30/2020 12:43, Daniil Titov wrote:
    >> Please review the change [1] that fixes the failure of 
    >> com/sun/jdi/JdwpListenTest.java
    >> and com/sun/jdi/JdwpAttachTest.java tests on OSX 10.15.
    >>
    >> The problem here is the similar to the one solved in [4] by 
    >> additional filtering
    >>   of unusual network interfaces in the test library class 
    >> jdk.test.lib.NetworkConfiguration.
    >> However,  the failing com/sun/jdi tests do not use 
    >> jdk.test.lib.NetworkConfiguration and
    >> Instead do repeat the same logic themselves.
    >>
    >> The fix changes these tests to start using 
    >> jdk.test.lib.NetworkConfiguration to find all local addresses.
    >>
    >> Initially the issue [2] also included 3 other failing tests from 
    >> sun/management/jdp package, but these tests fail
    >> for a different reason so I moved them in the new issue [3] and 
    >> updated the ProblemList.txt  for them.
    >>
    >>
    >> [1] Webrev: http://cr.openjdk.java.net/~dtitov/8241530/webrev.01/
    >> [2] Jira Issue: https://bugs.openjdk.java.net/browse/JDK-8241530
    >> [3] https://bugs.openjdk.java.net/browse/JDK-8241865
    >> [4] https://bugs.openjdk.java.net/browse/JDK-8241336
    >>
    >> Thank you,
    >> Daniil
    >>
    >>
    
    


Reply via email to