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

Unified Diff: third_party/instrumented_libraries/BUILD.gn

Issue 2775913002: Instrumented libraries: Remove dependence on GYP_DEFINES (Closed)
Patch Set: fix python syntax Created 3 years, 9 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
Index: third_party/instrumented_libraries/BUILD.gn
diff --git a/third_party/instrumented_libraries/BUILD.gn b/third_party/instrumented_libraries/BUILD.gn
index 9d7636a8eec3b6c5167776eec5050e32407db877..4d1e910ba4d34902cdd20c7280caefc7ddcecf95 100644
--- a/third_party/instrumented_libraries/BUILD.gn
+++ b/third_party/instrumented_libraries/BUILD.gn
@@ -57,18 +57,27 @@ if (prebuilt_instrumented_libraries_available) {
}
}
- # TODO(GYP): scripts/download_binaries.py uses GYP_DEFINES to decide whether
- # to download the archives extracted here.
+ action("download_prebuilt_instrumented_libraries") {
+ tarfile_name = "$archive_prefix-$instrumented_libraries_platform.tgz"
+ shafile = "binaries/$tarfile_name.sha1"
+ tarfile = "$target_out_dir/$tarfile_name"
Michael Achenbach 2017/03/28 07:55:05 Please note that now different out directories don
+ inputs = [ shafile ]
+ outputs = [ tarfile ]
+ script = "scripts/download_binaries.py"
+ args = [ rebase_path(shafile), rebase_path(tarfile) ]
+ }
+
# Note: This requires a clobber whenever Ubuntu version changes.
action("extract_prebuilt_instrumented_libraries") {
+ deps = [ ":download_prebuilt_instrumented_libraries" ]
visibility = [ ":prebuilt" ]
script = "scripts/unpack_binaries.py"
depfile = "$target_out_dir/$archive_prefix.d"
+ tarfile = get_target_outputs(":download_prebuilt_instrumented_libraries")
args = [
- archive_prefix,
- rebase_path("binaries"),
+ rebase_path(tarfile[0]),
+ rebase_path("$target_out_dir/$archive_prefix.txt"),
rebase_path(root_out_dir + "/instrumented_libraries_prebuilt"),
- rebase_path(target_out_dir, root_out_dir),
]
outputs = [
"$target_out_dir/$archive_prefix.txt",

Powered by Google App Engine
This is Rietveld 408576698