Skip to content

Commit 79b3aad

Browse files
committed
update readme
1 parent acea9d9 commit 79b3aad

File tree

1 file changed

+7
-18
lines changed

1 file changed

+7
-18
lines changed

packages/emnapi/README.md

Lines changed: 7 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ You will need to install:
4545
- npm `>= v8`
4646
- Emscripten `>= v3.1.9` / wasi-sdk / LLVM clang with wasm support
4747
- (Optional) CMake `>= v3.13`
48+
- (Optional) node-gyp `>= v10.2.0`
4849
- (Optional) ninja
4950
- (Optional) make
5051
- (Optional) [node-addon-api](https://github.com/nodejs/node-addon-api) `>= 6.1.0`
@@ -68,6 +69,9 @@ emcc -v
6869

6970
cmake --version
7071

72+
# if you use node-gyp
73+
node-gyp --version
74+
7175
# if you use ninja
7276
ninja --version
7377

@@ -629,15 +633,9 @@ Output code can run in recent version modern browsers and Node.js latest LTS. IE
629633
630634
### Using node-gyp (Experimental)
631635
632-
Currently node-gyp works on Linux only and don't support static library linking in cross-compiling.
633-
There are related PRs to try to make node-gyp work fine.
634-
635-
- https://github.com/nodejs/gyp-next/pull/222
636-
- https://github.com/nodejs/gyp-next/pull/240
637-
- https://github.com/nodejs/node-gyp/pull/2974
636+
Require node-gyp `>= 10.2.0`
638637
639-
If you experienced issues on Windows or macOS, please check the PRs for upstream changes detail and see
640-
[emnapi-node-gyp-test](https://github.com/toyobayashi/emnapi-node-gyp-test) for examples.
638+
See [emnapi-node-gyp-test](https://github.com/toyobayashi/emnapi-node-gyp-test) for examples.
641639
642640
- Variables
643641
@@ -685,19 +683,10 @@ declare var emnapi_manual_linking: 0 | 1
685683
["OS == 'emscripten'", {
686684
"product_extension": "js", # required
687685

688-
# Windows and Linux
689686
"cflags": [],
690687
"cflags_c": [],
691688
"cflags_cc": [],
692-
"ldflags": [],
693-
694-
# macOS uses following config
695-
'xcode_settings': {
696-
"WARNING_CFLAGS": [], # cflags
697-
"OTHER_CFLAGS": [], # cflags_c
698-
"OTHER_CPLUSPLUSFLAGS": [], # cflags_cc
699-
"OTHER_LDFLAGS": [] # ldflags
700-
}
689+
"ldflags": []
701690
}],
702691
["OS == 'wasi'", {
703692
# ...

0 commit comments

Comments
 (0)