diff options
author | davidak <davidak@users.noreply.github.com> | 2020-04-26 14:12:55 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-26 12:12:55 +0000 |
commit | 579e56fb9eb1ac259baf4badef9652f609cfa507 (patch) | |
tree | ee5d59f638c3e7e3c20818e396af54442da266dd /doc/contributing/submitting-changes.xml | |
parent | 73f3469c765b24ec0ab5b2a2108d028806f32069 (diff) | |
download | nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.tar nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.tar.gz nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.tar.bz2 nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.tar.lz nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.tar.xz nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.tar.zst nixlib-579e56fb9eb1ac259baf4badef9652f609cfa507.zip |
Update commit policy for stable release branches (#86026)
only very few people followed the strict policy in the last 5 years. the maintainers accept backports without reason when it's obvious, so i updated the policy to reflect that
Diffstat (limited to 'doc/contributing/submitting-changes.xml')
-rw-r--r-- | doc/contributing/submitting-changes.xml | 48 |
1 files changed, 36 insertions, 12 deletions
diff --git a/doc/contributing/submitting-changes.xml b/doc/contributing/submitting-changes.xml index 73222c1a49ac..a88965f5cc60 100644 --- a/doc/contributing/submitting-changes.xml +++ b/doc/contributing/submitting-changes.xml @@ -407,23 +407,47 @@ Additional information. <section xml:id="submitting-changes-stable-release-branches"> <title>Stable release branches</title> - <itemizedlist> + <para> + For cherry-picking a commit to a stable release branch (<quote>backporting</quote>), use <literal>git cherry-pick -x <original commit></literal> so that the original commit id is included in the commit. + </para> + + <para> + Add a reason for the backport by using <literal>git cherry-pick -xe <original commit></literal> instead when it is not obvious from the original commit message. It is not needed when it’s a minor version update that includes security and bug fixes but don’t add new features or when the commit fixes an otherwise broken package. + </para> + + <para> + Here is an example of a cherry-picked commit message with good reason description: + </para> + +<screen> +zfs: Keep trying root import until it works + +Works around #11003. + +(cherry picked from commit 98b213a11041af39b39473906b595290e2a4e2f9) + +Reason: several people cannot boot with ZFS on NVMe +</screen> + + <para> + Other examples of reasons are: + </para> + + <itemizedlist spacing="compact"> <listitem> <para> - If you're cherry-picking a commit to a stable release branch (“backporting”), always use <command>git cherry-pick -xe</command> and ensure the message contains a clear description about why this needs to be included in the stable branch. + Previously the build would fail due to, e.g., <literal>getaddrinfo</literal> not being defined </para> + </listitem> + <listitem> <para> - An example of a cherry-picked commit would look like this: + The previous download links were all broken + </para> + </listitem> + <listitem> + <para> + Crash when starting on some X11 systems </para> -<screen> -nixos: Refactor the world. - -The original commit message describing the reason why the world was torn apart. - -(cherry picked from commit abcdef) -Reason: I just had a gut feeling that this would also be wanted by people from -the stone age. -</screen> </listitem> </itemizedlist> </section> |