nw.js icon indicating copy to clipboard operation
nw.js copied to clipboard

Update Building NW.js.md

Open rebeccasf opened this issue 2 years ago • 2 comments

This is to ensure the documentation is update to support latest branch to avoid branch confusions among all the repos required in building nw.js.

Since chromium has started to shift focus to GN, GYP variables are now required explicit export in order to be able to generate ninja build files with GYP for Node.

Also, the gyp_chromium.py was written in Python2 syntax, hence any host that has Python3 set as default will have issue running the command.

rebeccasf avatar May 31 '22 06:05 rebeccasf

@rogerwang I would like to request your review on this doc update. Thank you.

rebeccasf avatar Jun 01 '22 01:06 rebeccasf

In fact it was mentioned in the past (I believe by @TheJaredWilcurt) that step 1 was outdated, namely that manual clone of node and v8 is anachronistic and that these should be handled by DEPS file in nw.js. I am sorry for not remembering the exact details. Am I striking something?

corwin-of-amber avatar Aug 12 '22 21:08 corwin-of-amber

The link in Build Faster for Linux should be changed to https://chromium.googlesource.com/chromium/src/+/HEAD/docs/linux/build_instructions.md#faster-builds.

div72 avatar Aug 17 '22 10:08 div72

I'm going to close this since this PR doesn't look like it's making any progress. I'm going to open a bunch of PRs to update the docs.

ayushmanchhabra avatar Aug 17 '22 17:08 ayushmanchhabra