Skip to content

Commit 28b3847

Browse files
tbalmblaschke
authored andcommitted
[BUGFIX] Fix composer run command in build script (#160)
* [BUGFIX] Fix composer run command in build script * [BUGFIX] Fix permissions for make build composer Runs the composer docker container with the current users uid/gid, otherwise composer writes all files as root user. Setting the composer cache dir (inside the container) to /tmp avoids multiple 'Cannot create cache directory'-warnings while the original composer cache dir is only writeable by root.
1 parent 09e12da commit 28b3847

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: bin/build.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ sectionHeader "Checking for composer.json ..."
2020
find "$CODE_DIR" -type f -name 'composer.json' | excludeFilter | while read FILE; do
2121
COMPOSER_JSON_DIR=$(dirname $($READLINK -f "$FILE"))
2222

23-
execInDir "$COMPOSER_JSON_DIR" "composer install --no-dev --no-interaction"
23+
execInDir "$COMPOSER_JSON_DIR" "docker run --rm --env COMPOSER_CACHE_DIR=/tmp --user $(id -u):$(id -g) -v \$(pwd):/app composer/composer:alpine install --no-dev --no-interaction"
2424
done
2525

2626

0 commit comments

Comments
 (0)