[PATCH] D22037: [RFC] Refactoring release docs

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 06:11:06 PDT 2016


hans added inline comments.

================
Comment at: RELEASE_TESTERS.TXT:2
@@ +1,3 @@
+This file is a list of the people responsible for ensuring that targets and
+environments get tested and validated during the upstream release process.
+They will also, in conjunction with the release manager and the code owners,
----------------
I don't think "upstream" makes sense when referring to ourselves.

================
Comment at: RELEASE_TESTERS.TXT:41
@@ +40,3 @@
+T: x86
+O: Darwin, Windows
+
----------------
Can you drop Darwin from my entry? I'll still build and test it if no one else does, but it would be great if we could find someone to sign up for it.

================
Comment at: docs/HowToReleaseLLVM.rst:91
@@ -94,3 +90,3 @@
    ``release_XY``, where ``X`` is the major and ``Y`` the minor release
    numbers.  The branches should be created using the following commands:
 
----------------
The branches are actually created by the utils/release/tag.sh script. I'd suggest removing the commands below.

================
Comment at: docs/HowToReleaseLLVM.rst:169
@@ +168,3 @@
+
+The minimum versions of the tools to compile ``llvm`` and ``clang`` are:
+
----------------
Isn't this already documented somewhere else? I worry that this will get out-dated, as the previous text was.


https://reviews.llvm.org/D22037





More information about the llvm-commits mailing list