From b0708b58f9d6490ed4e47ec688067bebdeb121aa Mon Sep 17 00:00:00 2001 From: Sijie Guo Date: Wed, 28 Nov 2018 18:10:38 -0800 Subject: [PATCH] [RELEASE] fix docker mount volume issue Descriptions of the changes in this PR: * Motivation #1674 introduced a regression for running release scripts on mac. the destination should be "/home/${USER}" rather than ${LOCAL_HOME}. because the docker os is linux. Otherwise the gpg keys can not propagated correctly from host machine to docker. * Changes Use `/home/${USER}` as the destination path Reviewers: Jia Zhai This closes #1845 from sijie/fix_release_scripts --- dev/release/000-run-docker.sh | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/dev/release/000-run-docker.sh b/dev/release/000-run-docker.sh index d63e2221b3b..259b46e9902 100755 --- a/dev/release/000-run-docker.sh +++ b/dev/release/000-run-docker.sh @@ -41,12 +41,10 @@ if [ "$(uname -s)" == "Linux" ]; then USER_NAME=${SUDO_USER:=$USER} USER_ID=$(id -u "${USER_NAME}") GROUP_ID=$(id -g "${USER_NAME}") - LOCAL_HOME="/home/${USER_NAME}" else # boot2docker uid and gid USER_NAME=$USER USER_ID=1000 GROUP_ID=50 - LOCAL_HOME="/Users/${USER_NAME}" fi docker build -t "${IMAGE_NAME}-${USER_NAME}" - <