FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

build: make build-addons execute parallel by danbev · Pull Request #12230 · nodejs/node · GitHub

/ node Public

build: make build-addons execute parallel - #12230

Closed
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:make-build-addons-parallel
Closed

build: make build-addons execute parallel#12230
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:make-build-addons-parallel

Conversation

danbev commented Apr 5, 2017

Copy link
Copy Markdown
Contributor

Currently the building of addons and addons-napi are done sequentially
but could be done in parallel which this commit tries to address.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

build

Currently the building of addons and addons-napi are done sequentially
but could be done in parallel which this commit tries to address.
nodejs-github-bot added the build Issues and PRs related to build files or the CI. label Apr 5, 2017

danbev commented Apr 5, 2017

Copy link
Copy Markdown
Contributor Author

danbev mentioned this pull request Apr 5, 2017
2 tasks

Copy link
Copy Markdown
Contributor

I am not in favour. Mainly because the compilation logs are interleaved and difficult to understand. That said, I am 0 on this.

Copy link
Copy Markdown
Member

I have a somewhat more rigorous approach in #12231 that also addresses the issue that caused #11628.

danbev commented Apr 5, 2017

Copy link
Copy Markdown
Contributor Author

Closing in favour of #12231

danbev closed this Apr 5, 2017
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL