diff options
author | Jeremiah Senkpiel <fishrock123@rocketmail.com> | 2019-09-17 12:45:23 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2019-09-18 08:18:38 -0700 |
commit | 4d798e1b1bbbbbf0a45fb478e153621fc37bb242 (patch) | |
tree | 9fc77ebed1b03a07e58707e38f85bd94ff828881 /node.gyp | |
parent | 9fb8b8d0f71e7576759e2b9ebf6f64403a336a13 (diff) | |
download | android-node-v8-4d798e1b1bbbbbf0a45fb478e153621fc37bb242.tar.gz android-node-v8-4d798e1b1bbbbbf0a45fb478e153621fc37bb242.tar.bz2 android-node-v8-4d798e1b1bbbbbf0a45fb478e153621fc37bb242.zip |
bootstrap: provide usable error on missing internal module
Due to how bootstrap/loaders.js itself is loaded and invoked,
stacktraces from it are munged and no longer point back to the error
source.
That resulted in the following unhelpful error if an internal module
was missing or misnamed:
```
internal/bootstrap/loaders.js:190
return mod.compile();
^
TypeError: Cannot read property 'compile' of undefined
```
This changes that to at least print the id that was attempted to be
loaded:
```
internal/bootstrap/loaders.js:189
if (!mod) throw new TypeError(`Missing internal module '${id}'`);
^
TypeError: Missing internal module 'internal/a'
```
PR-URL: https://github.com/nodejs/node/pull/29593
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'node.gyp')
0 files changed, 0 insertions, 0 deletions