Yulong Wang

Cannot require 'path' in napa.js @0.1.1

2

Change e0696c528807cac194db5d679cec7f0ed2bb7655 introduced this bug. When zone.execute('path', ...) , the first arg will be rewrited to join with the current folder, thus cause it not recognized as built-in module.

Build warning in latest node version (MSVC2017 + Node8.3.0)

3

...\.cmake-js\node-x64\v8.3.0\include\node\v8.h(4150): warning C4251: 'v8::WasmModuleObjectBuilder::received_buffers_': class 'std::vector' needs to have dll-interface to be used by clients of class 'v8::WasmModuleObjectBuilder'

### Problem: Build warning on Windows: > cl : Command line warning D9002: ignoring unknown option '-fno-rtti' > cl : Command line warning D9002: ignoring unknown option '-fPIC' ### Cause:...

It does not work because `contextDir` in `ModuleLoader::ModuleLoaderImpl::RequireModule` is set to `node_modules\napajs\lib\zone` instead of the working directory.

Napa.js doesn't have fully support on Windows UNC prefix `\\?\` in filename. - `path.join()` behaviors incorrectly in Napa.js when using UNC prefix - `path.resolve()` behaviors incorrectly in Napa.js when using...

Inconsistency for napa/node zone when accessing '__dirname' in global module

1

```js let napaZone = napa.zone.create('test-dirname-zone', { workers: 1 }); let nodeZone = napa.zone.node; napaZone.broadcast('var getDirname = function () { return __dirname; };'); nodeZone.broadcast('var getDirname = function () { return __dirname;...

The following examples can build on Windows and Linux, but fail on macOS * https://github.com/Microsoft/napajs/tree/master/examples/modules/async-number * https://github.com/Microsoft/napajs/tree/master/examples/modules/hello-world Error message: > > SOLINK(target) Release/../../bin/addon.napa > Undefined symbols for architecture x86_64: >...

- OSX (xcode8.3) unittest failure https://travis-ci.org/Microsoft/napajs/jobs/262890394

header file napa/module.h uses #pragma to disable warning C4100 when including v8.h. napa.h contains napa/module.h, however it fails to disable that warning because v8.h is included before napa/module.h Current workaround:...

**Description**: update branch name from 'master' to 'main' for script `npm run pull:wasm` (ONNX Runtime Web).