Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Gemfile breaks jekyll s #609

Closed
ghost opened this issue Nov 22, 2018 · 6 comments
Closed

Gemfile breaks jekyll s #609

ghost opened this issue Nov 22, 2018 · 6 comments
Assignees

Comments

@ghost
Copy link

ghost commented Nov 22, 2018

The current Gemfile includes GH pages, but for jekyll s to work, with jekyll-watch, you need

gem "jekyll", "~> 3.8.4"

@ghost ghost assigned Vincevrp Nov 22, 2018
@Vincevrp
Copy link
Contributor

Use bundle exec jekyll serve

@ghost
Copy link
Author

ghost commented Nov 22, 2018

Maybe add this to readme?

This still doesn't execute due to the GH pages dependency. Should that be installed on local machines as well?

@Vincevrp
Copy link
Contributor

Maybe add this to readme?

This still doesn't execute due to the GH pages dependency. Should that be installed on local machines as well?

This should work:

bundle install
bundle exec jekyll serve

If you can confirm this, I will add it to the README.

@ghost
Copy link
Author

ghost commented Nov 22, 2018

bundle install is an extremely long process. It took like 5 minutes. Isn't it possible to have a different Gemfile for gh pages and for people who clone the repo? Simply changing the gh pages gem to jekyll takes like one second and works.

We could just recommend people to run sed -i 's/gem "github-pages", group: :jekyll_plugins/gem "jekyll", "~> 3.8.4"/g' Gemfile, and if on Windows to make this change manually.

@Vincevrp
Copy link
Contributor

bundle install is an extremely long process. It took like 5 minutes.

This takes under 1 minute for me.

Isn't it possible to have a different Gemfile for gh pages and for people who clone the repo? Simply changing the gh pages gem to jekyll takes like one second and is quick.

As I wrote in #582, the reason we use it it is explained here.

@ghost
Copy link
Author

ghost commented Nov 25, 2018

I guess this could be related to how we initially installed Jekyll? Anyway, after running bundle install once, it's very fast the next time.

@ghost ghost closed this as completed Nov 25, 2018
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant