External/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2025-02-01 17:35:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

scripts: dev_cli: Fix post build permissions for the whole tree

Browse Source 
Fixes: #752

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
This commit is contained in:
Samuel Ortiz 2020-02-13 16:40:10 +01:00 committed by Sebastien Boeuf
parent 287897d702
commit 296ada9441
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
scripts/dev_cli.sh
View File

@ -113,12 +113,12 @@ ensure_build_dir() {
done done
} }
# Fix build/ dir permissions after a container ran as root # Fix main directory permissions after a container ran as root.
# Since the container ran as root, any files it creates will be owned by root. # Since the container ran as root, any files it creates will be owned by root.
# This fixes that by recursively changing the ownership of build/ to the # This fixes that by recursively changing the ownership of /cloud-hypervisor to the
# current user. # current user.
# #
fix_build_dir_perms() { fix_dir_perms() {
# Yes, running Docker to get elevated privileges, just to chown some files # Yes, running Docker to get elevated privileges, just to chown some files
# is a dirty hack. # is a dirty hack.
$DOCKER_RUNTIME run \ $DOCKER_RUNTIME run \
@ -127,7 +127,7 @@ fix_build_dir_perms() {
--volume /dev:/dev \ --volume /dev:/dev \
--volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \ --volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \
"$CTR_IMAGE" \ "$CTR_IMAGE" \
chown -R "$(id -u):$(id -g)" "$CTR_CLH_CARGO_BUILT_DIR" chown -R "$(id -u):$(id -g)" "$CTR_CLH_ROOT_DIR"
return $1 return $1
} }
@ -237,7 +237,7 @@ cmd_tests() {
--volume /dev:/dev \ --volume /dev:/dev \
--volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \ --volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \
"$CTR_IMAGE" \ "$CTR_IMAGE" \
./scripts/run_unit_tests.sh "$@" || fix_build_dir_perms $? || exit $? ./scripts/run_unit_tests.sh "$@" || fix_dir_perms $? || exit $?
fi fi
if [ "$cargo" = true ] ; then if [ "$cargo" = true ] ; then
@ -247,7 +247,7 @@ cmd_tests() {
--rm \ --rm \
--volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \ --volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \
"$CTR_IMAGE" \ "$CTR_IMAGE" \
./scripts/run_cargo_tests.sh || fix_build_dir_perms $? || exit $? ./scripts/run_cargo_tests.sh || fix_dir_perms $? || exit $?
fi fi
if [ "$integration" = true ] ; then if [ "$integration" = true ] ; then
@ -261,10 +261,10 @@ cmd_tests() {
--volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \ --volume "$CLH_ROOT_DIR:$CTR_CLH_ROOT_DIR" \
--volume "$CLH_INTEGRATION_WORKLOADS:$CTR_CLH_INTEGRATION_WORKLOADS" \ --volume "$CLH_INTEGRATION_WORKLOADS:$CTR_CLH_INTEGRATION_WORKLOADS" \
"$CTR_IMAGE" \ "$CTR_IMAGE" \
./scripts/run_integration_tests.sh "$@" || fix_build_dir_perms $? || exit $? ./scripts/run_integration_tests.sh "$@" || fix_dir_perms $? || exit $?
fi fi
fix_build_dir_perms $? fix_dir_perms $?
} }
cmd_build-container() { cmd_build-container() {