[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Should commit signing always be required for local work? [was Re: bu
From: |
Maxim Cournoyer |
Subject: |
Re: Should commit signing always be required for local work? [was Re: bug#63261: Recent changes to git config cause errors for non-committers] |
Date: |
Thu, 18 May 2023 23:24:23 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi Leo,
Leo Famulari <leo@famulari.name> writes:
> On Mon, May 15, 2023 at 09:38:33AM -0400, Maxim Cournoyer wrote:
>> Hi Leo,
>>
>> Leo Famulari <leo@famulari.name> writes:
>>
>> > Can someone help me by pointing to the original discussion of this
>> > change?
>>
>> It was the team.scm issue in https://issues.guix.gnu.org/58813 that led
>> to this change.
>
> Thanks for pointing it out.
>
> In my opinion, this change should be discussed more publicly so that a
> consensus can be reached about it.
>
> This change affects the workflow of everyone that hacks on Guix,
> including people who are trying it for the very first time.
>
> It's a big change.
>
> It's not okay to only discuss such a big change in a ticket with the
> title "can't substitute etc/teams.scm command as doc suggests".
>
> I am asking for this to be reverted until a consensus is reached.
Thanks for voicing your thoughts on this. I originally thought the
included fragments (via the 'include.path' git config option) could we
overridden by a user but it seems they can't, making the change more
intrusive than it should have been.
I've reverted just the offending bits:
--8<---------------cut here---------------start------------->8---
diff --git a/etc/git/gitconfig b/etc/git/gitconfig
index 831fbce6e7..907ad01804 100644
--- a/etc/git/gitconfig
+++ b/etc/git/gitconfig
@@ -1,6 +1,3 @@
-[commit]
- gpgsign = true
-
[diff "scheme"]
xfuncname = "^(\\(define.*)$"
--8<---------------cut here---------------end--------------->8---
Let me know if something is amiss.
--
Thanks,
Maxim