Merge lp://staging/~linaro-infrastructure/cbuild/cbuild-init_cbuild-lava into lp://staging/~linaro-toolchain-dev/cbuild/cbuild-init
Proposed by
Stevan Radaković
| Status: | Merged |
|---|---|
| Approved by: | Matthew Gretton-Dann |
| Approved revision: | 37 |
| Merged at revision: | 23 |
| Proposed branch: | lp://staging/~linaro-infrastructure/cbuild/cbuild-init_cbuild-lava |
| Merge into: | lp://staging/~linaro-toolchain-dev/cbuild/cbuild-init |
| Diff against target: |
120 lines (+39/-2) 8 files modified
README (+9/-0) branches.sh (+3/-2) cbuild-setup.sh (+8/-0) crontab (+1/-0) init.sh (+1/-0) root-setup.sh (+8/-0) tcwg-web.sh (+7/-0) toolsrc (+2/-0) |
| To merge this branch: | bzr merge lp://staging/~linaro-infrastructure/cbuild/cbuild-init_cbuild-lava |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Matthew Gretton-Dann | Approve | ||
| Paul Sokolovsky | Approve | ||
|
Review via email:
|
|||
Description of the change
Contains code updates for easier CBuild deployment. Also has initial scripts and job files for LAVA integration.
To post a comment you must log in.
Revision history for this message
| Milo Casagrande (milo) wrote | # |
Revision history for this message
| Stevan Radaković (stevanr) wrote | # |
Revision history for this message
| Matthew Gretton-Dann (matthew-gretton-dann) wrote | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The only changes I would make, but not necessary, is moving the README part about the new wrapper scripts at the beginning. When I went through it the first time I was going to perform the steps listed there, but it is much easier with the new scripts: yeah, I didn't read all the file at first.
Other than that, it looks OK.