From: Eric Engestrom Date: Sun, 7 Jun 2020 17:14:51 +0000 (+0200) Subject: docs: move `Fixes:` tag explanation to its own section X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=a096d4109419c7c9015d8652549c5066d7fd1205;p=mesa.git docs: move `Fixes:` tag explanation to its own section This also adds the ability to link directly to it: https://mesa3d.org/submittingpatches.html#fixes Signed-off-by: Eric Engestrom Reviewed-by: Eric Anholt Part-of: --- diff --git a/docs/submittingpatches.rst b/docs/submittingpatches.rst index 984c0aa3186..5cd5025fd88 100644 --- a/docs/submittingpatches.rst +++ b/docs/submittingpatches.rst @@ -60,27 +60,9 @@ Patch formatting Prefer the full url to just ``Closes: #1``, since the url makes it easier to get to the bug page from ``git log`` - **Do not use the Fixes: tag for this!** Mesa already uses Fixes for - something else. - -- If a patch addresses a issue introduced with earlier commit, that - should be noted in the commit message. For example: - - :: - - Fixes: d7b3707c612 "util/disk_cache: use stat() to check if entry is a directory" - -- You can produce those fixes lines by running - - :: - - git config --global alias.fixes "show -s --pretty='format:Fixes: %h (\"%s\")'" - - once and then using - - :: - - git fixes + **Do not use the ``Fixes:`` tag for this!** Mesa already uses + ``Fixes:`` for something else. + See `below <#fixes>`__. - If there have been several revisions to a patch during the review process, they should be noted such as in this example: @@ -120,6 +102,22 @@ Patch formatting This provides reviewers with quick feedback if the patch has already been reviewed. +.. _fixes: + +The ``Fixes:`` tag +------------------ + +If a patch addresses a issue introduced with earlier commit, that +should be noted in the commit message. For example:: + + Fixes: d7b3707c612 "util/disk_cache: use stat() to check if entry is a directory" + +You can produce those fixes lines by running this command once:: + + git config --global alias.fixes "show -s --pretty='format:Fixes: %h (\"%s\")'" + +After that, using ``git fixes `` will print the full line for you. + .. _testing: Testing Patches