Skip to content

Commit

Permalink
Merge pull request #1646 from riscv/revert-1630-fix_docker_user
Browse files Browse the repository at this point in the history
Revert "Run docker as the current user"
  • Loading branch information
wmat committed Sep 23, 2024
2 parents 78a4dc9 + 0d75d98 commit 1745bbf
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -28,18 +28,12 @@ ifneq ($(SKIP_DOCKER),true)
$(shell ! docker -v 2>&1 | grep podman >/dev/null ; echo $$?)
ifeq "$(DOCKER_IS_PODMAN)" "1"
DOCKER_VOL_SUFFIX = :z
else
# Real Docker requires this flag so that the files it creates
# are owned by the current user instead of root. Podman does
# that by default so this flag isn't necessary.
DOCKER_USER_ARG := --user $(shell id -u)
endif

DOCKER_CMD = \
docker run --rm \
-v ${PWD}/$@.workdir:/build${DOCKER_VOL_SUFFIX} \
-w /build \
$(DOCKER_USER_ARG) \
${DOCKER_IMG} \
/bin/sh -c
DOCKER_QUOTE := "
Expand Down

0 comments on commit 1745bbf

Please sign in to comment.