Skip to content

Conversation

@tartarini
Copy link
Contributor

@tartarini tartarini commented Jan 22, 2021

added a section to use the Jekyl docker container to build the site.
I have experienced a number of issued with my installation of macports OSX, so in case..

@tartarini tartarini requested a review from drj11 January 22, 2021 23:15
Copy link
Contributor

@willfurnass willfurnass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but suggest removing your bio given it's to be added via another PR.

```bash
cd <your_cloned_website_dir>
export JEKYLL_VERSION=3.8
docker run --rm --volume="$PWD:/srv/jekyll" -it jekyll/jekyll:$JEKYLL_VERSION jekyll build
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't get this working:

There was an error while trying to write to `/srv/jekyll/Gemfile.lock`. It is likely that you need to grant write permissions for that path.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I re-run the procedure on OSX 10.15.7 and redhat 7.x and I do not get that error.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, maybe it's just my machine

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's because the guest process in the docker container is trying to write to /srv/jekyll/Gemfile.lock, which is your pwd on the host. Docker doesn't necessarily have permission to do that, and it may depend how you've configured Docker. In any case, i'm not surprised that it varies according to host OS.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, let's revisit if needs be.

@@ -0,0 +1,36 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already got this content on the master branch so suggest removing the file from this PR.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tartarini If you remove this file from this PR we can get it merged.

@willfurnass
Copy link
Contributor

@frb-yousefi Did you follow this guide? If so, did these instructions work for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants