rdhabalia commented on a change in pull request #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658#discussion_r131995895
########## File path: site/scripts/docker-build.sh ########## @@ -35,6 +35,9 @@ echo "---- Build Pulsar website using image $IMAGE" docker pull $IMAGE -DOCKER_CMD="docker run --user $USER -i -v $ROOT_DIR:/pulsar $IMAGE" +CI_USER=$(id -u) +CI_GROUP=$(id -g) -$DOCKER_CMD bash -l -c 'cd /pulsar/site && rvm use . && make setup && make protobuf_doc_gen && make build' +DOCKER_CMD="docker run -i -e CI_USER=$CI_USER -e CI_GROUP=$CI_GROUP -v $ROOT_DIR:/pulsar $IMAGE" + +$DOCKER_CMD bash -l -c 'cd /pulsar/site && rvm use . && make setup && make protobuf_doc_gen && make build && chown -R $CI_USER:$CI_GROUP /pulsar/generated-site' Review comment: did you forget `'` in the end of the line? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
