[coreboot-gerrit] Patch set updated for coreboot: Documentation: Add information about patches from other git repos

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Wed Dec 30 20:36:59 CET 2015


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12804

-gerrit

commit df02ac3620fa3431a4c25d103ba645dd30105d51
Author: Martin Roth <martinroth at google.com>
Date:   Sat Dec 26 13:33:09 2015 -0700

    Documentation: Add information about patches from other git repos
    
    This is more tribal knowledge that I don't think I've seen written down
    anywhere else.  It's not a huge issue, but when looking through the git
    log, it helps to be able to differentiate the information from the old
    gerrit with the information from the new one.
    
    Change-Id: I7993bda1e9aab79dc26940aaba9ddc52382ed0df
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 Documentation/gerrit_guidelines.md | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/Documentation/gerrit_guidelines.md b/Documentation/gerrit_guidelines.md
index 3140ff5..1833b0a 100644
--- a/Documentation/gerrit_guidelines.md
+++ b/Documentation/gerrit_guidelines.md
@@ -236,6 +236,20 @@ request. For major problems where the change doesn’t work as intended or
 breaks other platforms, the change really needs to go into the original
 patch.
 
+* When bringing in a patch from another git repo, update the original
+git/gerrit tags by prepending the lines with 'Original-'.  Marking
+the original text this way makes it much easier to tell what changes
+happened in which repository. This applies to these lines, not the actual
+commit message itself:
+        Commit-Id:
+        Change-Id:
+        Signed-off-by:
+        Reviewed-on:
+        Tested-by:
+        Reviewed-by:
+The script 'util/gitconfig/rebase.sh' can be used to help automate this.
+Other tags such as 'Commit-Queue' can simply be removed.
+
 
 Expectations contributors should have:
 --------------------------------------



More information about the coreboot-gerrit mailing list