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

Unified Diff: sync_build_test.sh

Issue 3041033: Altered sync_build_test so that when --nousepkg is specified, it is also passed to setup_board. (Closed) Base URL: ssh://gitrw.chromium.org/crosutils.git
Patch Set: Created 10 years, 5 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: sync_build_test.sh
diff --git a/sync_build_test.sh b/sync_build_test.sh
index 4fb27151a51199b7d84f947a7c99ad6077e06f57..83e4cc055eed9739ed1674b55c987d8572cf27d4 100755
--- a/sync_build_test.sh
+++ b/sync_build_test.sh
@@ -515,20 +515,21 @@ function main() {
fi
if [[ ${FLAGS_build} -eq ${FLAGS_TRUE} ]]; then
+ local pkg_param=""
+ if [[ ${FLAGS_usepkg} -eq ${FLAGS_FALSE} ]]; then
+ pkg_param="--nousepkg"
+ fi
+
chdir_relative src/scripts
# Only setup board target if the directory does not exist
if [[ ! -d "${FLAGS_top}/chroot/build/${FLAGS_board}" ]]; then
run_phase_in_chroot "Setting up board target" \
- ./setup_board "${board_param}"
+ ./setup_board ${pkg_param} "${board_param}"
fi
local build_autotest_param=""
if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then
build_autotest_param="--withautotest"
fi
- local pkg_param=""
- if [[ ${FLAGS_usepkg} -eq ${FLAGS_FALSE} ]]; then
- pkg_param="--nousepkg"
- fi
run_phase_in_chroot "Building packages" \
./build_packages "${board_param}" \
« 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