This is an archive of the discontinued LLVM Phabricator instance.

test-release.sh: Add ability to do a test build using the trunk or branches.
ClosedPublic

Authored by dsanders on Dec 8 2014, 1:58 AM.

Details

Summary

Adds --test-trunk and '--test-branch BRANCH' that causes the script to export
the trunk or the specified branch. Otherwise the tag specified by -release, -rc,
etc. will be used. The version portion of the package name will be 'test-trunk'
or 'test-$branch' (any forward slashes in the branch name are replaced with
underscores), for example:

clang+llvm-test-branches_release_35-mips-linux-gnu.tar.xz

This is primarily useful for bringing new release packages up to standard
without needing to create and maintain a tag for the purpose.

Diff Detail

Repository
rL LLVM

Event Timeline

dsanders updated this revision to Diff 17034.Dec 8 2014, 1:58 AM
dsanders retitled this revision from to test-release.sh: Add ability to do a test build using the trunk or branches..
dsanders updated this object.
dsanders edited the test plan for this revision. (Show Details)
dsanders added a reviewer: tstellarAMD.
dsanders added a subscriber: Unknown Object (MLST).
dsanders updated this revision to Diff 29761.Jul 15 2015, 3:20 AM
dsanders updated this object.
dsanders added a reviewer: hans.

Update to 3.7 with D10715 applied since there's no 3.7.0rc1 tag yet

hans edited edge metadata.Jul 15 2015, 9:15 AM

Thanks! This seems useful.

release/test-release.sh
97 ↗(On Diff #29761)

I wonder if we could make this simpler by adding just one command-line option, so one could run the script for example like:

./test-relese.sh -svn-path trunk
or
./test-release.sh -svn-path branches/release_37

And that would get baked into the package name. What do you think?

dsanders added inline comments.Jul 16 2015, 3:19 AM
release/test-release.sh
97 ↗(On Diff #29761)

I don't mind merging them into one option. The reason they're separate is merely that this was originally two patches.

'-test-branch trunk' should already work for testing the trunk so it's a fairly minor change.

dsanders updated this revision to Diff 29894.Jul 16 2015, 5:44 AM
dsanders edited edge metadata.

Merged -test-branch and -test-trunk into -svn-path

hans accepted this revision.Jul 16 2015, 9:21 AM
hans edited edge metadata.

Thanks! Looks good to me.

release/test-release.sh
207 ↗(On Diff #29894)

This error message should probably include the $ExportBranch now to be more useful.

This revision is now accepted and ready to land.Jul 16 2015, 9:21 AM
This revision was automatically updated to reflect the committed changes.