Oops, 2.10.1 didn't compile on win32 with the new linklocal detection code.  Patchs at <a href="http://developer.pidgin.im/ticket/14802">http://developer.pidgin.im/ticket/14802</a> fixes the issue.<br><br><div class="gmail_quote">

On 9 December 2011 01:26, Linus Lüssing <span dir="ltr"><<a href="mailto:linus.luessing@web.de">linus.luessing@web.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Mark, hi Ethan, hi Kevin,<br>
<br>
Thanks for having a glance at these patches.<br>
<div class="im"><br>
On Tue, Nov 29, 2011 at 11:24:20PM -0500, Ethan Blanton wrote:<br>
> Mark Doliner spake unto us the following wisdom:<br>
> > On Tue, Nov 29, 2011 at 7:21 PM, Ethan Blanton <<a href="mailto:elb@pidgin.im">elb@pidgin.im</a>> wrote:<br>
> > > Mark Doliner spake unto us the following wisdom:<br>
> > >> I guess I'd prefer if we didn't commit these to 2.x.y.  Supporting<br>
> > >> ipv6 for bonjour doesn't seem critical to me.<br>
> > ><br>
> > > Please clarify what you mean by "these".<br>
> ><br>
> > I guess I meant all of them.  And I don't have a strong opinion about<br>
> > this.  I guess I'm mostly saying that _I_ wouldn't have committed them<br>
> > to 2.x.y.  It's probably not worth reverting the changes from 2.x.y.<br>
> > And I'm not going to stop anyone from committing the remaining patches<br>
> > to 2.x.y.  But we should be EXTREMELY careful not to cause other bugs.<br>
><br>
> I generally agree; however, I can confirm that Bonjour + IPv6 = not<br>
> good at the moment, and the patches which have already been applied<br>
> should not affect anything but Bonjour (they are all local to the<br>
> Bonjour plugin) on IPv6 (they come into play only for IPv6 addresses).<br>
> The way I see it, we're unlikely to break anything that isn't already<br>
> broken.<br>
<br>
</div>Yes, those ones are all supposed to be just fixes and should<br>
address critical issues with bonjour if IPv6 is activated in<br>
avahi.<br>
<div class="im"><br>
><br>
> The remaining patch is a bit of a different story.<br>
<br>
</div>Oki doki. As it's only related to the file transfering in bonjour,<br>
I guess I can live without it in a 2.10.1 release as long as it<br>
doesn't take too long for a following release.<br>
<br>
However I'm currently wondering whether other protocols might have<br>
issues with purple_network_do_listen() / purple_network_listen()<br>
on some operating issues. At first I thought having read somewhere<br>
that the order of addresses returned by getaddrinfo() was<br>
unspecified. Now I searched again and it looks like RFC3484 is<br>
responsible for the ordering. I couldn't quite figure this out yet<br>
- does it ensure that ::ffff:0.0.0.0 (mapped IPv4 wildcard) is<br>
always prefered over ::0 (IPv6 wildcard)?<br>
<br>
If not, wouldn't it be better to change AF_UNSPEC to AF_INET in<br>
purple_network_listen() until purple_network_do_listen() is doing<br>
what it is supposed to do?<br>
<div class="im"><br>
> Patch 2 (fix socket listen address) requires a minor version<br>
> bump, as it adds API.<br>
<br>
</div>Just for clarification (I'm not really familiar with the<br>
development/release process in pidgin): My impression was<br>
that in the 2.10.0 API purple_network_listen() was supposed to<br>
offer both IPv4 and IPv6 connectivity in case of AF_UNSPEC (which<br>
currently is not the case). Is any change that tries to allow this<br>
supposed behaviour considered an API change needing a minor<br>
version bump? When do you consider a behaviour change<br>
in the API being a fix instead?<br>
<br>
<br>
On another note, I would like to remove this Windows XP disclaimer<br>
in the second patch. It'd probably be the best to create two<br>
separate sockets for IPv4 and IPv6, right? Which as far as I know<br>
would require a far more invasive patch instead - or does anyone<br>
with some better knowledge of the libpurple API have an idea about<br>
how to achieve this easily?<br>
<br>
><br>
> Ethan<br>
<br>
Cheers, Linus<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@pidgin.im">Devel@pidgin.im</a><br>
<a href="http://pidgin.im/cgi-bin/mailman/listinfo/devel" target="_blank">http://pidgin.im/cgi-bin/mailman/listinfo/devel</a><br>
</div></div></blockquote></div><br>