use xargs for recursive chown

vladak · vladak · commit 9e478f2ced09 · 2026-01-21T23:38:57.000+01:00
diff --git a/docker/entrypoint.sh b/docker/entrypoint.sh
@@ -26,8 +26,8 @@ function fix_ownership() {
   # Perform the check per subdirectory to avoid unnecessary churn.
   # Assumes the ownership of the directories matches its subdirectories/files.
   #
-  find "$dir" -maxdepth 1 -mindepth 1 \! -user $OWNER_USER -o \! -group $OWNER_GROUP \
-      -exec chown -R $OWNER_USER:$OWNER_USER {} \;
+  find "$dir" -maxdepth 1 -mindepth 1 \! -user $OWNER_USER -o \! -group $OWNER_GROUP | \
+      xargs chown -R $OWNER_USER:$OWNER_USER
 }
 
 command -v gosu >/dev/null 2>&1 || { echo "gosu missing"; exit 1; }

Original file line number	Diff line number	Diff line change
`@@ -26,8 +26,8 @@ function fix_ownership() {`
`26`	`26`	`# Perform the check per subdirectory to avoid unnecessary churn.`
`27`	`27`	`# Assumes the ownership of the directories matches its subdirectories/files.`
`28`	`28`	`#`
`29`		`- find "$dir" -maxdepth 1 -mindepth 1 \! -user $OWNER_USER -o \! -group $OWNER_GROUP \`
`30`		`- -exec chown -R $OWNER_USER:$OWNER_USER {} \;`
	`29`	`+ find "$dir" -maxdepth 1 -mindepth 1 \! -user $OWNER_USER -o \! -group $OWNER_GROUP \| \`
	`30`	`+ xargs chown -R $OWNER_USER:$OWNER_USER`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`command -v gosu >/dev/null 2>&1 \|\| { echo "gosu missing"; exit 1; }`