Skip to content
This repository has been archived by the owner on Mar 25, 2018. It is now read-only.

add building-node-with-ninja.md #38

Closed

Conversation

Fishrock123
Copy link

People always ask me about how to do this, and there's no info about it really anywhere.

R=@nodejs/documentation

@Qard
Copy link
Member

Qard commented Sep 11, 2015

Cool, thanks!

Just a note: having the 2 sections covering the 4 steps feels a little weird to me. Reading it from the top down, as someone that hasn't read it before, I can see someone running the quoted steps in the first block and getting frustrated when it doesn't work how they thought it would. Then they'd get to the next block and realize that was the part they actually needed to be reading.

Perhaps it would be better to mix the two blocks together, putting the explanation bits from the first section in with the explicit instructional bits of the second.

@bengl
Copy link
Member

bengl commented Jan 14, 2016

Closing this since docs aren't intended to live here anymore. Maybe redo the PR against nodejs/nodejs.org?

@bengl bengl closed this Jan 14, 2016
@Fishrock123
Copy link
Author

note to self: do this

@chrisdickinson
Copy link
Contributor

@bengl Ah — I think we're pointing folks at PR'ing this material into nodejs/node itself now (ideally into doc/guides/). Although that's not built into the docs output right now, it shouldn't be too hard to add!

Fishrock123 added a commit to Fishrock123/node that referenced this pull request Feb 23, 2016
PR-URL: nodejs#4767
Refs: nodejs/docs#38
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
rvagg pushed a commit to nodejs/node that referenced this pull request Feb 27, 2016
PR-URL: #4767
Refs: nodejs/docs#38
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
rvagg pushed a commit to nodejs/node that referenced this pull request Feb 27, 2016
PR-URL: #4767
Refs: nodejs/docs#38
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Mar 10, 2016
PR-URL: #4767
Refs: nodejs/docs#38
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Mar 17, 2016
PR-URL: #4767
Refs: nodejs/docs#38
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Mar 21, 2016
PR-URL: #4767
Refs: nodejs/docs#38
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants