From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 59924 invoked by alias); 6 Feb 2020 16:18:02 -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 59915 invoked by uid 89); 6 Feb 2020 16:18:02 -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=40PM, 40pm 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; Thu, 06 Feb 2020 16:18:00 +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 016GHtvd017699; Thu, 6 Feb 2020 10:17:56 -0600 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 016GHsXd017698; Thu, 6 Feb 2020 10:17:54 -0600 Date: Thu, 06 Feb 2020 16:18:00 -0000 From: Segher Boessenkool To: Jakub Jelinek Cc: Jeff Law , Richard Sandiford , "H.J. Lu" , paul@mad-scientist.net, GCC Development Subject: Re: Git ChangeLog policy for GCC Testsuite inquiry Message-ID: <20200206161754.GE22482@gate.crashing.org> References: <1693150.uSzEE5ffML@polaris> <20200124214533.GB17695@tucnak> <6bc259050138912ea9b973ca4f43df72cb1bb7d8.camel@redhat.com> <20200205211801.GV22482@gate.crashing.org> <7743417118d2067cf0e9e1b840564f9f2d5e72dc.camel@redhat.com> <20200206145640.GK17695@tucnak> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200206145640.GK17695@tucnak> User-Agent: Mutt/1.4.2.3i X-IsSubscribed: yes X-SW-Source: 2020-02/txt/msg00070.txt.bz2 On Thu, Feb 06, 2020 at 03:56:40PM +0100, Jakub Jelinek wrote: > On Wed, Feb 05, 2020 at 06:43:54PM -0700, Jeff Law wrote: > > And FWIW, we're talking about the ChangeLog *file* here. If folks > > continued writing the same log messages and put them into git, I > > personally think that's sufficient to transition away from having a > > ChangeLog file in the source tree. > > A precondition for such change would be a script that would verify > that each commit being pushed into the repository has the ChangeLog snippets > available. And not just that. With ChangeLog files, one can fix up the > entries later on if somebody makes a mistake; it happens from time to time > to everybody, some people make the same mistakes again and again. Yes. Mistakes *will* happen, and for *everyone*. > If the ChangeLog entry is in the commit message, then in git (unlike svn) it > is not possible to fix it later once you push it into the public repository. > > So, we'd need a script that checks for the common mistakes, furthermore > verifies that e.g. all files in the commit that need to have a ChangeLog > entry have one etc. and reports back any issues + rejects the push. We also need a way to fix changelog entries for the errors that do seep through (and that are bad enough that they do need fixing). It doesn't have to be easy or convenient, but we need *some* way to do it. > We would need to agree how do we express stuff going into different former > ChangeLog files, whether we require gcc/cp/ etc. prefixes before the lines, > or say require empty line for different former ChangeLog files and let the > extraction script figure it out from the changed files in the commit, etc. The latter seems error-prone, but maybe I worry too much :-) > We could have a rule that say New test. entries for newly added files > in */testsuite/* would be optional and let the generating script add those > (but say require entries if existing tests have been changed). How much work does that safe? > Plus the extraction script, and test properly both scripts for a while > before making it live. Yes, absolutely. Maybe it could post it to gcc-cvs@ (as it will in the future) (but not actually commit the (generated) changelog parts yet)? Segher