8000 Fix #5730: add gcc and make to the list of requirements by zlatanvasovic · Pull Request #5731 · jekyll/jekyll · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix #5730: add gcc and make to the list of requirements #5731

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 17, 2017
Merged

Fix #5730: add gcc and make to the list of requirements #5731

merged 1 commit into from
Jan 17, 2017

Conversation

zlatanvasovic
Copy link
Contributor

I am just unsure should we provide the link to the GNU make page, as it's dull and its installation instructions are nothing less dull. Also, make can be installed via the build-essentials package.

@@ -21,6 +21,7 @@ requirements you’ll need to make sure your system has before you start.
- [NodeJS](https://nodejs.org/), or another JavaScript runtime (Jekyll 2 and
earlier, for CoffeeScript support).
- [Python 2.7](https://www.python.org/downloads/) (for Jekyll 2 and earlier)
- [GCC](https://gcc.gnu.org/install/) and [Make](https://www.gnu.org/software/make/) (in case your system doesn't have them installed)
Copy link
Member

Choose a reason for hiding this comment

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

Should we include a way to check whether these are installed, e.g. gcc --version and make --version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@parkr Yes, that seems great. Also, gcc -v and make -v work as well, so we can use the shorter versions.

What syntax do you suggest for adding the information? A note, a code block or just inline code?

Copy link
Member

Choose a reason for hiding this comment

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

inline code is fine for me.

@parkr
Copy link
Member
parkr commented Jan 7, 2017

Fixes #5730.

@zlatanvasovic
Copy link
Contributor Author

Updated.

Copy link
Member
@DirtyF DirtyF left a comment

Choose a reason for hiding this comment

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

Thanks!

@parkr
Copy link
Member
parkr commented Jan 17, 2017

@jekyllbot: merge +site

@jekyllbot jekyllbot merged commit bf7e24c into jekyll:master Jan 17, 2017
jekyllbot added a commit that referenced this pull request Jan 17, 2017
@zlatanvasovic zlatanvasovic deleted the gcc-make branch January 17, 2017 07:29
@jekyll jekyll locked and limited conversation to collaborators Jul 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0