From owner-svn-ports-head@freebsd.org Mon Jan 30 13:48:53 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 58A97CC7B73; Mon, 30 Jan 2017 13:48:53 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: from shepard.synsport.com (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 2E1EE895; Mon, 30 Jan 2017 13:48:52 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: from [127.0.0.1] (ip72-204-83-236.fv.ks.cox.net [72.204.83.236]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.com (Postfix) with ESMTP id DC1CD43C10; Mon, 30 Jan 2017 07:47:12 -0600 (CST) Subject: Re: svn commit: r432561 - head/lang/php71 To: Torsten Zuehlsdorff , marino@freebsd.org, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org References: <201701271852.v0RIqOvW033749@repo.freebsd.org> <6aebcd6a-0439-f23a-be9d-e06f46d3511b@toco-domains.de> <4dd6d654-4525-c440-83b1-6a22215f6020@toco-domains.de> <5725262c-4152-c711-e53b-a509742bcba1@marino.st> Cc: Mathieu Arnold Reply-To: marino@freebsd.org From: John Marino Message-ID: Date: Mon, 30 Jan 2017 07:48:48 -0600 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <5725262c-4152-c711-e53b-a509742bcba1@marino.st> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Antivirus: avast! (VPS 170130-0, 01/30/2017), Outbound message X-Antivirus-Status: Clean 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: Mon, 30 Jan 2017 13:48:53 -0000 On 1/30/2017 07:28, John Marino wrote: > On 1/30/2017 05:15, Torsten Zuehlsdorff wrote: >> >> I did exactly this. 2 months ago there was no DTRACE in php70. So >> copying the working-copy of php71 over the repo-copy would have not >> included the DTRACE option. Therefore the repo-copy wouldn't have had >> helped. > > EXACTLY. > Your mistake is leaving this at two months ago. > I meant check differences the day of committing, not check differences 2 > months ago. If you had not assumed that zero changes happened to php70 > that would affect php71 in a two month period, then you would rechecked > the diff before you actually commit and seen them. I would have noticed > the changes because I always run "svn diff" before committing. > > >> >>> You misunderstood r432567. >>> That wasn't to correct my commit. >>> That was to add the second missing item, DTRACE for aarch. >>> >>>> I just noted in this 2 month how hard it is to find somebody for review >>>> and second how hard it is to get such a patch reviewed. First the >>>> review.freebsd.org didn't work for me because it contains a bug with >>>> umlaut for years. And second a repocopy could not be displayed. >>> >>> Yes, the diff would have been displayed. >>> That's the benefit of repocopy. >> >> Maybe i wasn't clear enough: the repo-copy could not displayed in >> review.freebsd.org. If you say: its possible. Okay, than its just the >> umlaut-bug preventing the display of my patch. > > I am not talking about review.freebsd.org. > I'm talking about svn. > > >> But - no, you are not right this time. Doing a repo-copy without >> changing the copy will cause a svn diff to just list the files. The diff >> didn't even contain the revision copied from. To bring a path like this >> into the review tool you need at least the --show-copies-as-adds param, >> which defeats the purpose of a repo-copy in review. > > I don't think anybody was talking about review. > If you do repocopy correctly, we all get a commit mail that shows the > differences between the new version and the version it was copied from. > In this case, we got what looked like complete files added which is how > mat know that svncopy wasn't used. > To clarify, I get that you're saying that a repocopy you performed 2 months ago would have been based on the state of php71 two month ago. I'm basically saying that after 2 months, I either would have rebased the original proposal on the current php71 or at least reviewed each addition to php71 in the last two months. At some point the proposal "expires" and I think 2 months is definitely beyond that point. John --- This email has been checked for viruses by Avast antivirus software. https://www.avast.com/antivirus