From owner-svn-ports-head@freebsd.org Wed Aug 23 15:49:25 2017 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 74A0BDEA0DE; Wed, 23 Aug 2017 15:49:25 +0000 (UTC) (envelope-from romain@marvin.blogreen.org) Received: from marvin.blogreen.org (blogreen.org [151.127.25.53]) by mx1.freebsd.org (Postfix) with ESMTP id 29021754C2; Wed, 23 Aug 2017 15:49:23 +0000 (UTC) (envelope-from romain@marvin.blogreen.org) Received: by marvin.blogreen.org (Postfix, from userid 1001) id 1B9FFA125A; Wed, 23 Aug 2017 17:49:22 +0200 (CEST) Date: Wed, 23 Aug 2017 17:49:22 +0200 From: Romain =?iso-8859-1?Q?Tarti=E8re?= To: Mathieu Arnold Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r448620 - in head: devel/leatherman/files devel/llvm-devel/files sysutils/facter/files x11-wm/cde/files Message-ID: <20170823154922.GA47318@blogreen.org> References: <201708231403.v7NE38mL020611@repo.freebsd.org> <20170823144023.GD13824@blogreen.org> <92421b70-4930-f0f0-3d04-c225c00ab7e9@FreeBSD.org> <20170823145830.GE13824@blogreen.org> <3180d43a-c87d-54cb-7a6a-61ff05669c58@FreeBSD.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="x+6KMIRAuhnl3hBn" Content-Disposition: inline In-Reply-To: <3180d43a-c87d-54cb-7a6a-61ff05669c58@FreeBSD.org> X-PGP-Key: http://romain.blogreen.org/pubkey.asc User-Agent: Mutt/1.8.3 (2017-05-23) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.23 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: Wed, 23 Aug 2017 15:49:25 -0000 --x+6KMIRAuhnl3hBn Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Aug 23, 2017 at 05:32:53PM +0200, Mathieu Arnold wrote: > I do not really understand what this patch is trying to achieve >=20 > Baptiste and I only talked about patches, there are no patches in it. Okay, I might be missing something basic=E2=80=A6 $FreeBSD$ may be expanded to something that gives information about the svn revision the file was last modified. For this to happen, the 'svn:keywords' property should be set to something like 'FreeBSD=3D%H'. At some point, it was decided that this information was useless for patches and such lines where removed from them. I don't really see value here for the removal, but I agree that those keywords did not really bring value neither, so this looks OK to me. We have many files with "$FreeBSD$" in them and without svn:keywords set. There is therefore no keyword expansion in them, and I don't see more value added by these "$FreeBSD$" strings in these files than in patches=E2=80=A6 Hence the proposed patch to remove them. If these non-expanded keyword make sense for non-patches and do not make sense for patches, please explain this to me why, because I really don't understand the difference :-S. Thanks! --=20 Romain Tarti=C3=A8re http://people.FreeBSD.org/~romai= n/ pgp: 8234 9A78 E7C0 B807 0B59 80FF BA4D 1D95 5112 336F (ID: 0x5112336F) (plain text =3Dnon-HTML=3D PGP/GPG encrypted/signed e-mail much appreciated) --x+6KMIRAuhnl3hBn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAEBCAAdFiEEgjSaeOfAuAcLWYD/uk0dlVESM28FAlmdpAEACgkQuk0dlVES M29oywv8DBIAbxf1eOiJ5X5vpDnICqa+k6nGcB9h8g86I/UBGVvENct1dpk2MK8G Mtk0eD9DIcBsWPa+J4syRxahaxu+bZctlly+Je2GyJoDJHh1kJM4D8tvR9FCAsam 0QyvRJyqT5pH7eYWwypuw3quKE85zExCjTLmbk9J1hdNQErVCIrf/Pz9YHqxDqMd eNQVuOhhtWTs81VnrYm19+PnmxyU9l2zKpVBZyYtpwNMVqJbHEFoYw2u7KqRcGOp HcDl8rCwV2zckdL0eYgmmJWcd82ep1GLoMC6nbsWkMcZDXRsmNd/7aPwL+IrRUtg ZGG6rhd5LqNsr6Fncgkije+g08Kv36XZkHNE9TkAKs6+JeGTTJ7L/Vy7CY7KAvhN hFH3ukznAKZE3ukLhN5f3ucWT2xXu4r3keGxWS+gKYGgObJclkhnB3SrSUHBqtyX yTWTyaqJzyZfxh1S1HO/feitFL24Jl/IBYlN02IHDsHTdPMwIF0kvKcNOHgT8FCh 2vvvxAAt =yT79 -----END PGP SIGNATURE----- --x+6KMIRAuhnl3hBn--