diff mbox

[v2] manual: clarify policy when updating a package patch

Message ID 1449094406-29088-1-git-send-email-arnout@mind.be
State Accepted
Headers show

Commit Message

Arnout Vandecappelle Dec. 2, 2015, 10:13 p.m. UTC
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Cc: Renaud AUBIN <root@renaud.io>
---
v2: talk about 'From header' rather than 'author' (Baruch)
---
 docs/manual/patch-policy.txt | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Thomas Petazzoni Dec. 16, 2015, 10:44 p.m. UTC | #1
Dear Arnout Vandecappelle (Essensium/Mind),

On Wed, 2 Dec 2015 23:13:26 +0100, Arnout Vandecappelle
(Essensium/Mind) wrote:
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> Cc: Renaud AUBIN <root@renaud.io>
> ---
> v2: talk about 'From header' rather than 'author' (Baruch)
> ---
>  docs/manual/patch-policy.txt | 4 ++++
>  1 file changed, 4 insertions(+)

Applied, thanks.

Thomas
diff mbox

Patch

diff --git a/docs/manual/patch-policy.txt b/docs/manual/patch-policy.txt
index 6e27e71..0b4604e 100644
--- a/docs/manual/patch-policy.txt
+++ b/docs/manual/patch-policy.txt
@@ -106,6 +106,10 @@  upstream SCM software to generate the patch set.
 Otherwise, concatenate the header with the output of the
 +diff -purN package-version.orig/ package-version/+ command.
 
+If you update an existing patch (e.g. when bumping the package version),
+make sure the existing From header and Signed-off-by tags are not
+removed, but do update the rest of the patch comment when appropriate.
+
 At the end, the patch should look like:
 
 ---------------