doc: add author on cc to git fixline alias
authorHarry van Haaren <harry.van.haaren@intel.com>
Wed, 12 Jul 2017 13:38:52 +0000 (14:38 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Thu, 3 Aug 2017 13:27:26 +0000 (15:27 +0200)
commitf9ef083c6cfe1cf435360a2b9dfed019853b6ae0
treeeae275a22ee82177a5d21401be83a164526a5b70
parent7605d9b8f77b0d4ef456037665d29e75d84187f0
doc: add author on cc to git fixline alias

With this commit, the correct method to use git fixline to indicate
a fix of a previous commit has changed. The new rules require the
author of the original code that is being fixed to be on CC.

The logic behind this improvement is that if there is a genuine bug,
one of the ideal people to review is the author of the original code
being fixed. Adding them on Cc makes them aware of the patch, avoiding
it from being passed by accidentally while reading the mailing-list.

Given that the original author (now on Cc:) might not actually review,
there is no value in keeping the Cc: in git commit history. If the
original author performs a review, their Reviewed-by: or Acked-by: is
stored in git history (same as now).

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
doc/guides/contributing/patches.rst