Fix #224. Make sure we chown our var dir too.

envygeeks · Jul 7, 2018 · e96998e · e96998e
1 parent 70daad4
commit e96998e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/opts.yml b/opts.yml
@@ -11,4 +11,4 @@ tags:
   pages: pages
 releases:
   tag:
-    pages: 3.7.3 # 06/05/2018
+    pages: 3.7.3 # 07/07/2018
diff --git a/repos/jekyll/copy/all/usr/jekyll/bin/entrypoint b/repos/jekyll/copy/all/usr/jekyll/bin/entrypoint
@@ -20,14 +20,12 @@ export JEKYLL_GID
 if [ "$JEKYLL_UID" != "0" ] && [ "$JEKYLL_UID" != "$(id -u jekyll)" ]; then
   usermod  -u $JEKYLL_UID jekyll
   groupmod -g $JEKYLL_GID jekyll
+  chown_args=""
 
-  if [ "$FULL_CHOWN" ]; then
-    chown -R jekyll:jekyll \
-      $JEKYLL_DATA_DIR
-  else
-    chown jekyll:jekyll \
-      $JEKYLL_DATA_DIR
-  fi
+  [ "$FULL_CHOWN" ] && chown_args="-r"
+  for d in "$JEKYLL_DATA_DIR" "$JEKYLL_VAR_DIR"; do
+    chown $chown_args "$d"
+  done
 fi
 
 # --