From owner-svn-ports-head@FreeBSD.ORG Fri May 15 00:16:31 2015 Return-Path: Delivered-To: svn-ports-head@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 712F6DB7 for ; Fri, 15 May 2015 00:16:31 +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 4F1251744 for ; Fri, 15 May 2015 00:16:31 +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 t4F0GUSH024402 for ; Fri, 15 May 2015 00:16:30 GMT (envelope-from bdrewery@freefall.freebsd.org) Received: (from bdrewery@localhost) by freefall.freebsd.org (8.14.9/8.14.9/Submit) id t4F0GUqv024399 for svn-ports-head@freebsd.org; Fri, 15 May 2015 00:16:30 GMT (envelope-from bdrewery) Received: (qmail 83414 invoked from network); 14 May 2015 19:16:28 -0500 Received: from unknown (HELO ?10.10.1.139?) (freebsd@shatow.net@10.10.1.139) by sweb.xzibition.com with ESMTPA; 14 May 2015 19:16:28 -0500 Message-ID: <55553ADE.5070207@FreeBSD.org> Date: Thu, 14 May 2015 19:16:30 -0500 From: Bryan Drewery Organization: FreeBSD User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Adam Weinberger CC: "Philip M. Gollucci" , "ports-committers@freebsd.org" , "svn-ports-all@freebsd.org" , "svn-ports-head@freebsd.org" Subject: Re: svn commit: r386285 - head/security/keychain References: <201505140101.t4E11r9T031351@svn.freebsd.org> <08F806A7-3CD9-417A-BA4E-AF739E83B6A8@adamw.org> <082DB80D-6385-4F4F-8A86-2DB4561E39E8@adamw.org> <5554C761.3060609@FreeBSD.org> <07ED759D-DFAC-44D0-A22B-1639EE3E0CB8@adamw.org> <5554C8A1.50200@FreeBSD.org> <317F946C-1F0E-4F8E-B121-6442CC82BCAA@adamw.org> <5554C918.8090305@FreeBSD.org> <5554CB3B.9070807@FreeBSD.org> In-Reply-To: <5554CB3B.9070807@FreeBSD.org> OpenPGP: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="mD1rGQcSGj7AkjKb9JKbWvuLwVJXsp0UU" X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 May 2015 00:16:31 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --mD1rGQcSGj7AkjKb9JKbWvuLwVJXsp0UU Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 5/14/2015 11:20 AM, Bryan Drewery wrote: > On 5/14/2015 11:11 AM, Bryan Drewery wrote: >> On 5/14/2015 11:10 AM, Adam Weinberger wrote: >>>> On 14 May, 2015, at 10:09, Bryan Drewery wrot= e: >>>> >>>> On 5/14/2015 11:07 AM, Adam Weinberger wrote: >>>>>> On 14 May, 2015, at 10:03, Bryan Drewery wr= ote: >>>>>> >>>>>> On 5/14/2015 11:00 AM, Adam Weinberger wrote: >>>>>>>> On 13 May, 2015, at 21:20, Philip M. Gollucci wrote: >>>>>>>> >>>>>>>> Then it needs an option not an if exists() >>>>>>> >>>>>>> Bryan, can you please consider reverting it, or giving me approva= l to revert it? It may fix the case when the port is built in poudriere w= ithout sshd in base, but it is a regression for everybody else. >>>>>>> >>>>>>> # Adam >>>>>>> >>>>>>> >>>>>> >>>>>> No. The way RUN_DEPENDS is supposed to work is what I said in the >>>>>> commit. These exists() checks are redundant and confuse the packag= e >>>>>> dependencies. >>>>>> >>>>>> How are you building? It's possible the tool you are using is >>>>>> incorrectly handling RUN_DEPENDS. >>>>> >>>>> Ah! I see. It's portmaster that's doing the misbehaving. The proble= m is that openssh-portable still shows up in run-depends-list, and so por= tmaster decides that it is going to install openssh-portable first. >>>>> >>>>> [root@apnoea /usr/ports/security/keychain] which ssh-agent >>>>> /usr/bin/ssh-agent >>>>> [root@apnoea /usr/ports/security/keychain] make run-depends-list >>>>> /usr/ports/security/openssh-portable >>>>> /usr/ports/shells/bash >>>>> >>>> >>>> Ok. I'll look at portmaster and try to fix it now. We really need th= ese >>>> exists() to go away; it causes rebuilds on the package cluster for n= o >>>> reason. This specific case is not one, but we need them to generally= go >>>> away. >>> >>> Sorry about the noise, Bryan. >>> >> >> It's not noise! There's 2 bugs here. Portmaster installs and Poudriere= >> builds. Both need to be fixed :) >> >> >=20 > Portupgrade also incorrectly installs. >=20 At leat portupgrade-devel should now do the right things. Portmaster is next. --=20 Regards, Bryan Drewery --mD1rGQcSGj7AkjKb9JKbWvuLwVJXsp0UU Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJVVTreAAoJEDXXcbtuRpfP9bwH/RSuBsRMlUZiNr2DnDHCY9Jw wyRkHlXXQHtWs9GtJmvkREaZvYMafnonBkjHlaGYs/d6cM0xj7DjHrS9/e3HS8mQ rPcsXlNu1ZkjBUKrSr9CuZtcT65YBlAQpRRYmB1Xre3ELtqsLSDevKDMAnCPijch zVya8U3TRA5KEuyfcIBmoBMGPoU4BWvHj8LfpcAGvOEBpIR99DMi7R8+lCkHGoB3 rpZi1TJ6fi/V3Ay6iZfLNzmYt+Cop9CIxlG0G+wO3k3w6cQdLCBVqP5vz7D32F3F Qb719MXNrSKeMD/H4ETBuPWeZULNq5awWr31OVJGhpT6an4d9YCg/1tU/vX5KUA= =z1r4 -----END PGP SIGNATURE----- --mD1rGQcSGj7AkjKb9JKbWvuLwVJXsp0UU--