guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#64151] [PATCH] etc: Stop making sendemail behave strangely.


From: Liliana Marie Prikler
Subject: [bug#64151] [PATCH] etc: Stop making sendemail behave strangely.
Date: Sat, 08 Jul 2023 19:16:28 +0200
User-agent: Evolution 3.46.4

Am Donnerstag, dem 06.07.2023 um 11:49 -0400 schrieb Maxim Cournoyer:
> Hi Ludo,
> 
> Ludovic Courtès <ludo@gnu.org> writes:
> 
> > Hi,
> > 
> > Liliana Marie Prikler <liliana.prikler@gmail.com> skribis:
> > 
> > > Even with such a hypothetical --exclude-whomever switch added,
> > > I'd argue that it is wrong to magically install this
> > > configuration without any user interaction.
> > 
> > Maybe that’s the main grief: that the new config was suddenly
> > installed without fellow hackers realizing it (I, for one, wondered
> > initially what magic made ‘git send-email’ Cc: all my friends :-)).
> > 
> > Perhaps one way out is to suggest it in the manual without
> > installing it automatically?  WDYT, Maxim?
> 
> I still think it has more value by being automatically installed, as
> that way the pre-push hook that prevents unsigned commit is also
> configured automatically, and new contributors would be likely to not
> have it configured (by having missed it from the manual).
> 
> I see this similar to the .dir-locals.el file that automatically
> setups (there's a prompt, but its design forces you into accepting it
> or being repeatedly bothered ad aeternam).
There is also a way of enforcing a "no", though.  And in particular,
the version-controlled .dir-locals.el has only been really supported
since the introduction of .dir-locals-2.el, which allows local changes.

> If there are concrete issues with it, I think we should a) report
> them and b) look into fixing them.
Such as the concrete issue that it changes our workflow without asking?
Mind you, that I'm only not affected because I've been format-patching
manually for a long time at this point.

Cheers

> 





reply via email to

[Prev in Thread] Current Thread [Next in Thread]