From owner-svn-ports-head@FreeBSD.ORG Tue Sep 2 06:50:10 2014 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from ptrcrt.ch (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 77B77A53; Tue, 2 Sep 2014 06:50:08 +0000 (UTC) Date: Tue, 2 Sep 2014 06:50:05 +0000 From: Pietro Cerutti To: marino@freebsd.org Subject: Re: svn commit: r366841 - head/lang/tcl86/files Message-ID: <20140902065005.GC94467@ptrcrt.ch> Reply-To: gahr@FreeBSD.org References: <201409010731.s817Vrxf062753@svn.freebsd.org> <20140901074609.GA32100@FreeBSD.org> <65B530D9-4740-4A60-A2F5-40335A520C4E@adamw.org> <54048A3B.4030001@marino.st> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="tsOsTdHNUZQcU9Ye" Content-Disposition: inline In-Reply-To: <54048A3B.4030001@marino.st> X-PGP-Key: fp="DA6D E106 A5B8 54B8 5DD8 6D49 ADD0 D38E A192 089E"; id="0xA192089E"; get=; get=; User-Agent: Mutt/1.5.23 (2014-03-12) Cc: svn-ports-head@freebsd.org, Alexey Dokuchaev , ports-committers@freebsd.org, svn-ports-all@freebsd.org, Adam Weinberger X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.18-1 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: Tue, 02 Sep 2014 06:50:10 -0000 --tsOsTdHNUZQcU9Ye Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2014-Sep-01, 17:01, John Marino wrote: > On 9/1/2014 16:47, Adam Weinberger wrote: > > On 1 Sep, 2014, at 3:46, Alexey Dokuchaev wrote: > >> Since when did we abandon the practice of doing 'svn diff' before > >> commit? > >=20 > > Alexey, please direct your frustration elsewhere. Pietro fixed > > (hopefully) a problem that caused a lot of tcl-dependent ports to > > break. He should be commended, not sneered at. If you don=E2=80=99t lik= e the > > way makepatch works, take it up with portmgr, not with the committers > > who use it. I know for a fact that portmgr is interested in extending > > makepatch to DTRT. > >=20 > > Besides, there are simply situations where makepatch is just not the > > tool to use. Sometimes I *have* to run diff(1) by hand. Sometimes I > > need to run "make patch=E2=80=9D before I can edit the diff. Sometimes = I=E2=80=99ve > > edited multiple files and I want to test one diff at a time. > > Sometimes a file needs to be touched by two different patches (I > > don=E2=80=99t like it, but ten years ago it seems like people really di= d). >=20 >=20 > While I also pushed back against danfe@ on this post, it's obvious that > makepatch was used. The commit didn't do anything except change the > timestamp. I'm sorry to contradict the obvious, but I've never ever used makepatch in my life. I didn't even know it existed before danfe@'s email. I always generated my patch with diff -uN. Moreover, the commit clearly *did* do things other than change the timestamp. https://svnweb.freebsd.org/ports/head/lang/tcl86/files/patch-unix-Makefile.= in?r1=3D366841&r2=3D366840 --=20 Pietro Cerutti The FreeBSD Project gahr@FreeBSD.org PGP Public Key: http://gahr.ch/pgp --tsOsTdHNUZQcU9Ye Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQJ8BAEBCgBmBQJUBWidXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXREQTZERTEwNkE1Qjg1NEI4NUREODZENDlB REQwRDM4RUExOTIwODlFAAoJEK3Q046hkgiex3UP/0BFbOrggaEx5wTUlGN4F/wv aFJMVypmqZjgiXY/Yw+tsgGw7/EAXhUMbS5/5YcVgDMo1U1MOgGlF+bZUmiGT9kA p3Cmij8wZyv6avCwq2seJKpzPFHl4Z+GNnI3NqjEbxqXfzEGWGBZx61SXfhDXDFp Sns6xI/fDxjdtPV5YNnVv7iwOGXqL3cR1HRA+VwXcSmSBMXmT0dLF0jR1XqGtsZU yZjkYbIPyrYyy9UT06Gs59Tz0A/l6KVefY68T269zH4/6fvgNkg6zIWzu7K5aAsM xO0gscNQdsF8yni974AB8r+90xRBN+Dn+vQELFIH0IzCa6YbG+KdWlRB661yBnDw sThtIJuvLUyiJAqfdr60r7zySNHMmieKo1z42US1G8VKVYW2LBlosyXZLaVjaVUa lAbRKLbCaNyup6cCsZHsJAOKABDJToAzwufSYB09bljZZqY6kUW5SBlCs3fLOpUR klopLC6ipCqh1yfWAD84Xk+Kvvx0p8qYo+MY3llK1YLkZkbaNqGz1Z3QyEeB8E3J Jj6tDlqlzOB38DHj0TVA372ab32RY3Ig462/2MkrVpBEoG29bzAbgobnFwsowptm 4lNPWxyHgXfYtn4D80QjUsEgiWnPlCkKT9U1xvbSHsgwgJKyQM+AhyM0sYxVe0Ql iGj8gNA6RbNZdqRzGvKR =+emA -----END PGP SIGNATURE----- --tsOsTdHNUZQcU9Ye--