From owner-freebsd-ports Thu Jan 23 9:54:56 2003 Delivered-To: freebsd-ports@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A36F037B406 for ; Thu, 23 Jan 2003 09:54:54 -0800 (PST) Received: from straylight.ringlet.net (office.sbnd.net [217.75.140.130]) by mx1.FreeBSD.org (Postfix) with SMTP id F206E43EB2 for ; Thu, 23 Jan 2003 09:54:42 -0800 (PST) (envelope-from roam@ringlet.net) Received: (qmail 20364 invoked by uid 1000); 23 Jan 2003 17:53:44 -0000 Date: Thu, 23 Jan 2003 19:53:44 +0200 From: Peter Pentchev To: Yar Tikhiy Cc: Ceri Davies , Julian Mayer , knu@FreeBSD.org, ports@FreeBSD.org, hackers@FreeBSD.org Subject: Re: FreeBSD Port: portupgrade-20021216 Message-ID: <20030123175344.GC19717@straylight.oblivion.bg> Mail-Followup-To: Yar Tikhiy , Ceri Davies , Julian Mayer , knu@FreeBSD.org, ports@FreeBSD.org, hackers@FreeBSD.org References: <20030123081224.GA471@straylight.oblivion.bg> <20030123101216.GA2856@submonkey.net> <20030123134749.GA377@straylight.oblivion.bg> <20030123174706.GA8943@comp.chem.msu.su> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="K8nIJk4ghYZn606h" Content-Disposition: inline In-Reply-To: <20030123174706.GA8943@comp.chem.msu.su> User-Agent: Mutt/1.5.3i Sender: owner-freebsd-ports@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org --K8nIJk4ghYZn606h Content-Type: text/plain; charset=windows-1251 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jan 23, 2003 at 08:47:06PM +0300, Yar Tikhiy wrote: > On Thu, Jan 23, 2003 at 03:47:49PM +0200, Peter Pentchev wrote: > >=20 > > Attached is a patch to the libexec/ftpd source, which adds a new -P > > option taking an argument of either a numeric port number or a service > > name as described in the getaddrinfo(3) manual page. What do people > > think about adding this functionality? >=20 > Peter, here is a bit reworked version of your patch. > Does it look reasonable? Yes, this looks fine; the data connection port issue was brought up by Matthew Seaman in a private message to me, which I did not respond to immediately, because I wanted to feel the general thoughts on the subject. Now that you have incorporated it into your patch, it all sounds great :) I see that you have also changed my '.Nm' macro for the 'ftp' service to '.Dq'; that's fine too, I was in quite a bit of doubt myself over exactly which mdoc macro to use. The '.Nm' came from the telnet(1) manual page, but it did not look quite right to me. Thanks for your fast reaction :) I guess libexec/ftpd is your code, so it's your call to commit at will if nobody objects. G'luck, Peter --=20 Peter Pentchev roam@ringlet.net roam@FreeBSD.org PGP key: http://people.FreeBSD.org/~roam/roam.key.asc Key fingerprint FDBA FD79 C26F 3C51 C95E DF9E ED18 B68D 1619 4553 "yields falsehood, when appended to its quotation." yields falsehood, when = appended to its quotation. --K8nIJk4ghYZn606h Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (FreeBSD) iD8DBQE+MCwo7Ri2jRYZRVMRAg5bAJ9/LZIyfPy8auzMhWQ59m0riZ/wBQCgixWT AAPf9PIEJgyggJp000OWgq4= =gbNo -----END PGP SIGNATURE----- --K8nIJk4ghYZn606h-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-ports" in the body of the message