Skip to content

build: expose napi_build_version - #27800

Closed
NickNaso wants to merge 5 commits into
nodejs:masterfrom
NickNaso:napi-build-version
Closed

NickNaso wants to merge 5 commits into
nodejs:masterfrom
NickNaso:napi-build-version

Conversation

@NickNaso

@NickNaso NickNaso commented May 21, 2019 •

Copy link
Copy Markdown
Member

build: expose napi_build_version to allow node-gyp to make it available for add-ons.
Fixes: nodejs/node-gyp#1745
Refs: nodejs/abi-stable-node#371

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows [commit guidelines]

@nodejs-github-bot nodejs-github-bot added the build Issues and PRs related to Node.js builds or CI infrastructure. label May 21, 2019
@NickNaso NickNaso changed the title Fix issue 1745 on node-gyp n-api: expose napi_build_version May 21, 2019
@NickNaso NickNaso changed the title n-api: expose napi_build_version build: expose napi_build_version May 21, 2019
@richardlau richardlau added the node-api Issues and PRs related to Node-API. label May 23, 2019
@richardlau

Copy link
Copy Markdown
Member

cc @nodejs/n-api

@NickNaso

Copy link
Copy Markdown
Member Author

@richardlau I did a mistake with git and I'm closing this PR and reopening new one.

@NickNaso NickNaso closed this May 23, 2019
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 Node.js builds or CI infrastructure. node-api Issues and PRs related to Node-API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expose napi version as a node-gyp variable

3 participants