diff options
author | Ulf Adams <ulfjack@google.com> | 2017-01-13 11:08:04 +0000 |
---|---|---|
committer | Marcel Hlopko <hlopko@google.com> | 2017-01-13 13:53:54 +0000 |
commit | bb1799333157345087249f82448db1982f92edb4 (patch) | |
tree | a629153de0191bfeb03731b74091e617ce9e6673 /src/main/cpp/blaze.cc | |
parent | 55f69dc0baa0a4b8ee9a0276954885199aa4b742 (diff) |
Also add the duration as the test process time.
Technically, this code is incorrect, because we're calling into spawn strategy,
which might do any amount of additional waiting (e.g., to acquire local
resources). We should change spawn strategy to return structured information
about the subprocess execution, including the wall time, and use that to set
the test information. But for now, this seems good enough.
Fixes #1748.
Effectively a rollback of the rollback, but with a manual fix.
The problem was that the code was assuming exactly one process time,
which is incorrect in at least the case where the proto comes from a previous
Bazel version, because that didn't set the process time at all. We're now
copying over all the values, which should work in all cases.
--
PiperOrigin-RevId: 144425125
MOS_MIGRATED_REVID=144425125
Diffstat (limited to 'src/main/cpp/blaze.cc')
0 files changed, 0 insertions, 0 deletions