From owner-freebsd-ports@FreeBSD.ORG Thu Nov 13 01:59:58 2014 Return-Path: Delivered-To: ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 3AADA871 for ; Thu, 13 Nov 2014 01:59:58 +0000 (UTC) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 1C687A59 for ; Thu, 13 Nov 2014 01:59:58 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.9/8.14.9) with ESMTP id sAD1xvra014030 for ; Thu, 13 Nov 2014 01:59:57 GMT (envelope-from bdrewery@freefall.freebsd.org) Received: (from bdrewery@localhost) by freefall.freebsd.org (8.14.9/8.14.9/Submit) id sAD1xvw9014029 for ports@FreeBSD.org; Thu, 13 Nov 2014 01:59:57 GMT (envelope-from bdrewery) Received: (qmail 51477 invoked from network); 12 Nov 2014 19:59:55 -0600 Received: from unknown (HELO ?10.10.0.24?) (freebsd@shatow.net@10.10.0.24) by sweb.xzibition.com with ESMTPA; 12 Nov 2014 19:59:55 -0600 Message-ID: <5464109B.8020204@FreeBSD.org> Date: Wed, 12 Nov 2014 19:59:55 -0600 From: Bryan Drewery Organization: FreeBSD User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: John-Mark Gurney , ports@FreeBSD.org Subject: Re: review of new tcpcrypt port... References: <20141108100709.GF24601@funkthat.com> <20141113001649.GB24601@funkthat.com> <5463FCF6.4040901@FreeBSD.org> <20141113011157.GD24601@funkthat.com> In-Reply-To: <20141113011157.GD24601@funkthat.com> OpenPGP: id=6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="R7WR0HxatqxbQ07LtwvcK1RrIi5OcgERk" X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 13 Nov 2014 01:59:58 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --R7WR0HxatqxbQ07LtwvcK1RrIi5OcgERk Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 11/12/2014 7:11 PM, John-Mark Gurney wrote: > Bryan Drewery wrote this message on Wed, Nov 12, 2014 at 18:36 -0600: >> On 11/12/2014 6:16 PM, John-Mark Gurney wrote: >>> John-Mark Gurney wrote this message on Sat, Nov 08, 2014 at 02:07 -08= 00: >>>> I'd like someone to review the attached port for me to commit... It= >>>> contains a daemon to run on divert sockets to implement the tcpcrypt= >>>> protocol. I have tested this port w/ HEAD and it works fine w/ the >>>> attached patch... >>>> >>>> The included patch has been submitted upstream and committed, but a >>>> new release has not yet been released. >>>> >>>> portlint -A looks fine. >>> >>> As I haven't received a review, in a couple more days, probably Frida= y, >>> I'll commit the port... >>> >>> Thanks. >>> >> >> Can you include the security/Makefile bit in the patch too please? >=20 > Hmm... Thought I did, but I clearly did not... Fixed... I've attached= > the latest version of the port.. >=20 >> Have you ran it through poudriere? At the very least 'env DEVELOPER=3D= 1 >> make stage' and 'make check-plist' are required. Portlint is not >> sufficient, nor can it even be trusted in all cases. >=20 > I didn't do poudriere, but I did all the building w/ DEVELOPER=3D1 > set, which did fix a few things.. Well 'make check-plist' must be ran manually. I think I'll enable it automatically soon. I've avoided it because it would break Redports even more. >=20 > Looks like port test from porttools as documented at: > https://www.freebsd.org/doc/en/books/porters-handbook/testing-porttools= =2Ehtml >=20 > Is wrong.. you need to be in the port directory for it to work... >=20 Ugh. That's a bad page. I'm working on a blog article for this case to get out soon. I may get it into the Journal as well. >> If you check those and get back to me I'll approve it. >=20 > Thanks... >=20 >> Usually you'd need a port commit bit to commit anything, or an approva= l >> (which is different than a review). I hate our process, but that's how= >> it is. Poke people in #bsdports on EFNet, or me on IRC, for >> approvals/reviews in the future. >=20 > Well, I'm fine w/ someone else committing the port, just trying to save= > someone the work, and trying to get it into the tree in a timely > manner.. I viewed the review as approval... Guess I should have made > it more clear in my original request... >=20 > btw, the included patch has been upstreamed, but they haven't tagged a > new rc yet... >=20 3 comments: 1. No need for commented line in the Makefile 2. Only 1 WWW for pkg-descr 3. If 'make check-plist' passes then I approve it and you can commit yourself. I am mentoring several src committers for ports and getting them bits after enough work, so just ping me whenever you have more. Thanks, Bryan --R7WR0HxatqxbQ07LtwvcK1RrIi5OcgERk Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (MingW32) iQEcBAEBAgAGBQJUZBCbAAoJEDXXcbtuRpfPRJwH/iElqiBKaFaBbQ3Sz8s3wQZU cMBb4VoVQkNeSPILPCdCjUMyzFd4FKJ250f2LGqK4f4pYMtqZdMNS4dY518lAOS3 k+2XmteBdvQQVlbNU1NtwFP2h9Iz0GyoYNkwdenSj8f58GZkPWZdcT007Bvg2bt6 7PKjHmZMAiOjjAUTKurAKV+XHTnbIsIEbpja0lDPPfYWzA/0Ns7qdY9MFJx/EcWe THYYUBdsdc3MhU5vRQr7KJpfcMEEr4W4fgQHTgR3xnLkv7cZiKxMg+Bki62fTC7T FllH6dsRAVuoI4KDH7y413UqSWyrTZI4/UCVcGFWjABBS0qUN01EUTGKogRNceA= =sv0b -----END PGP SIGNATURE----- --R7WR0HxatqxbQ07LtwvcK1RrIi5OcgERk--