From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 40442 invoked by alias); 7 Feb 2020 10:08:41 -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 40434 invoked by uid 89); 7 Feb 2020 10:08:40 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.1 spammy=H*RU:209.85.128.65, H*MI:sk:CAH6eHd, H*f:sk:xUeZE5p, HX-Spam-Relays-External:209.85.128.65 X-HELO: mail-wm1-f65.google.com Received: from mail-wm1-f65.google.com (HELO mail-wm1-f65.google.com) (209.85.128.65) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 07 Feb 2020 10:08:39 +0000 Received: by mail-wm1-f65.google.com with SMTP id t14so2002300wmi.5 for ; Fri, 07 Feb 2020 02:08:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Ub+0AysFCtCT1r0+GuFsY+s9tSuJ9zj8+fYXokOY0ow=; b=U5hi1uPWReiuHquwQeI+hzxp/NvSKqk7vXxd7hljTTj3v004zDcbLwBtb+lkbZBQYF dQTLTaiPPJXDEpRGwVsvVfTrthGACYXcd5dmQ35ecROz4/kCQyfTovEjURC1+VjVnC/6 ays+l7egKtwsFDQ3r+yKAGFYJTYMMcMr74sqHMoOJ4gY258D0r/x95m6p/q4MnY38y4B +NE0Iho4SE3CYcSeMYLcX1PiDTk5y6GEGXldua72RPRr9jfm2wdTNAWwPnlNo2yv355k ttayzHWvoVwxV0AoAZYI2KjJWq/L0SwqDFwMDzeM5zzVoheXNns+cjpgBJvyCBV9mbLZ czkw== MIME-Version: 1.0 References: <20200124214533.GB17695@tucnak> <6bc259050138912ea9b973ca4f43df72cb1bb7d8.camel@redhat.com> <20200205211801.GV22482@gate.crashing.org> <7743417118d2067cf0e9e1b840564f9f2d5e72dc.camel@redhat.com> <20200206135101.GY22482@gate.crashing.org> <20200206222548.GH22482@gate.crashing.org> In-Reply-To: From: Jonathan Wakely Date: Fri, 07 Feb 2020 10:08:00 -0000 Message-ID: Subject: Re: Git ChangeLog policy for GCC Testsuite inquiry To: Richard Biener Cc: Segher Boessenkool , Jeff Law , Richard Sandiford , "H.J. Lu" , Paul Smith , GCC Development Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2020-02/txt/msg00080.txt.bz2 On Fri, 7 Feb 2020 at 09:20, Richard Biener wrote: > > On Thu, Feb 6, 2020 at 11:25 PM Segher Boessenkool > wrote: > > Instead of "git am" I had "patch -p1 <", distributing the changelog parts > > I just did in vi (as with git), then "svn ci", which pick up all modified > > files directly (sometimes an "svn add" first). It's pretty much the same > > for me. > > I'm a believer on committing from the tree I actually tested, so it With Git you can't really have unwanted local commits present in a tree if you use a sensible workflow, so if you tested in a tree that was at commit 1234abcd and you push from another machine that is at the same commit, you know there are no unintended differences. > was with SVN: patch, test, svn up, edit changelogs (cut&paste from the > patch header), svn diff | less (check wha'ts abotu to be committed), svn commit > with GIT its now: patch, test, git add/commit, There is no good reason to delay add+commit until after testing. Committing locally is a completely different operation from committing to svn, because nothing is "final" until you push to gcc.gnu.org. I commit locally, and then either test locally or push to larger machines where I test, or test in both places at once. Even if you only work on one machine, keeping uncommitted changes around is error prone (it's easier to lose the work, it's easier to not realise exactly what you're testing, it's harder to verify that what you tested is what you eventually pushed). Applying somebody else's patch to test it can be done in a local branch and committed locally. You can always delete that branch later to clear up (a naming convention like "patch-20200207-foo-bar" helps to keep those branches clearly labelled so you can easily decide which to remove later). Once you've committed something it has a commit hash to identify it, so you can easily compare the state at that revision to some other revision, at any later time. tl;dr make more use of branches.