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

Unified Diff: archive_build.sh

Issue 6527001: Remove old logic ... and don't needlessly modifiy dev image. (Closed) Base URL: http://git.chromium.org/git/crosutils.git@master
Patch Set: Remove debug info Created 9 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 | « no previous file | common.sh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: archive_build.sh
diff --git a/archive_build.sh b/archive_build.sh
index 94630059fd0017fee87bd6c25aa55f44a4fbe5c4..833c6cb3326cfef37d05ca1f22bae727da967b02 100755
--- a/archive_build.sh
+++ b/archive_build.sh
@@ -66,6 +66,7 @@ DEFINE_boolean official_build $FLAGS_FALSE \
DEFINE_string test_tarball "" "Optional path to test tarball to archive"
DEFINE_boolean test_mod $FLAGS_TRUE "Modify image for testing purposes"
DEFINE_boolean prebuilt_upload $FLAGS_FALSE "Upload prebuilt binary packages."
+DEFINE_boolean remove_dev $FLAGS_TRUE "Remove the de image during archive."
DEFINE_string to "$DEFAULT_TO" "Directory of build archive"
DEFINE_string zipname "image.zip" "Name of zip file to create."
@@ -138,27 +139,11 @@ echo "archive to file: $ZIPFILE"
rm -rf "$OUTDIR"
mkdir -p "$OUTDIR"
-
-SRC_IMAGE="${FLAGS_from}/chromiumos_image.bin"
-BACKUP_IMAGE="${FLAGS_from}/chromiumos_image_bkup.bin"
-
-# Apply mod_image_for_test to the developer image, and store the
-# result in a new location. Usage:
-# do_chroot_mod "$OUTPUT_IMAGE" "--flags_to_mod_image_for_test"
-function do_chroot_mod() {
- MOD_ARGS=$2
- OUTPUT_IMAGE=$1
- cp -f "${SRC_IMAGE}" "${BACKUP_IMAGE}"
- ./enter_chroot.sh -- ./mod_image_for_test.sh --board $FLAGS_board \
- --yes ${MOD_ARGS}
- mv "${SRC_IMAGE}" "${OUTPUT_IMAGE}"
- mv "${BACKUP_IMAGE}" "${SRC_IMAGE}"
-}
-
# Modify image for test if flag set.
if [ $FLAGS_test_mod -eq $FLAGS_TRUE ]; then
echo "Modifying image for test"
- do_chroot_mod "${FLAGS_from}/chromiumos_test_image.bin" ""
+ ./enter_chroot.sh -- ./mod_image_for_test.sh --board $FLAGS_board \
+ --noinplace --yes
pushd "${FLAGS_chroot}/build/${FLAGS_board}/usr/local"
echo "Archiving autotest build artifacts"
@@ -168,8 +153,8 @@ fi
if [ $FLAGS_factory_test_mod -eq $FLAGS_TRUE ]; then
echo "Modifying image for factory test"
- do_chroot_mod "${FLAGS_from}/chromiumos_factory_image.bin" \
- "--factory"
+ ./enter_chroot.sh -- ./mod_image_for_test.sh --board $FLAGS_board \
+ --yes --noinplace --factory
fi
# Modify for recovery
@@ -181,8 +166,8 @@ if [ $FLAGS_official_build -eq $FLAGS_TRUE ]; then
fi
# Remove the developer build if test image is also built.
-if [ $FLAGS_test_mod -eq $FLAGS_TRUE ] ; then
- rm -f ${SRC_IMAGE}
+if [ $FLAGS_remove_dev -eq $FLAGS_TRUE ]; then
+ rm -f "${FLAGS_from}/${CHROMEOS_IMAGE_NAME}"
sjg 2011/02/16 02:04:51 Yes this seems much better. I guess you are wanti
fi
# Build differently sized shims. Currently only factory install shim is
« no previous file with comments | « no previous file | common.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698