Code Review Request: 7183800: TEST_BUG: Update tests to run on Ubuntu 12.04 (localhost is 127.0.1.1)

Chris Hegarty chris.hegarty at oracle.com
Sun Jul 15 14:52:44 PDT 2012


Kurchi,

I don't have any problem with the source changes ( they make sense ), 
but I don't see why they are necessary.

The listening socket is bound to the wildcard address, so sending to the 
local address should work. I guess this is some kind of (default 
configuration error/issue). Anyway, your test changes are fine.

Adding the third test to the problem list makes sense ( as you say, it 
will require a more involved fix ), but you should create a new separate 
bug number for it and add it to the comment in the ProblemList. It may 
be a little confusing to see this bug number in the comment when the bug 
database will show it as fixed.

-Chris.

On 15/07/12 19:54, Kurchi Subhra Hazra wrote:
> Hi,
>
> Some tests in our test repository assume that InetAddress.getLocalHost()
> will return 127.0.0.1. However, with
> linux systems running Ubuntu 12.04 now returning 127.0.1.1 as localhost,
> these tests are failing. I have changed two
> of the NIO tests to remove their dependency on
> InetAddress.getLocalHost(). A third test has been added to
> the ProblemList.txt since the changes required for it are more involved.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7183800
> Webrev: http://cr.openjdk.java.net/~khazra/7183800/webrev.00/
>
>
> Thanks,
> Kurchi
>


More information about the nio-dev mailing list