[PATCH] D21880: Overhaul the release process docs

[Hans Wennborg via llvm-commits]

hans added a comment.

Sorry for the slow reply. I've been on vacation, and am still traveling this week.

In https://reviews.llvm.org/D21880#473499, @rengolin wrote:

> A few comments on this...
>
> 1. There is a lot of good information in HowToRelease that should not go to waste. I'm refactoring it now and should be *much* better.


Thanks for working on this. My feeling was that there was so much out-of-date info in these documents that it might be easier to nuke most of it and start over.

> 2. We actually need two documents. One of them is for release testers and managers, and the other is for the public. The former tells about the whole process, the people that do it, how to do it, what do we expect, etc. The latter tells about the schedules, time it takes, process to report a bug from outside and the kind of bug we mark as release blocker.


I think having two documents is confusing, at least in the current form.

ReleaseProcess.html is titled "How To Validate A Release". I think that's confusing, and that it should focus on describing the release process for LLVM developers and users.

HowToReleaseLLVM.html seems to have the purpose of describing what the release manager should do to create branches, etc. It seemed pretty out-dated so I deleted it. I suppose I'm the intended audience, but I never use it.

> 3. The test-suite is *still* not being done by the test-release script, nor by just running "make test-suite", so we can't remove the LIT part yet. For ARM and AArch64, I *always* run the test-suite, and I seriously recommend every release tester to do the same, so until the way we wan just use make/ninja, we need the info here.


OK, what do you think about removing the confusing HowToReleaseLLVM.html document, re-purposing ReleaseProcess.html to focus on describing the release process for LLVM devs and consumers, and have a sub-section in that about how to test a release?

I think the RELEASE_TESTERS.txt file is great, btw.


https://reviews.llvm.org/D21880