Subject: Re: bin/12876: NetBSD's INET6 patches to Postfix break non-INET6 features!
To: None <gnats-bugs@gnats.netbsd.org,>
From: None <itojun@iijlab.net>
List: netbsd-bugs
Date: 05/10/2001 05:29:16
>> 	try gnu/dist/postfix/src/smtpd/smtpd_peer.c revision 1.3.
>I don't have to try it to know that it absolutely cannot work.  You did
>not apply my changes.  You could not have done the tests I suggested, at
>least not without -DINET6 or you too would know that your commit doesn't
>fix the bug.

>Please apply *all* of my changes, exactly as I made them if you don't

	no.  i don't take patches that does not relate to the PR itself.

>understand them, and then *test* them!  They were primarily designed to

	fair enough, i'll try to test.  next time do not ever think of
	including tons of costmetic changes which is not maintainable.

>> 	i still don't understand this "more than 17 letters" part.
>> 	your changes to hostname buffer length are totally cosmetic.
>An array of sizeof("255.255.255.255"), even +1, is not anywhere near
>long enough to hold your average fully qualified hostname.....  I
>thought the comments I put in there to show what's being done at which
>stage would make this clear.

	ok.

>BTW, I'm guessing that Wietse won't accept any INET6 patches that are
>just #ifdefs.

	SO WHAT?  thanks for your unrelated comment.  it's up to wietse,
	not you.

>I think the best way to fully fix this maze of
>conditional compilation would be to first fix the code to call the
>already present peer_name() from libglobal and then to provide an
>alternate peer_name6.c that's been changed to use getnameinfo() and
>getaddrinfo() exclusively.

	bzzzt.  i confirmed with wietse and he said that peer_name is
	deprecated and the new code should use smtpd/smtpd_peer.c.
	you wrong.

itojun