Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 13 Apr 2021 13:53:07 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        Alex Richardson <arichardson@freebsd.org>, src-committers <src-committers@freebsd.org>,  "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-main@freebsd.org
Subject:   Re: git: 73c14cc76b5f - main - Remove history.immutable from .arcconfig
Message-ID:  <CANCZdfpaF3jZUExMsrhwaZv1Fg1TrUJntiqPCxiXrrf5kRN6Ow@mail.gmail.com>
In-Reply-To: <9e0d4574-bcad-b24c-78ad-c172d6d2c221@FreeBSD.org>
References:  <202104131138.13DBcgrt085662@gitrepo.freebsd.org> <9e0d4574-bcad-b24c-78ad-c172d6d2c221@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Apr 13, 2021 at 9:23 AM John Baldwin <jhb@freebsd.org> wrote:

> On 4/13/21 4:38 AM, Alex Richardson wrote:
> > The branch main has been updated by arichardson:
> >
> > URL:
> https://cgit.FreeBSD.org/src/commit/?id=73c14cc76b5f815c6b1bd93089ebafdd9269d343
> >
> > commit 73c14cc76b5f815c6b1bd93089ebafdd9269d343
> > Author:     Alex Richardson <arichardson@FreeBSD.org>
> > AuthorDate: 2021-04-13 11:36:24 +0000
> > Commit:     Alex Richardson <arichardson@FreeBSD.org>
> > CommitDate: 2021-04-13 11:36:25 +0000
> >
> >      Remove history.immutable from .arcconfig
> >
> >      The `history.immutable` setting prevents arcanist from updating
> >      the commit messages with the Differential URL and therefore
> >      makes updating patches awkward with a rebase workflow.
> >
> >      In case this new behaviour is not wanted the old one can be restored
> >      by running `arc set-config --local history.immutable true`.
> >
> >      Test Plan: `arc diff --create HEAD^` adds the metadata now.
> >
> >      Reviewed By:    #phabric-admin, imp, lwhsu
> >      Differential Revision: https://reviews.freebsd.org/D27971
>
> FYI, this might break git arc since arc will now start rewriting commits
> rather than leaving them alone.  If so, we can look use 'arc diff'
> with --head to turn off some of its magic.  It would also avoid having
> 'git arc' have to adjust the working copy.
>

I'd honestly rather have 'git arc create' stamp this in from the git go.
I really dislike having it not there until just before I go to commit.

Warner



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpaF3jZUExMsrhwaZv1Fg1TrUJntiqPCxiXrrf5kRN6Ow>