Skip to content

Commit c634698

Browse files
DrakiaXYZMylesBorins
authored andcommitted
lib: switch to internalBinding for cjs loader
Switch the cjs loader to use internalBinding instead of process.binding for reading command line options. PR-URL: #23492 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
1 parent 16175c1 commit c634698

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

lib/internal/modules/cjs/loader.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,10 @@ const {
4040
stripBOM,
4141
stripShebang
4242
} = require('internal/modules/cjs/helpers');
43-
const preserveSymlinks = !!process.binding('config').preserveSymlinks;
44-
const preserveSymlinksMain = !!process.binding('config').preserveSymlinksMain;
45-
const experimentalModules = !!process.binding('config').experimentalModules;
43+
const options = internalBinding('options');
44+
const preserveSymlinks = options.getOptions('--preserve-symlinks');
45+
const preserveSymlinksMain = options.getOptions('--preserve-symlinks-main');
46+
const experimentalModules = options.getOptions('--experimental-modules');
4647

4748
const {
4849
ERR_INVALID_ARG_TYPE,

0 commit comments

Comments
 (0)