From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 48925 invoked by alias); 16 Dec 2019 22:42:58 -0000 Mailing-List: contact gcc-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-owner@gcc.gnu.org Received: (qmail 48916 invoked by uid 89); 16 Dec 2019 22:42:57 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-2.3 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy=claimed, H*i:sk:fe11dcc, love, quality X-HELO: gate.crashing.org Received: from gate.crashing.org (HELO gate.crashing.org) (63.228.1.57) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 16 Dec 2019 22:42:55 +0000 Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.14.1) with ESMTP id xBGMgl4F007578; Mon, 16 Dec 2019 16:42:48 -0600 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id xBGMgi5T007577; Mon, 16 Dec 2019 16:42:44 -0600 Date: Mon, 16 Dec 2019 22:42:00 -0000 From: Segher Boessenkool To: Jeff Law Cc: Joseph Myers , Mark Wielaard , Maxim Kuvyrkov , "Richard Earnshaw (lists)" , gcc@gcc.gnu.org, esr@thyrsus.com Subject: Re: Proposal for the transition timetable for the move to GIT Message-ID: <20191216224244.GI3152@gate.crashing.org> References: <1685e719-738f-dd4e-c39c-c08e495b202e@arm.com> <9E009921-96EA-44A2-A06A-232711227E69@linaro.org> <0fb81074d87c96b3312565800b8bfc25cfcbe179.camel@redhat.com> <20191216215927.GG3152@gate.crashing.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.3i X-IsSubscribed: yes X-SW-Source: 2019-12/txt/msg00254.txt.bz2 Hi! On Mon, Dec 16, 2019 at 03:13:49PM -0700, Jeff Law wrote: > On Mon, 2019-12-16 at 15:59 -0600, Segher Boessenkool wrote: > > In particular, we should look carefully at the commit > > > attributions in both conversions and Maxim's may well give ideas for > > > improving the reposurgeon changelogs command (Richard came up with three > > > ideas recently, which I've just filed in the reposurgeon issue tracker). > > > But I also think: > > > > > > * reposurgeon is a safer approach than ad hoc scripts, provided we get > > > clean verification of basic properties such as branch tip contents. > > > > I totally do not agree. Black boxes are not safe. *New* black boxes > > are even worse. > > > > I trust scripts that have low internal complexity much better. > Perhaps. But there's also limits to what scripts can do. Absolutely. Just this trust in complicated things is very misplaced, in my opinion. > > There is absolutely no reason to trust a system that supposedly was > > already very mature, but that required lots of complex modifications, > > and even a complete rewrite in a different language, that even has its > > own bug tracker, to work without problems (although we all have *seen* > > some of its many problems over the last years), and at the same time > > bad-mouthing simple scripts that simply work, and have simple problems. > I'd disagree. THe experience and testsuites from that system are a > significant benefit. That isn't what I said. I said that freshly constructed complex software will have more and deeper errors than stupid simple scripts do (or I implied that at least, maybe it wasn't clear). And I only say this because the opposite was claimed, which is laughable imnsho. > > > * Richard's improvements to commit messages are a great improvement to the > > > resulting repository (and it's OK if a small percentage end up misleading > > > because someone used the wrong PR number, sometimes people use the wrong > > > commit message or commit changes they didn't mean to and so having some > > > misleading messages is unavoidable). > > > > As long as the original commit message is kept, verbatim, and you only > > add a new summary line, all is fine. If not -> nope, not okay. > Sorry, have to disagree here. I think what Richard has done is a > significant step forward. We talked about it for days, and as far as I understand it Richard agreed. But, there is no way I can verify this yet, or is there? Is there a repo we can look at? Something close to final. > > > * As we're part of the free software community as a whole rather than > > > something in isolation, choosing to make a general-purpose tool work for > > > our conversion is somewhat preferable to choosing an ad hoc approach > > > because it contributes something of value for other repository conversions > > > by other projects in future. > > > > This, I don't agree with at all either: having some lean-and-mean > > scripts that worked for the GCC conversion will be at least as helpful > > for another conversion as a "general" tool that first requires you to > > build a custom machine before you can use it properly, would be. > > > > > > Anyway: yes, please verify all conversion candidates for your criteria. > > Thanks! > And if they're the same, then I'm still going to prefer reposurgeon. But they aren't the same, so you will have to make an actual choice, based on actual data ;-) > So unless there's something Maxim's scripts are getting right that > aren't by reposurgeon, then reposurgeon is the right choice. ... in your opinion. Anyway, I'd love too hear Richard's input too, but we will have to wait for that a few days. The quality of the conversion should be judged by the output of it, not by anything else. You do not want to see how sausage is made, as they say. The GCC community has nothing to gain from a generic conversion tool: what we want and need is *one* conversion, and it should be a good one. So let us compare *that*! Segher