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

Unified Diff: tools/linux-tick-processor

Issue 118306: Add an ability to explicitly specify path to d8 shell to linux-tick-processor. (Closed)
Patch Set: Created 11 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/linux-tick-processor
diff --git a/tools/linux-tick-processor b/tools/linux-tick-processor
index e2f38b1b50530c32edc1c8d54cad1f903b213380..7ee99715b6fc385c1d3a80974a30e6f177b73631 100755
--- a/tools/linux-tick-processor
+++ b/tools/linux-tick-processor
@@ -1,16 +1,14 @@
#!/bin/sh
-tools_dir=$(dirname "$0")
-d8_exec=$tools_dir/../d8
+tools_path=$(dirname "$0")
+[ "$D8_PATH" ] || D8_PATH=$tools_path/..
+d8_exec=$D8_PATH/d8
# compile d8 if it doesn't exist.
-if [ ! -x $d8_exec ]
-then
- scons -C $tools_dir/.. d8
-fi
+[ -x $d8_exec ] || scons -C $D8_PATH d8
Kevin Millikin (Chromium) 2009/06/05 13:37:27 I think this second half needs to be: scons -C $D
Mikhail Naganov 2009/06/05 15:49:41 OK, done. Please let me know if it will not work.
# nm spits out 'no symbols found' messages to stderr.
-$d8_exec $tools_dir/splaytree.js $tools_dir/codemap.js \
- $tools_dir/csvparser.js $tools_dir/consarray.js \
- $tools_dir/profile.js $tools_dir/profile_view.js \
- $tools_dir/tickprocessor.js -- $@ 2>/dev/null
+$d8_exec $tools_path/splaytree.js $tools_path/codemap.js \
+ $tools_path/csvparser.js $tools_path/consarray.js \
+ $tools_path/profile.js $tools_path/profile_view.js \
+ $tools_path/tickprocessor.js -- $@ 2>/dev/null
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698