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

Issue 865463004: Teach mojo_cache_linker how to use a cache (Closed)

Created:
5 years, 11 months ago by eseidel
Modified:
5 years, 11 months ago
Reviewers:
abarth-chromium
CC:
abarth-chromium, Aaron Boodman, ben+mojo_chromium.org, darin (slow to review), esprehn, mojo-reviews_chromium.org, ojan, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, yzshen+watch_chromium.org
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Teach mojo_cache_linker how to use a cache Many of the .so files in out/Debug are hundreds of megabytes in size. md5sum can take seconds to churn through them all, so I've added a small caching layer which makes this script take no time in the common case. R=abarth@chromium.org BUG= Committed: https://chromium.googlesource.com/external/mojo/+/871366140a41a7422ff9e1173534358accc438f8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+81 lines, -23 lines) Patch
M sky/tools/mojo_cache_linker.py View 2 chunks +81 lines, -23 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
eseidel
5 years, 11 months ago (2015-01-24 00:34:40 UTC) #1
abarth-chromium
lgtm
5 years, 11 months ago (2015-01-24 00:41:35 UTC) #2
eseidel
5 years, 11 months ago (2015-01-24 00:42:22 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
871366140a41a7422ff9e1173534358accc438f8 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698