Re: Wget Patch for 1.8.1 witch IPv6

2002-01-16 Thread Hrvoje Niksic
Daniel Stenberg [EMAIL PROTECTED] writes: `struct addrinfo' contains a `struct sockaddr', which carries the necessary scoping information (I think). The question at the time was whether I could extract only the address(es) and ignore everything else, as it was possible with IPv4. Itojune

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Hrvoje Niksic
Markus Buchhorn [EMAIL PROTECTED] writes: Reading back, that was itojun's proposal, and I suspect probably a good choice, even if it seems less clean. Itojun is one of the leading lights in IPv6 development, along with the whole WIDE group in Japan, and heavily involved in the v6 stacks for

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Hrvoje Niksic
Daniel Stenberg [EMAIL PROTECTED] writes: I'd suggest that you instead pass around a 'struct hostent *' on IPv4 only platforms Why? The rest of the code never needs anything from `struct hostent' except the list of addresses, and this is what my code extracts. By extension, the idea was for

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Daniel Stenberg
On Tue, 15 Jan 2002, Hrvoje Niksic wrote: I'd suggest that you instead pass around a 'struct hostent *' on IPv4 only platforms Why? The rest of the code never needs anything from `struct hostent' except the list of addresses, and this is what my code extracts. Well, why extract the

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Hrvoje Niksic
Daniel Stenberg [EMAIL PROTECTED] writes: On Tue, 15 Jan 2002, Hrvoje Niksic wrote: I'd suggest that you instead pass around a 'struct hostent *' on IPv4 only platforms Why? The rest of the code never needs anything from `struct hostent' except the list of addresses, and this is what

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Daniel Stenberg
On Tue, 15 Jan 2002, Hrvoje Niksic wrote: Well, why extract the addresses when you can just leave them in the struct and pass a pointer to that? Because I'm caching the result of the lookup, and making a deep copy of `struct hostent' is not exactly easy. (Yes, I know libcurl does it, but

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Hrvoje Niksic
Daniel Stenberg [EMAIL PROTECTED] writes: On Tue, 15 Jan 2002, Hrvoje Niksic wrote: Well, why extract the addresses when you can just leave them in the struct and pass a pointer to that? Because I'm caching the result of the lookup, and making a deep copy of `struct hostent' is not

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-15 Thread Hrvoje Niksic
Thomas Lussnig [EMAIL PROTECTED] writes: Ok first we don't need this difference. I think it's not so easy than it first seem's. Because IPv6 is an superset of IPv4 there is an representation fo IPv4 Adresses. But is it desirable to use it in preference to native IPv4 calls? I apologize if

Wget Patch for 1.8.1 witch IPv6

2002-01-14 Thread Thomas Lussnig
Hi, i have make an hopefull clean patch for IPv6 support. It works fine (testet (compile and v6 retrive) But im not already happy with it. 1. Now if IPv6 enabled it only fetch IPv6 IPv4 sites faile 2. Need an switch to change the preffered mode 4/6 3. Makefile optional v6 support Point 1 i

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-14 Thread Hrvoje Niksic
Thomas Lussnig [EMAIL PROTECTED] writes: 1. Now if IPv6 enabled it only fetch IPv6 IPv4 sites faile This is a problem, and part of the reason why the patch is so simple in its current form. A correct patch must modify struct address_list to hold a list of IP addresses, each of which can be

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-14 Thread Markus Buchhorn
At 03:16 AM 15/01/2002 +0100, Hrvoje Niksic wrote: Thomas Lussnig [EMAIL PROTECTED] writes: 1. Now if IPv6 enabled it only fetch IPv6 IPv4 sites faile This is a problem, and part of the reason why the patch is so simple in its current form. [...] Another possibility is to store struct

Re: Wget Patch for 1.8.1 witch IPv6

2002-01-14 Thread Daniel Stenberg
On Tue, 15 Jan 2002, Hrvoje Niksic wrote: 1. Now if IPv6 enabled it only fetch IPv6 IPv4 sites faile This is a problem, and part of the reason why the patch is so simple in its current form. It is indeed. If done correctly, using the proper API, there won't be any difference between IPv6