This is an archive of the discontinued LLVM Phabricator instance.

HowToReleaseLLVM: Clean up document and remove references to SVN
ClosedPublic

Authored by tstellar on May 21 2020, 11:57 AM.

Diff Detail

Event Timeline

tstellar created this revision.May 21 2020, 11:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 21 2020, 11:57 AM
efriedma added inline comments.
llvm/docs/HowToReleaseLLVM.rst
97

I think this is of the form X.0.0git now?

336

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.

351

Missing step "checkout llvm-www"?

hans edited reviewers, added: hans; removed: hansw.May 25 2020, 4:45 AM
hans added a subscriber: hans.

Thanks! This doc could certainly use some attention :-) I put some comments, feel free to ignore if you don't want to get into all of them now.

llvm/docs/HowToReleaseLLVM.rst
75

Actually, I always bump the trunk version number before creating the branch, and then create the branch from the commit right before the version number bump. So the dance is more like:

  1. Bump trunk version (and also clear release notes; not necessarily in the same commit)
  2. Create release branch from right before the bump
  3. Drop "git" prefix on the git branch.

Also the version bump gets tagged with "llvmorg-x-init" to generate nice "git describe" results.

78

Maybe note somewhere that the same should be done for the 'test-suite' repo, since the test and export scripts expect it to be versioned similarly.

295

maybe "use the git cherry-pick command with the -x option" to really spell it out so people don't forget the -x part.

316

"Merge both changes from mainline into the release branch." sounds wrong. The changes should be done on the release branch directly.

317

One thing that's easy to miss is that some docs must be generated. This is my go-to command for that:

bin/clang-tblgen -gen-diag-docs -I../clang/include \
      -I../clang/include/clang/Basic/ \
      ../clang/include/clang/Basic/Diagnostic.td -o \
      ../clang/docs/DiagnosticsReference.rst && \
bin/clang-tblgen -gen-attr-docs -I../clang/include \
        ../clang/include/clang/Basic/Attr.td -o \
        ../clang/docs/AttributeReference.rst && \
bin/clang-tblgen -gen-opt-docs -I../clang/include \
        -I../clang/include/clang/Driver -I../llvm/include \
        ../clang/include/clang/Driver/ClangOptionDocs.td -o \
        ../clang/docs/ClangCommandLineReference.rst
327

I've been doing "git tag -a", but I'm not sure if it makes any real difference?

340

These steps seem to refer to things that don't exist anymore, like BUILD_FOR_WEBSITE=1. Maybe they can be written a bit less specific: like check in the docs, update download.html to link to the binaries on github, etc. etc.

353

another Subversion mention here :-)

tstellar marked an inline comment as done.May 26 2020, 7:14 AM
tstellar added inline comments.
llvm/docs/HowToReleaseLLVM.rst
317

I fixed this in trunk, so this should not be needed any more (LLVM 11+).

hans added inline comments.May 26 2020, 9:46 AM
llvm/docs/HowToReleaseLLVM.rst
317

Oooh, that's excellent, thanks! (For anyone following along, that was D72875).

tstellar updated this revision to Diff 266612.May 27 2020, 11:41 AM
tstellar marked 14 inline comments as done.

Address review comments.

llvm/docs/HowToReleaseLLVM.rst
327

Do you also use -a for the llvmorg-N-init tag?

336

I will take a look at this.

hans added inline comments.May 28 2020, 12:59 AM
llvm/docs/HowToReleaseLLVM.rst
327

Yes.

tstellar updated this revision to Diff 279887.Jul 22 2020, 10:59 AM

Add example for tagging the -init tag.

tstellar marked an inline comment as done.Jul 22 2020, 11:00 AM
hans accepted this revision.Jul 23 2020, 1:22 AM

lgtm, thanks!

This revision is now accepted and ready to land.Jul 23 2020, 1:22 AM