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

Unified Diff: src/scripts/sync_build_test.sh

Issue 660189: Changes to enable buildbot to build autotests and package them (Closed)
Patch Set: Created 10 years, 10 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 | « src/scripts/run_remote_tests.sh ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/sync_build_test.sh
diff --git a/src/scripts/sync_build_test.sh b/src/scripts/sync_build_test.sh
index 5dfe7cf10f6902ed735b41042635a01ab7a2eb53..b81dd629067d0e857ea016a90124bd0c20917971 100755
--- a/src/scripts/sync_build_test.sh
+++ b/src/scripts/sync_build_test.sh
@@ -224,6 +224,9 @@ function describe_steps() {
fi
echo " * Build image${withdev}${jobs}"
set_passwd=${FLAGS_TRUE}
+ if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then
+ echo " * Cross-build autotest client tests (build_autotest)"
+ fi
fi
if [[ ${FLAGS_master} -eq ${FLAGS_TRUE} ]]; then
echo " * Master image (build_image)"
@@ -246,9 +249,6 @@ function describe_steps() {
echo " * Set chronos password randomly"
fi
fi
- if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then
- echo " * Cross-build autotest client tests (build_autotest)"
- fi
if [[ -n "${FLAGS_image_to_usb}" ]]; then
echo " * Write the image to USB device ${FLAGS_image_to_usb}"
fi
@@ -492,9 +492,14 @@ function main() {
run_phase_in_chroot "Setting up board target" \
./setup_board "${board_param}"
fi
+ local build_autotest_param=""
+ if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then
+ build_autotest_param="--withautotest"
+ fi
+
run_phase_in_chroot "Building packages" \
./build_packages "${board_param}" \
- ${jobs_param} ${withdev_param}
+ ${jobs_param} ${withdev_param} ${build_autotest_param}
# TODO(kmixter): Enable this once build_tests works, but even
# then only do it when not cross compiling.
@@ -534,12 +539,6 @@ function main() {
./image_to_live.sh "--remote=${FLAGS_remote}" --update_known_hosts
fi
- if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then
- chdir_relative src/scripts
- run_phase_in_chroot "Building autotest" "./build_autotest.sh" \
- --noprompt "${board_param}"
- fi
-
if [[ -n "${FLAGS_test}" ]]; then
chdir_relative src/scripts
# We purposefully do not quote FLAGS_test below as we expect it may
« no previous file with comments | « src/scripts/run_remote_tests.sh ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698