<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 21, 2022 at 8:15 PM Yadd <<a href="mailto:yadd@debian.org">yadd@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
Le 21 mars 2022 19:04:24 GMT+01:00, "Jérémy Lal" <<a href="mailto:kapouer@melix.org" target="_blank">kapouer@melix.org</a>> a écrit :<br>
>On Mon, Mar 21, 2022 at 6:57 PM Yadd <<a href="mailto:yadd@debian.org" target="_blank">yadd@debian.org</a>> wrote:<br>
><br>
>><br>
>><br>
>> On 21/03/2022 18:15, Yadd wrote:<br>
>> > Package: node-builtins<br>
>> > Version: 4.0.0-1<br>
>> > Severity: important<br>
>> ><br>
>> > Hi,<br>
>> ><br>
>> > node-builtins returns a static modules list which is up-to date for<br>
>> > Node.js < 11. It should be patched to use node-builtin-modules result<br>
>> > for node.js >= 12<br>
>> ><br>
>><br>
>> Comparison:<br>
>><br>
>> $ node<br>
>>  > require("builtins")()<br>
>> [<br>
>>    'assert',         'buffer',      'child_process',<br>
>>    'cluster',        'console',     'constants',<br>
>>    'crypto',         'dgram',       'dns',<br>
>>    'domain',         'events',      'fs',<br>
>>    'http',           'https',       'module',<br>
>>    'net',            'os',          'path',<br>
>>    'punycode',       'querystring', 'readline',<br>
>>    'repl',           'stream',      'string_decoder',<br>
>>    'sys',            'timers',      'tls',<br>
>>    'tty',            'url',         'util',<br>
>>    'vm',             'zlib',        'v8',<br>
>>    'process',        'inspector',   'async_hooks',<br>
>>    'http2',          'perf_hooks',  'trace_events',<br>
>>    'worker_threads'<br>
>> ]<br>
>>  > require("builtin-modules")<br>
>> [<br>
>>    'assert',        'async_hooks',    'buffer',<br>
>>    'child_process', 'cluster',        'console',<br>
>>    'constants',     'crypto',         'dgram',<br>
>>    'dns',           'domain',         'events',<br>
>>    'fs',            'http',           'http2',<br>
>>    'https',         'inspector',      'module',<br>
>>    'net',           'os',             'path',<br>
>>    'perf_hooks',    'process',        'punycode',<br>
>>    'querystring',   'readline',       'repl',<br>
>>    'stream',        'string_decoder', 'timers',<br>
>>    'tls',           'trace_events',   'tty',<br>
>>    'url',           'util',           'v8',<br>
>>    'vm',            'wasi',           'worker_threads',<br>
>>    'zlib'<br>
>> ]<br>
>><br>
><br>
>Good catch,<br>
>do you want me to do it ? </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
><br>
>Jérémy<br>
<br>
Yes, thanks !</blockquote><div><br></div><div>Suggested approach would break the module API:</div><br>require('builtins')({ version: '6.0.0' })<br>require('builtins')({ version: '6.0.0', experimental: true }) </div><div class="gmail_quote"><br></div><div class="gmail_quote">it's supposed to be queryable.</div><div class="gmail_quote"><br></div><div class="gmail_quote">I'd rather fix the list.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Jérémy</div><div class="gmail_quote"><br></div></div>