Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(197)

Issue 3076012: Added output to tar command so it does not cause timeout to kick in. (Closed)

Created:
10 years, 4 months ago by zel
Modified:
9 years, 6 months ago
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Base URL:
ssh://gitrw.chromium.org/crosutils.git
Visibility:
Public.

Description

Added output to tar command so it does not cause timeout to kick in.

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M archive_build.sh View 2 chunks +2 lines, -2 lines 2 comments Download

Messages

Total messages: 5 (0 generated)
zel
10 years, 4 months ago (2010-07-28 17:12:13 UTC) #1
Christopher Covington
http://codereview.chromium.org/3076012/diff/1/2 File archive_build.sh (right): http://codereview.chromium.org/3076012/diff/1/2#newcode144 archive_build.sh:144: tar cjfv "${FLAGS_from}/autotest.tar.bz2" autotest Would --checkpoint be more suitable? ...
10 years, 4 months ago (2010-07-28 17:30:50 UTC) #2
zel
http://codereview.chromium.org/3076012/diff/1/2 File archive_build.sh (right): http://codereview.chromium.org/3076012/diff/1/2#newcode144 archive_build.sh:144: tar cjfv "${FLAGS_from}/autotest.tar.bz2" autotest On 2010/07/28 17:30:51, Christopher Covington ...
10 years, 4 months ago (2010-07-28 18:27:05 UTC) #3
Christopher Covington
On 2010/07/28 18:27:05, zel wrote: > http://codereview.chromium.org/3076012/diff/1/2 > File archive_build.sh (right): > > http://codereview.chromium.org/3076012/diff/1/2#newcode144 > ...
10 years, 4 months ago (2010-07-28 19:22:03 UTC) #4
sosa
10 years, 4 months ago (2010-07-29 00:02:01 UTC) #5
I agree that checkpoint might be a better choice.

Powered by Google App Engine
This is Rietveld 408576698