Bug#977269: [Pkg-javascript-devel] Bug#977269: Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Xavier
Control: fixed -1 7.0.2-3
Control: tags -1 + confirmed

Le 13/12/2020 à 22:52, Jonas Smedegaard a écrit :
> Quoting Xavier (2020-12-13 22:00:53)
>> Le 13/12/2020 à 20:53, Jonas Smedegaard a écrit :
>>> Control: severity 977269 important
>>> Control: tags 977269 +moreinfo +unreproducible
>>>
>>> Quoting Jonas Smedegaard (2020-12-13 20:29:46)
 Quoting Jonas Smedegaard (2020-12-13 17:22:05)
> Quoting Xavier Guimard (2020-12-13 13:19:47)
>> Package: node-rollup-plugin-terser
>> Version: 7.0.2-2
>> Severity: grave
>> Justification: renders package unusable
>>
>> When trying current rollup-plugin-terser (7.0.2)  with current
>> node-terser (4.1.2), package is unuseable:
>>
>> $ rollup -c
>>
>> index.js → dist/pako.js, dist/pako.min.js...
>> [!] (plugin terser) Error: Cannot find module 
>> '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
>>  Please verify that the package.json has a valid "main" entry
> 
>> index.js → dist/pako.js, dist/pako.min.js...
>> minify(...).then is not a function
> 
> "Cannot find module" and "minify(...).then is not a function" are 
> clearly differen issues.  But ok, if the latter is the issue being 
> tracked with this report then I am confident it is solved with the 
> package in incoming.
> 
> Please do close this bugreport when you confirm that is the case.
> 
>  - Jonas
> 



Bug#977269: [Pkg-javascript-devel] Bug#977269: Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Jonas Smedegaard
Quoting Xavier (2020-12-13 22:00:53)
> Le 13/12/2020 à 20:53, Jonas Smedegaard a écrit :
> > Control: severity 977269 important
> > Control: tags 977269 +moreinfo +unreproducible
> > 
> > Quoting Jonas Smedegaard (2020-12-13 20:29:46)
> >> Quoting Jonas Smedegaard (2020-12-13 17:22:05)
> >>> Quoting Xavier Guimard (2020-12-13 13:19:47)
>  Package: node-rollup-plugin-terser
>  Version: 7.0.2-2
>  Severity: grave
>  Justification: renders package unusable
> 
>  When trying current rollup-plugin-terser (7.0.2)  with current
>  node-terser (4.1.2), package is unuseable:
> 
>  $ rollup -c
> 
>  index.js → dist/pako.js, dist/pako.min.js...
>  [!] (plugin terser) Error: Cannot find module 
>  '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
>   Please verify that the package.json has a valid "main" entry

> index.js → dist/pako.js, dist/pako.min.js...
> minify(...).then is not a function

"Cannot find module" and "minify(...).then is not a function" are 
clearly differen issues.  But ok, if the latter is the issue being 
tracked with this report then I am confident it is solved with the 
package in incoming.

Please do close this bugreport when you confirm that is the case.

 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

signature.asc
Description: signature


Bug#977269: [Pkg-javascript-devel] Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Xavier
Le 13/12/2020 à 20:53, Jonas Smedegaard a écrit :
> Control: severity 977269 important
> Control: tags 977269 +moreinfo +unreproducible
> 
> Quoting Jonas Smedegaard (2020-12-13 20:29:46)
>> Quoting Jonas Smedegaard (2020-12-13 17:22:05)
>>> Quoting Xavier Guimard (2020-12-13 13:19:47)
 Package: node-rollup-plugin-terser
 Version: 7.0.2-2
 Severity: grave
 Justification: renders package unusable

 When trying current rollup-plugin-terser (7.0.2)  with current
 node-terser (4.1.2), package is unuseable:

 $ rollup -c

 index.js → dist/pako.js, dist/pako.min.js...
 [!] (plugin terser) Error: Cannot find module 
 '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
  Please verify that the package.json has a valid "main" entry
>>>
>>> Neither node-rollup-plugin-terser nor node-terser asks for files below 
>>> /home/xavier - please try locate which code did that.
>>
>> Something _is_ broken with node-rollup-plugin-terser - the provided 
>> error message is just not really helpful for me.
>>
>> I might have a fix for the error I located - we'll see if that turns out 
>> to also cure the issue you are experiencing, but for future sake please 
>> try unwrap any node_modules tricks when reporting bugs, as they hide 
>> ability to trace what is going on where (at least for me - I have no 
>> doubt that you are fluent in your tools).
> 
> If problem persist with node-rollup-plugin-terser 7.0.2-3 then please 
> try isolate a test causing the failure without node_modules wrapping.
> 
> Otherwise please close.
> 
>  - Jonas

Sorry, in my different tests, I didn't post the good logs. Here are the
relevant logs:

$ rollup -c

index.js → dist/pako.js, dist/pako.min.js...
minify(...).then is not a function

[!] (plugin terser) TypeError: minify(...).then is not a function
TypeError: minify(...).then is not a function
at Object.transform
(/usr/share/nodejs/rollup-plugin-terser/transform.js:5:32)
at execFunction
(/usr/share/nodejs/jest-worker/build/workers/processChild.js:135:17)
at execHelper
(/usr/share/nodejs/jest-worker/build/workers/processChild.js:117:5)
at execMethod
(/usr/share/nodejs/jest-worker/build/workers/processChild.js:121:5)
at process.messageListener
(/usr/share/nodejs/jest-worker/build/workers/processChild.js:46:7)
at process.emit (events.js:314:20)
at emit (internal/child_process.js:876:12)
at processTicksAndRejections (internal/process/task_queues.js:85:21)

When replacing /usr/share/nodejs/terser by version 5 (as recommended in
rollup-plugin-terser package.json), everything works fine.

The rollup.config.js has nothing special
(https://github.com/nodeca/pako/blob/master/rollup.config.js)



Bug#977269: [Pkg-javascript-devel] Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Xavier
Le 13/12/2020 à 22:00, Xavier a écrit :
> Le 13/12/2020 à 20:53, Jonas Smedegaard a écrit :
>> Control: severity 977269 important
>> Control: tags 977269 +moreinfo +unreproducible
>>
>> Quoting Jonas Smedegaard (2020-12-13 20:29:46)
>>> Quoting Jonas Smedegaard (2020-12-13 17:22:05)
 Quoting Xavier Guimard (2020-12-13 13:19:47)
> Package: node-rollup-plugin-terser
> Version: 7.0.2-2
> Severity: grave
> Justification: renders package unusable
>
> When trying current rollup-plugin-terser (7.0.2)  with current
> node-terser (4.1.2), package is unuseable:
>
> $ rollup -c
>
> index.js → dist/pako.js, dist/pako.min.js...
> [!] (plugin terser) Error: Cannot find module 
> '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
>  Please verify that the package.json has a valid "main" entry

 Neither node-rollup-plugin-terser nor node-terser asks for files below 
 /home/xavier - please try locate which code did that.
>>>
>>> Something _is_ broken with node-rollup-plugin-terser - the provided 
>>> error message is just not really helpful for me.
>>>
>>> I might have a fix for the error I located - we'll see if that turns out 
>>> to also cure the issue you are experiencing, but for future sake please 
>>> try unwrap any node_modules tricks when reporting bugs, as they hide 
>>> ability to trace what is going on where (at least for me - I have no 
>>> doubt that you are fluent in your tools).
>>
>> If problem persist with node-rollup-plugin-terser 7.0.2-3 then please 
>> try isolate a test causing the failure without node_modules wrapping.
>>
>> Otherwise please close.
>>
>>  - Jonas
> 
> Sorry, in my different tests, I didn't post the good logs. Here are the
> relevant logs:
> 
> $ rollup -c
> 
> index.js → dist/pako.js, dist/pako.min.js...
> minify(...).then is not a function
> 
> [!] (plugin terser) TypeError: minify(...).then is not a function
> TypeError: minify(...).then is not a function
> at Object.transform
> (/usr/share/nodejs/rollup-plugin-terser/transform.js:5:32)
> at execFunction
> (/usr/share/nodejs/jest-worker/build/workers/processChild.js:135:17)
> at execHelper
> (/usr/share/nodejs/jest-worker/build/workers/processChild.js:117:5)
> at execMethod
> (/usr/share/nodejs/jest-worker/build/workers/processChild.js:121:5)
> at process.messageListener
> (/usr/share/nodejs/jest-worker/build/workers/processChild.js:46:7)
> at process.emit (events.js:314:20)
> at emit (internal/child_process.js:876:12)
> at processTicksAndRejections (internal/process/task_queues.js:85:21)
> 
> When replacing /usr/share/nodejs/terser by version 5 (as recommended in
> rollup-plugin-terser package.json), everything works fine.
> 
> The rollup.config.js has nothing special
> (https://github.com/nodeca/pako/blob/master/rollup.config.js)

I pushed node-pako "as is" to salsa.d.o if you want to take a look. It
seems that node-pako is the first Debian package which will use
rollup-plugin-terser (looking at reverse dependencies).



Bug#977269: [Pkg-javascript-devel] Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Pirate Praveen




On Sun, Dec 13, 2020 at 1:19 pm, Xavier Guimard  wrote:

Package: node-rollup-plugin-terser
Version: 7.0.2-2
Severity: grave
Justification: renders package unusable

When trying current rollup-plugin-terser (7.0.2)  with current
node-terser (4.1.2), package is unuseable:

$ rollup -c

index.js → dist/pako.js, dist/pako.min.js...
[!] (plugin terser) Error: Cannot find module 
'/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'. 
Please verify that the package.json has a valid "main" entry
Error: Cannot find module 
'/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.


There is a possibility that this is caused by symbolic links.

I'm hit by symbolic link issue with yarn so I suspect this could be the 
case here too.


Try adding a custom path to node-resolve plugin instead of adding a 
symbolic link


see this example
https://salsa.debian.org/js-team/node-chart.js/commit/979576e93b7aa17083ae5149e9753a5f6b1a522d



Bug#977269: [Pkg-javascript-devel] Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Jonas Smedegaard
Control: severity 977269 important
Control: tags 977269 +moreinfo +unreproducible

Quoting Jonas Smedegaard (2020-12-13 20:29:46)
> Quoting Jonas Smedegaard (2020-12-13 17:22:05)
> > Quoting Xavier Guimard (2020-12-13 13:19:47)
> > > Package: node-rollup-plugin-terser
> > > Version: 7.0.2-2
> > > Severity: grave
> > > Justification: renders package unusable
> > > 
> > > When trying current rollup-plugin-terser (7.0.2)  with current
> > > node-terser (4.1.2), package is unuseable:
> > > 
> > > $ rollup -c
> > > 
> > > index.js → dist/pako.js, dist/pako.min.js...
> > > [!] (plugin terser) Error: Cannot find module 
> > > '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
> > >  Please verify that the package.json has a valid "main" entry
> > 
> > Neither node-rollup-plugin-terser nor node-terser asks for files below 
> > /home/xavier - please try locate which code did that.
> 
> Something _is_ broken with node-rollup-plugin-terser - the provided 
> error message is just not really helpful for me.
> 
> I might have a fix for the error I located - we'll see if that turns out 
> to also cure the issue you are experiencing, but for future sake please 
> try unwrap any node_modules tricks when reporting bugs, as they hide 
> ability to trace what is going on where (at least for me - I have no 
> doubt that you are fluent in your tools).

If problem persist with node-rollup-plugin-terser 7.0.2-3 then please 
try isolate a test causing the failure without node_modules wrapping.

Otherwise please close.

 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

signature.asc
Description: signature


Bug#977269: [Pkg-javascript-devel] Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Jonas Smedegaard
Quoting Jonas Smedegaard (2020-12-13 17:22:05)
> Quoting Xavier Guimard (2020-12-13 13:19:47)
> > Package: node-rollup-plugin-terser
> > Version: 7.0.2-2
> > Severity: grave
> > Justification: renders package unusable
> > 
> > When trying current rollup-plugin-terser (7.0.2)  with current
> > node-terser (4.1.2), package is unuseable:
> > 
> > $ rollup -c
> > 
> > index.js → dist/pako.js, dist/pako.min.js...
> > [!] (plugin terser) Error: Cannot find module 
> > '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
> >  Please verify that the package.json has a valid "main" entry
> 
> Neither node-rollup-plugin-terser nor node-terser asks for files below 
> /home/xavier - please try locate which code did that.

Something _is_ broken with node-rollup-plugin-terser - the provided 
error message is just not really helpful for me.

I might have a fix for the error I located - we'll see if that turns out 
to also cure the issue you are experiencing, but for future sake please 
try unwrap any node_modules tricks when reporting bugs, as they hide 
ability to trace what is going on where (at least for me - I have no 
doubt that you are fluent in your tools).


 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

signature.asc
Description: signature


Bug#977269: [Pkg-javascript-devel] Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Jonas Smedegaard
Quoting Xavier Guimard (2020-12-13 13:19:47)
> Package: node-rollup-plugin-terser
> Version: 7.0.2-2
> Severity: grave
> Justification: renders package unusable
> 
> When trying current rollup-plugin-terser (7.0.2)  with current
> node-terser (4.1.2), package is unuseable:
> 
> $ rollup -c
> 
> index.js → dist/pako.js, dist/pako.min.js...
> [!] (plugin terser) Error: Cannot find module 
> '/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
>  Please verify that the package.json has a valid "main" entry

Neither node-rollup-plugin-terser nor node-terser asks for files below 
/home/xavier - please try locate which code did that.


 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

signature.asc
Description: signature


Bug#977269: node-rollup-plugin-terser seems incompatible with current node-terser

2020-12-13 Thread Xavier Guimard
Package: node-rollup-plugin-terser
Version: 7.0.2-2
Severity: grave
Justification: renders package unusable

When trying current rollup-plugin-terser (7.0.2)  with current
node-terser (4.1.2), package is unuseable:

$ rollup -c

index.js → dist/pako.js, dist/pako.min.js...
[!] (plugin terser) Error: Cannot find module 
'/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
 Please verify that the package.json has a valid "main" entry
Error: Cannot find module 
'/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/terser/dist/bundle.min.js'.
 Please verify that the package.json has a valid "main" entry
at tryPackage (internal/modules/cjs/loader.js:315:19)
at Function.Module._findPath (internal/modules/cjs/loader.js:528:18)
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:818:27)
at Function.Module._load (internal/modules/cjs/loader.js:687:27)
at Module.require (internal/modules/cjs/loader.js:903:19)
at require (internal/modules/cjs/helpers.js:74:18)
at Object. 
(/home/xavier/dev/debian/src/pkg-js/packages/node-pako/node_modules/rollup-plugin-terser/transform.js:1:20)
at Module._compile (internal/modules/cjs/loader.js:1015:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1035:10)
at Module.load (internal/modules/cjs/loader.js:879:32)


-- System Information:
Debian Release: bullseye/sid
  APT prefers testing
  APT policy: (900, 'testing'), (500, 'unstable')

Kernel: Linux 5.9.0-4-amd64
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages node-rollup-plugin-terser depends on:
ii  node-babel77.12.9+~cs150.130.99-1
ii  node-jest-worker   26.6.3+repack+~cs61.38.31-2
ii  node-serialize-javascript  5.0.1-2
ii  node-terser4.1.2-7

node-rollup-plugin-terser recommends no packages.

node-rollup-plugin-terser suggests no packages.

-- no debconf information