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

Issue 329303004: Make timestamp of fuzz harness archives finer grained. (Closed)

Created:
6 years, 6 months ago by Michael Starzinger
Modified:
6 years, 6 months ago
Reviewers:
Michael Achenbach
CC:
v8-dev
Visibility:
Public.

Description

Make timestamp of fuzz harness archives finer grained. R=machenbach@chromium.org

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M tools/fuzz-harness.sh View 1 chunk +1 line, -1 line 2 comments Download

Messages

Total messages: 6 (0 generated)
Michael Starzinger
The old timestamp was built with the assumption in mind that the fuzzer runs once ...
6 years, 6 months ago (2014-06-12 10:16:22 UTC) #1
Michael Achenbach
What about the possibility to let buildbot decide the file path and name?
6 years, 6 months ago (2014-06-12 12:25:58 UTC) #2
Michael Achenbach
lgtm - if it comes with a follow up...
6 years, 6 months ago (2014-06-12 12:43:38 UTC) #3
Michael Achenbach
https://codereview.chromium.org/329303004/diff/1/tools/fuzz-harness.sh File tools/fuzz-harness.sh (right): https://codereview.chromium.org/329303004/diff/1/tools/fuzz-harness.sh#newcode88 tools/fuzz-harness.sh:88: tar -cjf fuzz-results-$(date +%Y%m%d%H%M%S).tar.bz2 err-* w* small y turned ...
6 years, 6 months ago (2014-06-12 12:44:59 UTC) #4
Michael Starzinger
Thanks for the review. Landing. Will work on a follow-up to allow the builder to ...
6 years, 6 months ago (2014-06-12 18:52:48 UTC) #5
Michael Starzinger
6 years, 6 months ago (2014-06-12 19:05:12 UTC) #6
Message was sent while issue was closed.
Manually closed because my "git cl" crashed on me.

Powered by Google App Engine
This is Rietveld 408576698