This is an archive of the discontinued LLVM Phabricator instance.

[zorg] llgo: add logfiles to build_libgo
ClosedPublic

Authored by axw on Apr 26 2015, 9:56 AM.

Details

Summary

llgo's "libgo" target builds libgo by executing
a Make subprocess; the Make build sends output
to log files on disk, and not to stdio. On the
llgo-builder slave, the libgo build step takes
more than the default timeout (1200s) to complete,
which is killed due to lack of output on stdio.

This diff adds the log files to the build step,
so the step doesn't time out and to assist in
diagnosing build failures. We also explicitly
specify the libgo and llgoi targets as steps.

Diff Detail

Event Timeline

axw updated this revision to Diff 24444.Apr 26 2015, 9:56 AM
axw retitled this revision from to [zorg] llgo: add logfiles to build_libgo.
axw updated this object.
axw edited the test plan for this revision. (Show Details)
axw added reviewers: gkistanova, pcc.
axw added a subscriber: Unknown Object (MLST).
gkistanova accepted this revision.Apr 27 2015, 12:56 PM
gkistanova edited edge metadata.

LGTM

This revision is now accepted and ready to land.Apr 27 2015, 12:56 PM
axw closed this revision.Apr 27 2015, 4:22 PM