From owner-svn-ports-head@freebsd.org Thu Jun 30 12:03:38 2016 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 47AA7B864A8; Thu, 30 Jun 2016 12:03:38 +0000 (UTC) (envelope-from mat@FreeBSD.org) Received: from prod2.absolight.net (mx3.absolight.net [IPv6:2a01:678:2:100::25]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "plouf.absolight.net", Issuer "CAcert Class 3 Root" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id E06982C68; Thu, 30 Jun 2016 12:03:37 +0000 (UTC) (envelope-from mat@FreeBSD.org) Received: from prod2.absolight.net (localhost [127.0.0.1]) by prod2.absolight.net (Postfix) with ESMTP id 30842BDC9A; Thu, 30 Jun 2016 14:03:35 +0200 (CEST) Received: from gw.in.absolight.net (gw-ecl.in.absolight.net [79.143.241.69]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "gw.in.absolight.net", Issuer "CA Cert Signing Authority" (not verified)) by prod2.absolight.net (Postfix) with ESMTPSA id 0644BBDC73; Thu, 30 Jun 2016 14:03:35 +0200 (CEST) Received: from ogg.in.absolight.net (ogg.in.absolight.net [79.143.241.239]) by gw.in.absolight.net (Postfix) with ESMTP id 423F06124; Thu, 30 Jun 2016 14:03:31 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by ogg.in.absolight.net (Postfix) with ESMTP id C97092B843DB; Thu, 30 Jun 2016 14:03:28 +0200 (CEST) Date: Thu, 30 Jun 2016 14:03:26 +0200 From: Mathieu Arnold To: Baptiste Daroussin cc: Bryan Drewery , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r417364 - head/Mk/Scripts Message-ID: In-Reply-To: <20160630111918.nhilkb4xgrgqwlde@ivaldir.etoilebsd.net> References: <201606231131.u5NBVFur021580@repo.freebsd.org> <075ef3c9-89f1-8e95-c86a-bf00831c373e@FreeBSD.org> <20160630054109.beqlksn3juwodf72@ivaldir.etoilebsd.net> <685BD37565065F0C493733CE@atuin.in.mat.cc> <20160630111918.nhilkb4xgrgqwlde@ivaldir.etoilebsd.net> X-Mailer: Mulberry/4.0.8 (Mac OS X) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="==========A71BC19C8234FC6D68AC==========" X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.22 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: Thu, 30 Jun 2016 12:03:38 -0000 --==========A71BC19C8234FC6D68AC========== Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: inline +--On 30 juin 2016 13:19:18 +0200 Baptiste Daroussin wrote: | On Thu, Jun 30, 2016 at 10:21:03AM +0200, Mathieu Arnold wrote: |> |> |> +--On 30 juin 2016 07:41:09 +0200 Baptiste Daroussin |> wrote: |> | On Wed, Jun 29, 2016 at 04:33:52PM -0700, Bryan Drewery wrote: |> |> On 6/23/2016 4:31 AM, Mathieu Arnold wrote: |> |> > Author: mat |> |> > Date: Thu Jun 23 11:31:15 2016 |> |> > New Revision: 417364 |> |> > URL: https://svnweb.freebsd.org/changeset/ports/417364 |> |> > |> |> > Log: |> |> > Extend the symlink checks to warn on absolute targets. |> |> > |> |> > Sponsored by: Absolight |> |> > Differential Revision: https://reviews.freebsd.org/D6930 |> |> |> |> Why? This seems like noise. |> | |> | Quite true as it is I overlooked it. |> | |> | To ease relocatable packages (to prepare sysroot for exemple) symlinks |> | that points to file within the same package should be absolute links. |> | |> | After looking at this patch again I think it should be reverted and |> | replaced later by something that will check symlinks on the same plist |> |> Mmmm, I can add a check to see if the target is in STAGEDIR and only emit |> the warning if it is. | | would definitly help I did that in r417821 (now poudriere doesn't have any symlinks warnings) I added a second warning if the symlink points to something that does't exist anywhere, feel free to remove it if it is annoying too :-) -- Mathieu Arnold --==========A71BC19C8234FC6D68AC========== Content-Type: application/pgp-signature Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQJ8BAEBCgBmBQJXdQqQXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQzQUI2OTc4OUQyRUQxMjEwNjQ0MEJBNUIz QTQ1MTZGMzUxODNDRTQ4AAoJEDpFFvNRg85IjbMQAKvU9gEKo4xileq7U6xDxJWR /hWfL4TOSHEaw+geNRFoGUS3b0LFCf+nTnWAu36aYnLB+RlNfXsX266AVEjBpcUR JIT4FlRWATgkcFI6txuxDCSFV4/UiXIMscyB3PdrTZAnlUIAO6C1LsZWFA1FAHg2 k94YlCcUfp85uVIJw+xHMPsU0U1Y2l0OmGEeug2vIzH0SZmw3Yqp2+GDY2n4K6xt adpYXqHxGQC2dtkgziJH4nt0cM3zmQV12YP9oxwMxamxXnJ/4bPOsHL2oPgVJvYe ZHBlJ0hNIRuCUcG+Cq7KN+ox1CR7NWI8uPoovPGLtD5iqNuc2yfh3Zzs5KTywZyi yi7Zglzv8rDloPpxOhlxVOlz0+kWww+znu821Nz9ILpMMXsMTe6l5Yd6AuNo8ma2 qszE04KCX7Yg1zg1m7LQFwOwXOpPxVk/3tNHipNceVj5w2TA0/vrz7DerZZxHoJK uxdmyOJJwzNaYsOeNNR3tcFZd8r2KjOV+ZUg2WpeVWQl2CWQNDAcdHm1Zt1F2phf OVmEgVWLgoW2pVLwG7nLYbZVc7TcWGhwGj82svewcmm0vKLH2K3b6BJ02nFBD9wZ 0OtN4o1SMVZCYeMvLBzW011rWp0gowVyLMImaYHOLib4VUKUBjn2nJlNgB6Rq8RM 8pZWENDfMtkdPR5XTC9L =SCVw -----END PGP SIGNATURE----- --==========A71BC19C8234FC6D68AC==========--