bug#69096: [PATCH] Set default value for 'vc-default-patch-addressee'

bug#69096: [PATCH] Set default value for 'vc-default-patch-addressee'

[Philip Kaludercic]

[-- Attachment #1: Type: text/plain, Size: 139 bytes --]

Apparently we forgot to set this option back when it was added, but it
makes sense to have it along with 'vc-prepare-patches-separately'.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: [PATCH] ; * .dir-locals.el: Set 'vc-default-patch-addressee' --]
[-- Type: text/x-patch, Size: 1046 bytes --]

From a9be9adaf6ae0c30a62d98c1705cfb308cffd69f Mon Sep 17 00:00:00 2001
From: Philip Kaludercic <philipk@posteo.net>
Date: Tue, 13 Feb 2024 10:52:24 +0100
Subject: [PATCH] ; * .dir-locals.el: Set 'vc-default-patch-addressee'

---
 .dir-locals.el | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/.dir-locals.el b/.dir-locals.el
index 89fb76a55f3..58fc135e202 100644
--- a/.dir-locals.el
+++ b/.dir-locals.el
@@ -14,7 +14,8 @@
             ("/[ \t]*DEFVAR_[A-Z_ \t(]+\"\\([^\"]+\\)\"/\\1/"
              "/[ \t]*DEFVAR_[A-Z_ \t(]+\"[^\"]+\",[ \t]\\([A-Za-z0-9_]+\\)/\\1/"))))
          (etags-regen-ignores . ("test/manual/etags/"))
-         (vc-prepare-patches-separately . nil)))
+         (vc-prepare-patches-separately . nil)
+         (vc-default-patch-addressee . "bug-gnu-emacs@gnu.org")))
  (c-mode . ((c-file-style . "GNU")
             (c-noise-macro-names . ("INLINE" "NO_INLINE" "ATTRIBUTE_NO_SANITIZE_UNDEFINED"
                                     "UNINIT" "CALLBACK" "ALIGN_STACK" "ATTRIBUTE_MALLOC"
-- 
2.43.0

bug#69096: [PATCH] Set default value for 'vc-default-patch-addressee'

[Philip Kaludercic]

Philip Kaludercic <philipk@posteo.net> writes:

> Apparently we forgot to set this option back when it was added, but it
> makes sense to have it along with 'vc-prepare-patches-separately'.

ping?

bug#69096: [PATCH] Set default value for 'vc-default-patch-addressee'

[Eli Zaretskii]

> From: Philip Kaludercic <philipk@posteo.net>
> Date: Fri, 03 May 2024 07:36:34 +0000
> 
> Philip Kaludercic <philipk@posteo.net> writes:
> 
> > Apparently we forgot to set this option back when it was added, but it
> > makes sense to have it along with 'vc-prepare-patches-separately'.
> 
> ping?

Dmitry?

I'm okay with this, FWIW.

bug#69096: [PATCH] Set default value for 'vc-default-patch-addressee'

[Dmitry Gutov]

Hi Eli,

On 04/05/2024 14:44, Eli Zaretskii wrote:
>> From: Philip Kaludercic<philipk@posteo.net>
>> Date: Fri, 03 May 2024 07:36:34 +0000
>>
>> Philip Kaludercic<philipk@posteo.net>  writes:
>>
>>> Apparently we forgot to set this option back when it was added, but it
>>> makes sense to have it along with 'vc-prepare-patches-separately'.
>> ping?
> Dmitry?
> 
> I'm okay with this, FWIW.

I didn't think this was my area, but the change also LGTM.

bug#69096: [PATCH] Set default value for 'vc-default-patch-addressee'

[Philip Kaludercic]

Dmitry Gutov <dmitry@gutov.dev> writes:

> Hi Eli,
>
> On 04/05/2024 14:44, Eli Zaretskii wrote:
>>> From: Philip Kaludercic<philipk@posteo.net>
>>> Date: Fri, 03 May 2024 07:36:34 +0000
>>>
>>> Philip Kaludercic<philipk@posteo.net>  writes:
>>>
>>>> Apparently we forgot to set this option back when it was added, but it
>>>> makes sense to have it along with 'vc-prepare-patches-separately'.
>>> ping?
>> Dmitry?
>> I'm okay with this, FWIW.
>
> I didn't think this was my area, but the change also LGTM.

Yes, I worked on vc-prepare-patch and related functionality for the most
part, hence the suggestion.

-- 
	Philip Kaludercic on peregrine