From owner-svn-ports-all@FreeBSD.ORG Mon Sep 1 15:01:52 2014 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 859D3852; Mon, 1 Sep 2014 15:01:52 +0000 (UTC) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 5998F1786; Mon, 1 Sep 2014 15:01:52 +0000 (UTC) Received: from [192.168.0.22] (unknown [130.255.19.191]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id EE03343BC5; Mon, 1 Sep 2014 10:01:33 -0500 (CDT) Message-ID: <54048A3B.4030001@marino.st> Date: Mon, 01 Sep 2014 17:01:15 +0200 From: John Marino Reply-To: marino@freebsd.org User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Adam Weinberger , Alexey Dokuchaev Subject: Re: svn commit: r366841 - head/lang/tcl86/files References: <201409010731.s817Vrxf062753@svn.freebsd.org> <20140901074609.GA32100@FreeBSD.org> <65B530D9-4740-4A60-A2F5-40335A520C4E@adamw.org> In-Reply-To: <65B530D9-4740-4A60-A2F5-40335A520C4E@adamw.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org, Pietro Cerutti X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Sep 2014 15:01:52 -0000 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? > > 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’t like 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. > > 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” before I can edit the diff. Sometimes I’ve > 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’t like it, but ten years ago it seems like people really did). 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. Danfe *did* rework makepatch, I helped him, and it's stuck in review. His proposed version would not have resulted in a commit that just changed timestamps. It does solve this problem. > If you REALLY want to solve that issue, make a change to > src/gnu/usr.bin/diff so that it will take a flag to not put > timestamps in there. As I said, the issue has been solved, and the solution is good. Nobody dislikes the new changes internally*, but patch-naming has turned into a impasse. John * since today, antoine says he thinks -p option on diff is "ugly". it's a highly useful option so now we have yet another hurdle to jump. If not for phabric we could have had this in ports weeks ago, but now are stuck in an impasse (which I suspect was the outcome desired by the people that wanted it reviewed in phabric tbh)