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

Unified Diff: DEPS

Issue 306543006: Bring Syzygy binaries in using a script rather than as a dependency. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Bugfix. Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | build/get_syzygy_binaries.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: DEPS
diff --git a/DEPS b/DEPS
index 8527282eec4249fd2da4bf122cdf0fa2189caba2..ac08fcbd4217dbb5fb6e76250af857c829879573 100644
--- a/DEPS
+++ b/DEPS
@@ -368,11 +368,6 @@ deps_os = {
"src/third_party/yasm/binaries":
"/trunk/deps/third_party/yasm/binaries@154708",
- # Binary level profile guided optimizations. This points to the
- # latest release binaries for the toolchain.
- "src/third_party/syzygy/binaries":
- (Var("googlecode_url") % "sawbuck") + "/trunk/syzygy/binaries@2154",
-
# Binaries for nacl sdk.
"src/third_party/nacl_sdk_binaries":
"/trunk/deps/third_party/nacl_sdk_binaries@111576",
@@ -780,6 +775,17 @@ hooks = [
"-s", "src/third_party/drmemory/drmemory-windows-sfx.exe.sha1",
],
},
+ # Pull the Syzygy binaries, used for optimization and instrumentation.
+ {
+ "name": "syzygy-binaries",
+ "pattern": ".",
+ "action": ["python",
+ "src/build/get_syzygy_binaries.py",
+ "--output-dir=src/third_party/syzygy/binaries",
+ "--revision=b08fb72610963d31cc3eae33f746a04e263bd860",
+ "--overwrite",
+ ],
+ },
{
# A change to a .gyp, .gypi, or to GYP itself should run the generator.
"name": "gyp",
« no previous file with comments | « no previous file | build/get_syzygy_binaries.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698