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

Unified Diff: build/install-build-deps.sh

Issue 660893002: [Telemetry] Implement automatic installation of optional openCV dependecy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 6 years, 2 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 | tools/telemetry/telemetry/core/platform/__init__.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/install-build-deps.sh
diff --git a/build/install-build-deps.sh b/build/install-build-deps.sh
index a0f11db7e13d31d0d1bff670ecf08432e9e724c3..930e1323bc07afacdf9cb37be011c2b4c55e1fc8 100755
--- a/build/install-build-deps.sh
+++ b/build/install-build-deps.sh
@@ -155,6 +155,9 @@ nacl_list="g++-mingw-w64-i686 lib32z1-dev
libxrandr2:i386 libxss1:i386 libxtst6:i386 texinfo xvfb
${naclports_list}"
+# Packages needed for telemetry only
+telemetry_list="python-opencv"
tonyg 2014/10/23 16:56:34 Please land this change first. I believe we'll hav
+
# Find the proper version of libgbm-dev. We can't just install libgbm-dev as
# it depends on mesa, and only one version of mesa can exists on the system.
# Hence we must match the same version or this entire script will fail.
@@ -293,8 +296,8 @@ else
fi
packages="$(
- echo "${dev_list} ${lib_list} ${dbg_list} ${arm_list} ${nacl_list}" |
- tr " " "\n" | sort -u | tr "\n" " "
+ echo "${dev_list} ${lib_list} ${dbg_list} ${arm_list} ${nacl_list} " \
+ "${telemetry_list}" | tr " " "\n" | sort -u | tr "\n" " "
)"
if [ 1 -eq "${do_quick_check-0}" ] ; then
« no previous file with comments | « no previous file | tools/telemetry/telemetry/core/platform/__init__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698