<div dir="auto">Okay, thanks!</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 16 Feb 2022 at 10:12, Yadd <<a href="mailto:yadd@debian.org">yadd@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 16/02/2022 09:24, Caleb Adepitan wrote:<br>
> <br>
> <br>
> On 2/16/22 6:16 AM, Yadd wrote:<br>
>> On 15/02/2022 18:44, Caleb Adepitan wrote:<br>
>>> Hello,<br>
>>><br>
>>> I just updated node-esprima which was some time ago reported to have <br>
>>> failed build with webpack 5. We've fixed issue with webpack 5 and <br>
>>> I've also updated node-esprima as needed.<br>
>>><br>
>>> I've made sure it builds successfully and passes tests including <br>
>>> autopkgtest tests.<br>
>>><br>
>>> Here is the repo that has the changes: <br>
>>> <a href="https://salsa.debian.org/calebpitan/node-esprima" rel="noreferrer" target="_blank">https://salsa.debian.org/calebpitan/node-esprima</a><br>
>>><br>
>>> I'd love to request for sponsorship and uploading.<br>
>>><br>
>>> Thanks,<br>
>>><br>
>>> Caleb Adepitan.<br>
>><br>
>> Here is the changelog.<br>
>>   * first: 4.0.1+ds+~4.0.3-3 wasn't pushed (to NEW)<br>
> So should I keep this version rather than bumping the revision to 4?<br>
> <br>
>>   * second: I don't see any change related to webpack5<br>
> Turns out node-esprima wasn't buggy. It was webpack 5 all along, so <br>
> node-esprima doesn't need any changes related to webpack 5 as it builds <br>
> fine. The only problem with node-esprima was the missing files which I <br>
> tried adding to debian/node-esprima.install.<br>
<br>
Problem exists only on non published version. unstable published all files:<br>
<br>
$ dpkg -L node-esprima<br>
/.<br>
/usr<br>
/usr/bin<br>
/usr/share<br>
/usr/share/doc<br>
/usr/share/doc/node-esprima<br>
/usr/share/doc/node-esprima/README.md<br>
/usr/share/doc/node-esprima/changelog.Debian.gz<br>
/usr/share/doc/node-esprima/changelog.gz<br>
/usr/share/doc/node-esprima/copyright<br>
/usr/share/javascript<br>
/usr/share/javascript/esprima<br>
/usr/share/javascript/esprima/esprima.js<br>
/usr/share/man<br>
/usr/share/man/man1<br>
/usr/share/man/man1/esparse.1.gz<br>
/usr/share/man/man1/esvalidate.1.gz<br>
/usr/share/nodejs<br>
/usr/share/nodejs/@types<br>
/usr/share/nodejs/@types/esprima<br>
/usr/share/nodejs/@types/esprima/index.d.ts<br>
/usr/share/nodejs/@types/esprima/package.json<br>
/usr/share/nodejs/esprima<br>
/usr/share/nodejs/esprima/bin<br>
/usr/share/nodejs/esprima/bin/esparse.js<br>
/usr/share/nodejs/esprima/bin/esvalidate.js<br>
/usr/share/nodejs/esprima/package.json<br>
/usr/share/nodejs/esprima/src<br>
/usr/share/nodejs/esprima/src/assert.js<br>
/usr/share/nodejs/esprima/src/character.js<br>
/usr/share/nodejs/esprima/src/comment-handler.js<br>
/usr/share/nodejs/esprima/src/error-handler.js<br>
/usr/share/nodejs/esprima/src/esprima.js<br>
/usr/share/nodejs/esprima/src/jsx-nodes.js<br>
/usr/share/nodejs/esprima/src/jsx-parser.js<br>
/usr/share/nodejs/esprima/src/jsx-syntax.js<br>
/usr/share/nodejs/esprima/src/messages.js<br>
/usr/share/nodejs/esprima/src/nodes.js<br>
/usr/share/nodejs/esprima/src/parser.js<br>
/usr/share/nodejs/esprima/src/scanner.js<br>
/usr/share/nodejs/esprima/src/syntax.js<br>
/usr/share/nodejs/esprima/src/token.js<br>
/usr/share/nodejs/esprima/src/tokenizer.js<br>
/usr/share/nodejs/esprima/src/xhtml-entities.js<br>
/usr/bin/esparse<br>
/usr/bin/esvalidate<br>
/usr/share/nodejs/esprima/dist<br>
<br>
> Do you think I should report the missing files as a bug in node-esprima, <br>
> probably for historical purpose, although this changes fixes it?<br>
<br>
No, just fix it in salsa.d.o/js-team/node-esprima. Note that your <br>
debian/install was wrong, use mine<br>
</blockquote></div></div>