[Buildroot] [PATCH] manual: clarify policy when updating a package patch

[Baruch Siach <baruch@tkos.co.il>]

Hi Arnout,

On Tue, Dec 01, 2015 at 09:41:55PM +0100, Arnout Vandecappelle (Essensium/Mind) wrote:
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> Cc: Renaud AUBIN <root@renaud.io>
> ---
>  docs/manual/patch-policy.txt | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/docs/manual/patch-policy.txt b/docs/manual/patch-policy.txt
> index 6e27e71..bb859bd 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 original author and existing Signed-off-by tags are not
> +removed.

Since not all patches we carry have an author (From:) header this text might 
be a little confusing. Maybe:

   ... make sure the original author (the 'From:' header, when the patch has 
   one) and existing Signed-off-by ...

It maybe also be worth a mention that not all existing patches have a sign-off 
tag.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -