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

Unified Diff: src/scripts/mod_image_for_test.sh

Issue 845007: Change mod_for_test_script to make it work with arm os image. (Closed)
Patch Set: patch 6. Created 10 years, 9 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/mod_for_test_scripts/test_setup.sh ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/mod_image_for_test.sh
diff --git a/src/scripts/mod_image_for_test.sh b/src/scripts/mod_image_for_test.sh
index ef783a98d034bab0e049ba06ee2cf362eb89c2ee..fb35572443736b9bcda4a0f7219c3d1d40e927bb 100755
--- a/src/scripts/mod_image_for_test.sh
+++ b/src/scripts/mod_image_for_test.sh
@@ -50,10 +50,6 @@ cleanup_rootfs_mounts() {
echo "Killing process that has open file on our rootfs: $cmdline"
! sudo kill $pid # Preceded by ! to disable ERR trap.
done
- if [[ -d "${ROOT_FS_DIR}/modify_scripts" ]]; then
- echo "Cleaned up modify_scripts mount"
- ! sudo umount "${ROOT_FS_DIR}/modify_scripts"
- fi
}
cleanup_rootfs_loop() {
@@ -100,15 +96,9 @@ else
fi
MOD_SCRIPTS_ROOT="${GCLIENT_ROOT}/src/scripts/mod_for_test_scripts"
-sudo mkdir -p "${ROOT_FS_DIR}/modify_scripts"
-
-sudo mount --bind "${MOD_SCRIPTS_ROOT}" "${ROOT_FS_DIR}/modify_scripts"
-
# Run test setup script inside chroot jail to modify the image
-sudo chroot "${ROOT_FS_DIR}" "/modify_scripts/test_setup.sh"
-
-sudo umount "${ROOT_FS_DIR}/modify_scripts"
-sudo rmdir "${ROOT_FS_DIR}/modify_scripts"
+sudo GCLIENT_ROOT=${GCLIENT_ROOT} ROOT_FS_DIR=${ROOT_FS_DIR} \
+ "${MOD_SCRIPTS_ROOT}/test_setup.sh"
cleanup
trap - EXIT
« no previous file with comments | « src/scripts/mod_for_test_scripts/test_setup.sh ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698