emacs-bug-tracker
[Top][All Lists]
Advanced

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

bug#43946: closed ([PATCH] doc: Add item to "Submitting Patches" section


From: GNU bug Tracking System
Subject: bug#43946: closed ([PATCH] doc: Add item to "Submitting Patches" section.)
Date: Wed, 22 Sep 2021 13:04:02 +0000

Your message dated Wed, 22 Sep 2021 15:02:57 +0200
with message-id <87bl4khif2.fsf_-_@gnu.org>
and subject line Re: bug#43946: [PATCH] doc: Add item to "Submitting Patches" 
section.
has caused the debbugs.gnu.org bug report #43946,
regarding [PATCH] doc: Add item to "Submitting Patches" section.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
43946: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=43946
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] doc: Add item to "Submitting Patches" section. Date: Mon, 12 Oct 2020 10:20:03 +0200
* doc/contributing.texi (Submitting Patches): Add item about 'git-format-patch
--base'.
---
 doc/contributing.texi | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/contributing.texi b/doc/contributing.texi
index af3601442e..5ea3cb1899 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -932,6 +932,12 @@ Before submitting a patch that adds or modifies a package 
definition,
 please run through this check list:
 
 @enumerate
+@cindex @code{git format-patch}
+@cindex @code{git-format-patch}
+@item
+We recommend to use the command @code{git format-patch --base} to
+include the commit where your patch applies.
+
 @item
 If the authors of the packaged software provide a cryptographic
 signature for the release tarball, make an effort to verify the

base-commit: e746d30ec15852ec50e4247ea0a07ab2a9a69aa8
-- 
2.28.0




--- End Message ---
--- Begin Message --- Subject: Re: bug#43946: [PATCH] doc: Add item to "Submitting Patches" section. Date: Wed, 22 Sep 2021 15:02:57 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
Hi,

zimoun <zimon.toutoune@gmail.com> skribis:

> Seeing the number of patches in the patch tracker, most of them does not
> apply anymore.  It is not encouraging to review and decrease the queue
> when first the reviewer has to guess onto which commit the patch
> applies.

Yeah in hindsight I think I agree with you.  I went ahead and (finally!)
applied the patch.

Thanks for persevering, and apologies for my stubbornness!  :-)

Ludo’.


--- End Message ---

reply via email to

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