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

Reply via email to