<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 6, 2022 at 6:30 PM Jonas Smedegaard <<a href="mailto:jonas@jones.dk" target="_blank">jonas@jones.dk</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">Quoting Yadd (2022-04-06 18:12:30)<br>
> On 19/03/2022 08:38, Yadd wrote:<br>
> > On 19/03/2022 08:11, Yadd wrote:<br>
> >> On 17/03/2022 19:21, Jérémy Lal wrote:<br>
> >>> Source: node-lodash<br>
> >>> Version: 4.17.21+dfsg+~cs8.31.196.20210220-2<br>
> >>> Severity: important<br>
> >>> Tags: ftbfs<br>
> >>><br>
> >>> This package fails to rebuild. Build log is attached.<br>
> >><br>
> >> The real problem comes from uglify-js call. Build works fine with <br>
> >> uglifyjs 2 (as required by node-locash/package.json) but fails with <br>
> >> uglify-js 3. Here is the real error:<br>
> >><br>
> >>    error: JS_Parse_Error [SyntaxError]: Invalid flags supplied to <br>
> >> RegExp constructor 'node'<br>
> >>        at JS_Parse_Error.get (eval at <anonymous> <br>
> >> (/build/node-lodash-wQJpO3/node-lodash-4.17.21+dfsg+~cs8.31.198.20210220/node_modules/uglify-js/tools/node.js:18:1), <br>
> >> <anonymous>:63:23)<br>
> >>        at formatError (internal/util/inspect.js:1149:38)<br>
> >>        at formatRaw (internal/util/inspect.js:919:14)<br>
> >>        at formatValue (internal/util/inspect.js:774:10)<br>
> >>        at formatProperty (internal/util/inspect.js:1660:11)<br>
> >>        at formatRaw (internal/util/inspect.js:987:9)<br>
> >>        at formatValue (internal/util/inspect.js:774:10)<br>
> >>        at inspect (internal/util/inspect.js:319:10)<br>
> >>        at formatWithOptionsInternal (internal/util/inspect.js:1979:40)<br>
> >>        at formatWithOptions (internal/util/inspect.js:1861:10) {<br>
> >>      filename: '0',<br>
> >>      line: 1,<br>
> >>      col: 0,<br>
> >>      pos: 0<br>
> > <br>
> > For now, I dropped internal minifier and pushed result to experimental. <br>
> > This may break some builds: minified files have to be generated separately.<br>
> > <br>
> > Can someone else have a look at this ?<br>
> <br>
> I'm unable to reproduce this failure in an up-to-date schroot, but build <br>
> fails in salsa CI [1], that's why I reopened this issue.<br>
> <br>
> [1]: <a href="https://salsa.debian.org/js-team/node-lodash/-/jobs/2648172" rel="noreferrer" target="_blank">https://salsa.debian.org/js-team/node-lodash/-/jobs/2648172</a><br>
<br>
Maybe check if upstream has a branch or pending MR with code to upgrade <br>
to UglifyJS v3?<br></blockquote><div><br></div><div>It's mainly useless code coming from lodash-cli.</div><div>The mistake here is that build-dist.js is actually called.</div><div>It does just copy a file and wrongly call uglifyjs, which is done in debian/rules properly anyway.</div><div><br></div><div>Please someone remind us to upload the fix when build deps conflicts are fixed...</div><div><br></div><div>Jérémy</div><div><br></div><div><br></div></div></div>