Bug#970910: [Pkg-javascript-devel] Bug#970910: Bug#970910: node-rollup-plugin-babel: autopkgtest failures [PATCH]

2020-09-27 Thread Xavier
Le 26/09/2020 à 11:56, Gianfranco Costamagna a écrit :
> Hello,
> 
> On Sat, 26 Sep 2020 08:41:38 + (UTC) Gianfranco Costamagna 
>  wrote:
>> Hello,
>>
>>> debian/tests/autopkgtest-pkg-nodejs.conf contains
>>>
>>>   "extra_depends=mocha, node-tape, node-rollup-plugin-json (>= 4.1.0)"
>>>
>>> which should be enough for autodep8 ≥ 0.23. If you use autodep8 ≥ 0.23,
>>> this bug should be reassigned to autodep8.
>>
>>
>> interesting, I don't see autodep8 installed in both Debian and Ubuntu, but 
>> maybe its outside the build log (of course)
>> https://autopkgtest.ubuntu.com/packages/n/node-rollup-plugin-babel/groovy/amd64
>>
>> Do you have any clue?
>>
>> Gianfranco
>>
>>
> 
> I confirm it works on my machine after updating autodep8.
> 
> Thanks for the help!
> I hope Ubuntu folks will update autodep8 soon too.
> (for now I workarounded adding the 3 dependencies as build-deps)

Thanks for your work!



Bug#970910: [Pkg-javascript-devel] Bug#970910: node-rollup-plugin-babel: autopkgtest failures [PATCH]

2020-09-26 Thread Gianfranco Costamagna
Hello,

On Sat, 26 Sep 2020 08:41:38 + (UTC) Gianfranco Costamagna 
 wrote:
> Hello,
> 
> >debian/tests/autopkgtest-pkg-nodejs.conf contains
> >
> >  "extra_depends=mocha, node-tape, node-rollup-plugin-json (>= 4.1.0)"
> >
> >which should be enough for autodep8 ≥ 0.23. If you use autodep8 ≥ 0.23,
> >this bug should be reassigned to autodep8.
> 
> 
> interesting, I don't see autodep8 installed in both Debian and Ubuntu, but 
> maybe its outside the build log (of course)
> https://autopkgtest.ubuntu.com/packages/n/node-rollup-plugin-babel/groovy/amd64
> 
> Do you have any clue?
> 
> Gianfranco
> 
> 

I confirm it works on my machine after updating autodep8.

Thanks for the help!
I hope Ubuntu folks will update autodep8 soon too.
(for now I workarounded adding the 3 dependencies as build-deps)

G.



Bug#970910: [Pkg-javascript-devel] Bug#970910: node-rollup-plugin-babel: autopkgtest failures [PATCH]

2020-09-26 Thread Gianfranco Costamagna
Hello,

>debian/tests/autopkgtest-pkg-nodejs.conf contains
>
>  "extra_depends=mocha, node-tape, node-rollup-plugin-json (>= 4.1.0)"
>
>which should be enough for autodep8 ≥ 0.23. If you use autodep8 ≥ 0.23,
>this bug should be reassigned to autodep8.


interesting, I don't see autodep8 installed in both Debian and Ubuntu, but 
maybe its outside the build log (of course)
https://autopkgtest.ubuntu.com/packages/n/node-rollup-plugin-babel/groovy/amd64

Do you have any clue?

Gianfranco



Bug#970910: [Pkg-javascript-devel] Bug#970910: node-rollup-plugin-babel: autopkgtest failures [PATCH]

2020-09-25 Thread Xavier
Control: tags + moreinfo

Le 25/09/2020 à 12:42, Gianfranco Costamagna a écrit :
> Source: node-rollup-plugin-babel
> Version: 5.2.1+~4.4.0-2
> Severity: serious
> tags: patch
> 
> Hello, looks like the autopkgtestsuite fails because of 
> autopkgtest [08:35:49]: test pkg-js-autopkgtest: 
> /usr/share/pkg-js-autopkgtest/runner
> autopkgtest [08:35:49]: test pkg-js-autopkgtest: [---
> Found debian/nodejs/main, package.json is packages/babel/package.json
> Using packages/babel/package.json
> Node module name is @rollup/plugin-babel
> # Copy test files
> Looking at /usr/lib/nodejs/@rollup/plugin-babel
> Looking at /usr/lib/*/nodejs/@rollup/plugin-babel
> Looking at /usr/share/nodejs/@rollup/plugin-babel
> Found /usr/share/nodejs/@rollup/plugin-babel
> Linking /usr/share/nodejs/@rollup/plugin-babel/dist
> Linking /usr/share/nodejs/@rollup/plugin-babel/package.json
> Linking /usr/share/nodejs/@rollup/plugin-babel/types
> Launch debian/tests/pkg-js/test with sh -ex
> + cd packages/babel
> + tape test/as-input-plugin.js test/as-output-plugin.js
> debian/tests/pkg-js/test: 2: tape: not found
> autopkgtest [08:35:49]: test pkg-js-autopkgtest: ---]
> autopkgtest [08:35:50]: test pkg-js-autopkgtest:  - - - - - - - - - - results 
> - - - - - - - - - -
> 
> 
> Adding node-tape to build-dependencies might fix the issue.
> 
> thanks for having a look,
> 
> G.

Hi,

debian/tests/autopkgtest-pkg-nodejs.conf contains

  "extra_depends=mocha, node-tape, node-rollup-plugin-json (>= 4.1.0)"

which should be enough for autodep8 ≥ 0.23. If you use autodep8 ≥ 0.23,
this bug should be reassigned to autodep8.

Cheers,
Xavier



Bug#970910: node-rollup-plugin-babel: autopkgtest failures [PATCH]

2020-09-25 Thread Gianfranco Costamagna
Hello,

> Adding node-tape to build-dependencies might fix the issue.
> 

please note, it might not fail because node-tape is added by some other rollup 
dependency, so its already installed.

But I think if a dependency is directly used by the tool, it should be declared 
explictly, we shouldn't rely
on something that is already available (specially because non essential)

G.



Bug#970910: node-rollup-plugin-babel: autopkgtest failures [PATCH]

2020-09-25 Thread Gianfranco Costamagna
Source: node-rollup-plugin-babel
Version: 5.2.1+~4.4.0-2
Severity: serious
tags: patch

Hello, looks like the autopkgtestsuite fails because of 
autopkgtest [08:35:49]: test pkg-js-autopkgtest: 
/usr/share/pkg-js-autopkgtest/runner
autopkgtest [08:35:49]: test pkg-js-autopkgtest: [---
Found debian/nodejs/main, package.json is packages/babel/package.json
Using packages/babel/package.json
Node module name is @rollup/plugin-babel
# Copy test files
Looking at /usr/lib/nodejs/@rollup/plugin-babel
Looking at /usr/lib/*/nodejs/@rollup/plugin-babel
Looking at /usr/share/nodejs/@rollup/plugin-babel
Found /usr/share/nodejs/@rollup/plugin-babel
Linking /usr/share/nodejs/@rollup/plugin-babel/dist
Linking /usr/share/nodejs/@rollup/plugin-babel/package.json
Linking /usr/share/nodejs/@rollup/plugin-babel/types
Launch debian/tests/pkg-js/test with sh -ex
+ cd packages/babel
+ tape test/as-input-plugin.js test/as-output-plugin.js
debian/tests/pkg-js/test: 2: tape: not found
autopkgtest [08:35:49]: test pkg-js-autopkgtest: ---]
autopkgtest [08:35:50]: test pkg-js-autopkgtest:  - - - - - - - - - - results - 
- - - - - - - - -


Adding node-tape to build-dependencies might fix the issue.

thanks for having a look,

G.