DescriptionRevert of Fix javac command never caching input .md5s (patchset #8 id:140001 of https://codereview.chromium.org/1308083002/ )
Reason for revert:
Causing try-bot failures:
e.g.: http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromium_gn_compile_dbg/builds/111130/steps/compile%20%28without%20patch%29/logs/stdio
Original issue's description:
> Fix javac command never caching input .md5s
>
> This was broken over a year ago by http://crrev.com/a78eb479
>
> This change also removes --classes-dir (which is currently unused), and makes --jar-path (always used) required.
>
> It also forces the command to run if the output is missing (rather than relying completely on the .stamp file)
>
> On my ubiquity machine:
> Add a log statement to a file in base_java then build chrome_apk:
>
> Before:
> real 2m25.188s
> user 4m52.950s
> sys 1m10.064s
>
> After:
> real 1m16.989s
> user 1m13.795s
> sys 0m44.040s
>
> BUG=523420, 379190
>
> Committed: https://crrev.com/1c366880cb68fbf25050f1c6d78684eb6ae2e644
> Cr-Commit-Position: refs/heads/master@{#346975}
TBR=jbudorick@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=523420, 379190
Committed: https://crrev.com/a936d4429436bd69009eecdc8a8fa97763dcbe22
Cr-Commit-Position: refs/heads/master@{#347011}
Patch Set 1 #
Messages
Total messages: 5 (0 generated)
|