On Tue, Mar 23, 2010 at 4:35 PM, Wouter Verhelst <w...@uter.be> wrote:
> Hi,
>
> On Wed, Mar 17, 2010 at 11:04:39PM +0100, Wouter Verhelst wrote:
>> On Thu, Mar 18, 2010 at 04:13:05AM +0700, Neutron Soutmun wrote:
>> > Package: nbd
>> > Version: 2.9.14-3
>> > Severity: normal
>> >
>> > -----BEGIN PGP SIGNED MESSAGE-----
>> > Hash: SHA1
>> >
>> > On Tue, Mar 16, 2010 at 01:24:54PM +0100, Wouter Verhelst wrote:
>> > > On Tue, Mar 16, 2010 at 06:36:22PM +0700, Neutron Soutmun wrote:
>> > > I should probably point out that there's a git tree for the upstream
>> > > code: it's on github ("/yoe/nbd"), and on sourceforge
>> > > (nbd.git.sourceforge.net).
>> >
>> > $ git clone http://github.com/yoe/nbd.git
>> >
>> > now, follow up but keep working continue on the old patch which won't get
>> > me
>> > more confusion.
>>
>> Well, part of the reason I pointed you to that repository is the fact
>> that there's a patch in there (for #557810) that conflicts with yours.
>> It'd be cool if you could update it so that it would apply cleanly to
>> that version, and then I can easily do an upstream release immediately
>> followed by a Debian package.
>
> Are you working on that? If not, that's not a problem, I'll do it
> myself; it's just that it'd probably be easier for you at this point.
> But if you don't have the time, or are tired of it, or are working on
> something else, I'll finish this off.
Sorry for late response, I have something else to do, but I still thinking
of the solution for #557810 which the user parsing the hostname for
listening address that case for example
# nbd-server localhost:12345 `pwd`/nbd-file 1G -C /dev/null
which localhost may resolve in both IPv4 (127.0.0.1) and IPv6 (::1)
Therefore, I think that if I'm using getaddrinfo() to get each information
and add to server listening pool only if it valid, it's better, isn't it ?
Or localhost:12345 is only mean 127.0.0.1 port 12345 ?
If it is this case, when parsing this command line should set the
socket family to AF_INET.
If now this case is ignored, I think I can prepare the patch for you which
diff from git repository that I reviewed already that the patch does
not conflict much.
I test that hostname parsing is OK but only the first info that
getaddrinfo() return could
setup for listening.
Regards,
Neutron Soutmun
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
