[PATCH] D37936: docs: Add instructions for how to submit a merge request

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 15 15:03:44 PDT 2017


hans added a comment.

Good stuff!



================
Comment at: docs/HowToReleaseLLVM.rst:266
+#. Use the ``utils/release/merge-request.sh`` script which will automatically file
+a bug at bugs.llvm.org requesting that the patch be merged. e.g.
+``$PROJ.src/utils/release/merge-request.sh -stable-version X.Y -r 12345 -user bugzilla at example.com``
----------------
Might as well make bugs.llvm.org a link I suppose?


================
Comment at: docs/HowToReleaseLLVM.rst:267
+a bug at bugs.llvm.org requesting that the patch be merged. e.g.
+``$PROJ.src/utils/release/merge-request.sh -stable-version X.Y -r 12345 -user bugzilla at example.com``
+
----------------
s/$PROJ/llvm/ ?

Maybe expand the example a little in case people are confused about X and Y, e.g.

To request a patch to be merged to the branch for the 5.0.1 stable release, use this...


================
Comment at: docs/HowToReleaseLLVM.rst:269
+
+#. Manually file a bug at bugs.llvm.org with the subject: "Merge r12345 into the X.Y branch"
+and enter the commit(s) that you want merged in the "Fixed by Commit(s)" field.
----------------
Link again, and maybe mention marking it as a blocker of the current release bug.. which I suppose one would have to search for, or know from the mailing list.


https://reviews.llvm.org/D37936





More information about the llvm-commits mailing list