hughpearse commented on code in PR #4162:
URL: https://github.com/apache/calcite/pull/4162#discussion_r1929596216


##########
site/docker-compose.yml:
##########
@@ -14,20 +14,20 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-version: '3'
 services:
   dev:
-    image: jekyll/jekyll:4
-    command: jekyll serve --watch --force_polling
+    image: ruby:3.3.7-slim-bullseye
+    working_dir: /root
+    command: sh -c "apt-get update && apt-get install -y build-essential 
libssl-dev zlib1g-dev ruby-dev bundler && bundle install && bundle exec jekyll 
serve --host=0.0.0.0 --watch --force_polling"
     ports:
       - 4000:4000
     volumes:
-      - .:/srv/jekyll
+      - .:/root
   build-site:
-    image: jekyll/jekyll:4
-    command: jekyll build
+    image: ruby:3.3.7-slim-bullseye
+    command: sh -c "groupadd -g 1000 jekyll && useradd -u 1000 -g 1000 
--create-home jekyll && rm -rf /home/jekyll/.bundle /home/jekyll/Gemfile.lock 
/home/jekyll/.jekyll-cache && apt-get update && apt-get install -y 
build-essential libssl-dev zlib1g-dev ruby-dev bundler && su jekyll -c 'cd ~ && 
bundle install && bundle exec jekyll build'"

Review Comment:
   can now be run with
   
   ```bash
   foor@host$ env JEKYLL_UID=$(id -u) JEKYLL_GID=$(id -g) docker-compose run 
build-site
   ``
   
   and defaults to uid gid 1000 when not set



##########
site/docker-compose.yml:
##########
@@ -14,20 +14,20 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-version: '3'
 services:
   dev:
-    image: jekyll/jekyll:4
-    command: jekyll serve --watch --force_polling
+    image: ruby:3.3.7-slim-bullseye
+    working_dir: /root
+    command: sh -c "apt-get update && apt-get install -y build-essential 
libssl-dev zlib1g-dev ruby-dev bundler && bundle install && bundle exec jekyll 
serve --host=0.0.0.0 --watch --force_polling"
     ports:
       - 4000:4000
     volumes:
-      - .:/srv/jekyll
+      - .:/root
   build-site:
-    image: jekyll/jekyll:4
-    command: jekyll build
+    image: ruby:3.3.7-slim-bullseye
+    command: sh -c "groupadd -g 1000 jekyll && useradd -u 1000 -g 1000 
--create-home jekyll && rm -rf /home/jekyll/.bundle /home/jekyll/Gemfile.lock 
/home/jekyll/.jekyll-cache && apt-get update && apt-get install -y 
build-essential libssl-dev zlib1g-dev ruby-dev bundler && su jekyll -c 'cd ~ && 
bundle install && bundle exec jekyll build'"

Review Comment:
   can now be run with
   
   ```bash
   foor@host$ env JEKYLL_UID=$(id -u) JEKYLL_GID=$(id -g) docker-compose run 
build-site
   ```
   
   and defaults to uid gid 1000 when not set



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to