[PATCH] D80395: HowToReleaseLLVM: Clean up document and remove references to SVN

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/docs/HowToReleaseLLVM.rst:97
 ``CMakeLists.txt`` versions from '``X.Ysvn``' to '``X.Y``'.
-Update it on mainline as well to be the next version ('``X.Y+1svn``').
+Update it on mainline as well to be the next version ('``X+1.Ysvn``').
 
----------------
I think this is of the form `X.0.0git` now?


================
Comment at: llvm/docs/HowToReleaseLLVM.rst:336
 
-#. Check out the ``www`` module from Subversion.
+#. Check out the ``www-releases`` module from GitHub.
 
----------------
Not really a question about this patch, but it looks like this step might be have been skipped for a few past releases?  https://releases.llvm.org/9.0.1/docs/index.html and https://releases.llvm.org/7.1.0/docs/index.html (which are linked from https://releases.llvm.org/) are 404.


================
Comment at: llvm/docs/HowToReleaseLLVM.rst:351
 
 #. Finally, update the main page (``index.html`` and sidebar) to point to the
    new release and release announcement.  Make sure this all gets committed back
----------------
Missing step "checkout llvm-www"?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80395/new/

https://reviews.llvm.org/D80395