Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-22 Thread Andrew Hughes
- Original Message - > On 21/04/2015 16:04, Michael McMahon wrote: > > : > > > > http://cr.openjdk.java.net/~michaelm/8078276/webrev.1/ > Before you push this, shouldn't the test have copyright Red Hat? There's > an example in the same directory. > > -Alan > Yes, I was thinking the sam

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-22 Thread Severin Gehwolf
On Tue, 2015-04-21 at 16:09 +0100, Alan Bateman wrote: > On 21/04/2015 16:04, Michael McMahon wrote: > > On 21/04/15 14:56, Alan Bateman wrote: > >> : > >>> > >> Can the test be moved to jdk/test/com/sun/jndi/dns as that seems to > >> be where the other tests are? Also I assume this will need a >

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-21 Thread Alan Bateman
On 21/04/2015 16:04, Michael McMahon wrote: : http://cr.openjdk.java.net/~michaelm/8078276/webrev.1/ Before you push this, shouldn't the test have copyright Red Hat? There's an example in the same directory. -Alan

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-21 Thread Alan Bateman
On 21/04/2015 16:04, Michael McMahon wrote: On 21/04/15 14:56, Alan Bateman wrote: : Can the test be moved to jdk/test/com/sun/jndi/dns as that seems to be where the other tests are? Also I assume this will need a copyright header before it is pushed. -Alan I had pushed it already. So, I

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-21 Thread Michael McMahon
On 21/04/15 14:56, Alan Bateman wrote: On 20/04/2015 18:32, Severin Gehwolf wrote: : OK fixed: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6991580/webrev.02/ FWIW, I don't think the test needs IP addresses of DNS servers to be functional, though. All it really does is passing it to InetAd

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-21 Thread Alan Bateman
On 20/04/2015 18:32, Severin Gehwolf wrote: : OK fixed: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6991580/webrev.02/ FWIW, I don't think the test needs IP addresses of DNS servers to be functional, though. All it really does is passing it to InetAddress.getByName(). Prior this patch DnsC

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-21 Thread Michael McMahon
On 20/04/15 18:32, Severin Gehwolf wrote: On Mon, 2015-04-20 at 12:24 -0400, Andrew Hughes wrote: - Original Message - Adding in net-dev. On Mon, 2015-04-20 at 14:02 +0200, Severin Gehwolf wrote: Hi, Could I please get a review and a sponsor for the following patch? The issue is tha

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-20 Thread Severin Gehwolf
On Mon, 2015-04-20 at 12:24 -0400, Andrew Hughes wrote: > - Original Message - > > Adding in net-dev. > > > > On Mon, 2015-04-20 at 14:02 +0200, Severin Gehwolf wrote: > > > Hi, > > > > > > Could I please get a review and a sponsor for the following patch? > > > > > > The issue is that J

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-20 Thread Andrew Hughes
- Original Message - > Adding in net-dev. > > On Mon, 2015-04-20 at 14:02 +0200, Severin Gehwolf wrote: > > Hi, > > > > Could I please get a review and a sponsor for the following patch? > > > > The issue is that JDK's internal /etc/resolv.conf nameserver parsing > > does not properly ac

Re: RFR(xs): 6991580: IPv6 Nameservers in resolv.conf throws NumberFormatException

2015-04-20 Thread Severin Gehwolf
Adding in net-dev. On Mon, 2015-04-20 at 14:02 +0200, Severin Gehwolf wrote: > Hi, > > Could I please get a review and a sponsor for the following patch? > > The issue is that JDK's internal /etc/resolv.conf nameserver parsing > does not properly account for IPv6 addresses on Linux/Unix. While t

Re: IPv6 Nameservers in resolv.conf

2010-10-18 Thread Chris Hegarty
Hi Guenther, Before we can accept patches or changes from an external contributor you must sign the Contributor Agreement [1]. I looked at the list of contributors and cannot find you listed. I haven't looked at the patch yet, but I have looked at the testcase that's in CR 6991580. Have you

Re: IPv6 Nameservers in resolv.conf

2010-10-15 Thread Guenther Niess
The related Bug has the number 6991580. I wrote a small patch: diff -ur openjdk.orig/jdk/src/share/classes/com/sun/jndi/dns/DnsClient.java openjdk/jdk/src/share/classes/com/sun/jndi/dns/DnsClient.java --- openjdk.orig/jdk/src/share/classes/com/sun/jndi/dns/DnsClient.java 2010-08-23 13:55:13.000

Re: IPv6 Nameservers in resolv.conf

2010-10-12 Thread Chris Hegarty
[bcc'ing off jdk7-dev and cc'ing net-dev, to continue the conversation] Alan Bateman wrote: Guenther Niess wrote: . I don't think there is a mailing list for JNDI here but given that this seems to be an issue with the DNS provider, then net-dev would be a good place to bring it up. It's