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

Side by Side Diff: sync_build_test.sh

Issue 3031056: Moved running unit tests to occur after building Chrome browser in sync_build_test.sh. (Closed) Base URL: ssh://gitrw.chromium.org/crosutils.git
Patch Set: Created 10 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/bin/bash 1 #!/bin/bash
2 2
3 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved. 3 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 # Script to sync your checkout, build a Chromium OS image, and test it all 7 # Script to sync your checkout, build a Chromium OS image, and test it all
8 # with one command. Can also check out a new Chromium OS checkout and 8 # with one command. Can also check out a new Chromium OS checkout and
9 # perform a subset of the above operations. 9 # perform a subset of the above operations.
10 # 10 #
(...skipping 515 matching lines...) Expand 10 before | Expand all | Expand 10 after
526 fi 526 fi
527 local build_autotest_param="" 527 local build_autotest_param=""
528 if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then 528 if [[ ${FLAGS_build_autotest} -eq ${FLAGS_TRUE} ]]; then
529 build_autotest_param="--withautotest" 529 build_autotest_param="--withautotest"
530 fi 530 fi
531 531
532 run_phase_in_chroot "Building packages" \ 532 run_phase_in_chroot "Building packages" \
533 ./build_packages "${board_param}" \ 533 ./build_packages "${board_param}" \
534 ${jobs_param} ${withdev_param} ${build_autotest_param} \ 534 ${jobs_param} ${withdev_param} ${build_autotest_param} \
535 ${pkg_param} 535 ${pkg_param}
536
537 if [[ "${FLAGS_board}" == "x86-generic" ]]; then
538 run_phase_in_chroot "Running unit tests" ./cros_run_unit_tests \
539 ${board_param}
540 fi
541 fi 536 fi
542 537
543 if [[ ${FLAGS_chrome_root} ]]; then 538 if [[ ${FLAGS_chrome_root} ]]; then
544 run_phase_in_chroot "Building Chromium browser" \ 539 run_phase_in_chroot "Building Chromium browser" \
545 BOARD="${FLAGS_board}" USE="build_tests" FEATURES="-usersandbox" \ 540 BOARD="${FLAGS_board}" USE="build_tests" FEATURES="-usersandbox" \
546 CHROME_ORIGIN=LOCAL_SOURCE emerge-${FLAGS_board} chromeos-chrome 541 CHROME_ORIGIN=LOCAL_SOURCE emerge-${FLAGS_board} chromeos-chrome
547 fi 542 fi
548 543
544 if [[ ${FLAGS_unittest} -eq ${FLAGS_TRUE} ]] && [[ "${FLAGS_board}" == \
545 "x86-generic" ]] ; then
546 run_phase_in_chroot "Running unit tests" ./cros_run_unit_tests \
547 ${board_param}
548 fi
549
549 if [[ ${FLAGS_master} -eq ${FLAGS_TRUE} ]]; then 550 if [[ ${FLAGS_master} -eq ${FLAGS_TRUE} ]]; then
550 chdir_relative src/scripts 551 chdir_relative src/scripts
551 if [[ -n "${FLAGS_chronos_passwd}" ]]; then 552 if [[ -n "${FLAGS_chronos_passwd}" ]]; then
552 run_phase_in_chroot "Setting default chronos password" \ 553 run_phase_in_chroot "Setting default chronos password" \
553 ./enter_chroot.sh "echo '${FLAGS_chronos_passwd}' | \ 554 ./enter_chroot.sh "echo '${FLAGS_chronos_passwd}' | \
554 ~/trunk/src/scripts/set_shared_user_password.sh" 555 ~/trunk/src/scripts/set_shared_user_password.sh"
555 fi 556 fi
556 run_phase_in_chroot "Mastering image" ./build_image \ 557 run_phase_in_chroot "Mastering image" ./build_image \
557 "${board_param}" --replace ${withdev_param} \ 558 "${board_param}" --replace ${withdev_param} \
558 ${jobs_param} 559 ${jobs_param}
(...skipping 27 matching lines...) Expand all
586 fi 587 fi
587 588
588 trap - EXIT 589 trap - EXIT
589 echo "Successfully used ${FLAGS_top} to:" 590 echo "Successfully used ${FLAGS_top} to:"
590 describe_steps 591 describe_steps
591 show_duration 592 show_duration
592 } 593 }
593 594
594 595
595 main $@ 596 main $@
OLDNEW
« 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