Bug#976409: glibc breaks node-iconv autopkgtest: AssertionError [ERR_ASSERTION]: Missing expected exception

2020-12-04 Thread Aurelien Jarno
On 2020-12-04 21:54, Aurelien Jarno wrote:
> On 2020-12-04 21:38, Xavier wrote:
> > Control: tags -1 + ftbfs
> > 
> > Le 04/12/2020 à 20:46, Paul Gevers a écrit :
> > > Source: glibc, node-iconv
> > > Control: found -1 glibc/2.31-5
> > > Control: found -1 node-iconv/2.3.5-4
> > > Severity: serious
> > > Tags: sid bullseye
> > > X-Debbugs-CC: debian...@lists.debian.org
> > > User: debian...@lists.debian.org
> > > Usertags: breaks needs-update
> > > 
> > > Dear maintainer(s),
> > > 
> > > With a recent upload of glibc the autopkgtest of node-iconv fails in
> > > testing on all architectures when that autopkgtest is run with the
> > > binary packages of glibc from unstable. It passes when run with only
> > > packages from testing. In tabular form:
> > > 
> > >passfail
> > > glibc  from testing2.31-5
> > > node-iconv from testing2.3.5-4
> > > all others from testingfrom testing
> > > 
> > > I copied some of the output at the bottom of this report.
> > > 
> > > Currently this regression is blocking the migration of glibc to testing
> > > [1]. Due to the nature of this issue, I filed this bug report against
> > > both packages. Can you please investigate the situation and reassign the
> > > bug to the right package?
> > > 
> > > More information about this bug and the reason for filing it can be found 
> > > on
> > > https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> > > 
> > > Paul
> > Hi,
> > 
> > I rebuild node-iconv (nocheck) and rebuild/autopkgtest reverse
> > dependencies of node-iconv, all passed:
> >  * rebuild: node-body-parser, node-encoding, node-expat,
> > node-iconv-lite, node-raw-body
> >  * autopkgtest: node-body-parser, node-d3-dsv, node-raw-body
> > 
> > So maybe we can just ignore 2 failing tests for now (a "should throw"
> > which doesn't throw).
> > 
> 
> I gave a quick look at the node-iconv code, and looking at
> debian/patches/use-glibc-iconv.patch the issue might be related to the
> changes in the C.UTF-8 locale that does a bit more transliteration than
> before.

The problem is that the 'が' wasn't considered as valid UTF-8 in glibc
2.31-4:

| $ echo 'ça va が' | LC_ALL=C.UTF-8 iconv -f utf-8 -t ascii//translit
| ca va iconv: illegal input sequence at position 7

With glibc 2.31-5 it is considered as valid, although not
transliterable:

| $ echo 'ça va が' | LC_ALL=C.UTF-8 iconv -f utf-8 -t ascii//translit
| ca va ?

Aurelien

-- 
Aurelien Jarno  GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aurel32.net



Bug#976409: glibc breaks node-iconv autopkgtest: AssertionError [ERR_ASSERTION]: Missing expected exception

2020-12-04 Thread Aurelien Jarno
On 2020-12-04 21:38, Xavier wrote:
> Control: tags -1 + ftbfs
> 
> Le 04/12/2020 à 20:46, Paul Gevers a écrit :
> > Source: glibc, node-iconv
> > Control: found -1 glibc/2.31-5
> > Control: found -1 node-iconv/2.3.5-4
> > Severity: serious
> > Tags: sid bullseye
> > X-Debbugs-CC: debian...@lists.debian.org
> > User: debian...@lists.debian.org
> > Usertags: breaks needs-update
> > 
> > Dear maintainer(s),
> > 
> > With a recent upload of glibc the autopkgtest of node-iconv fails in
> > testing on all architectures when that autopkgtest is run with the
> > binary packages of glibc from unstable. It passes when run with only
> > packages from testing. In tabular form:
> > 
> >passfail
> > glibc  from testing2.31-5
> > node-iconv from testing2.3.5-4
> > all others from testingfrom testing
> > 
> > I copied some of the output at the bottom of this report.
> > 
> > Currently this regression is blocking the migration of glibc to testing
> > [1]. Due to the nature of this issue, I filed this bug report against
> > both packages. Can you please investigate the situation and reassign the
> > bug to the right package?
> > 
> > More information about this bug and the reason for filing it can be found on
> > https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> > 
> > Paul
> Hi,
> 
> I rebuild node-iconv (nocheck) and rebuild/autopkgtest reverse
> dependencies of node-iconv, all passed:
>  * rebuild: node-body-parser, node-encoding, node-expat,
> node-iconv-lite, node-raw-body
>  * autopkgtest: node-body-parser, node-d3-dsv, node-raw-body
> 
> So maybe we can just ignore 2 failing tests for now (a "should throw"
> which doesn't throw).
> 

I gave a quick look at the node-iconv code, and looking at
debian/patches/use-glibc-iconv.patch the issue might be related to the
changes in the C.UTF-8 locale that does a bit more transliteration than
before.

Aurelien

-- 
Aurelien Jarno  GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aurel32.net



Bug#976409: glibc breaks node-iconv autopkgtest: AssertionError [ERR_ASSERTION]: Missing expected exception

2020-12-04 Thread Xavier
Control: tags -1 + ftbfs

Le 04/12/2020 à 20:46, Paul Gevers a écrit :
> Source: glibc, node-iconv
> Control: found -1 glibc/2.31-5
> Control: found -1 node-iconv/2.3.5-4
> Severity: serious
> Tags: sid bullseye
> X-Debbugs-CC: debian...@lists.debian.org
> User: debian...@lists.debian.org
> Usertags: breaks needs-update
> 
> Dear maintainer(s),
> 
> With a recent upload of glibc the autopkgtest of node-iconv fails in
> testing on all architectures when that autopkgtest is run with the
> binary packages of glibc from unstable. It passes when run with only
> packages from testing. In tabular form:
> 
>passfail
> glibc  from testing2.31-5
> node-iconv from testing2.3.5-4
> all others from testingfrom testing
> 
> I copied some of the output at the bottom of this report.
> 
> Currently this regression is blocking the migration of glibc to testing
> [1]. Due to the nature of this issue, I filed this bug report against
> both packages. Can you please investigate the situation and reassign the
> bug to the right package?
> 
> More information about this bug and the reason for filing it can be found on
> https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> 
> Paul
Hi,

I rebuild node-iconv (nocheck) and rebuild/autopkgtest reverse
dependencies of node-iconv, all passed:
 * rebuild: node-body-parser, node-encoding, node-expat,
node-iconv-lite, node-raw-body
 * autopkgtest: node-body-parser, node-d3-dsv, node-raw-body

So maybe we can just ignore 2 failing tests for now (a "should throw"
which doesn't throw).

NB: I tried also with node-iconv 3.0.0, same problem in test.



Bug#976409: glibc breaks node-iconv autopkgtest: AssertionError [ERR_ASSERTION]: Missing expected exception

2020-12-04 Thread Paul Gevers
Source: glibc, node-iconv
Control: found -1 glibc/2.31-5
Control: found -1 node-iconv/2.3.5-4
Severity: serious
Tags: sid bullseye
X-Debbugs-CC: debian...@lists.debian.org
User: debian...@lists.debian.org
Usertags: breaks needs-update

Dear maintainer(s),

With a recent upload of glibc the autopkgtest of node-iconv fails in
testing on all architectures when that autopkgtest is run with the
binary packages of glibc from unstable. It passes when run with only
packages from testing. In tabular form:

   passfail
glibc  from testing2.31-5
node-iconv from testing2.3.5-4
all others from testingfrom testing

I copied some of the output at the bottom of this report.

Currently this regression is blocking the migration of glibc to testing
[1]. Due to the nature of this issue, I filed this bug report against
both packages. Can you please investigate the situation and reassign the
bug to the right package?

More information about this bug and the reason for filing it can be found on
https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation

Paul

[1] https://qa.debian.org/excuses.php?package=glibc

https://ci.debian.net/data/autopkgtest/testing/amd64/n/node-iconv/8618841/log.gz

autopkgtest [07:11:24]: test pkg-js-autopkgtest:
/usr/share/pkg-js-autopkgtest/runner
autopkgtest [07:11:24]: test pkg-js-autopkgtest: [---
# Using package.json
# Node module name is iconv
# Build files found:
# Test files found: test
# Files/dir to be installed from source: test
# Copy test files
# Copy debian/tests/pkg-js content
'debian/tests/pkg-js' ->
'/tmp/autopkgtest-lxc.djwhve30/downtmp/autopkgtest_tmp/smokeZDEmls/debian/tests/pkg-js'
'debian/tests/pkg-js/test' ->
'/tmp/autopkgtest-lxc.djwhve30/downtmp/autopkgtest_tmp/smokeZDEmls/debian/tests/pkg-js/test'
# Searching module in /usr/lib/nodejs/iconv
# Searching module in /usr/lib/aarch64-linux-gnu/nodejs/iconv
# Found /usr/lib/aarch64-linux-gnu/nodejs/iconv
# Searching files to link in /usr/lib/aarch64-linux-gnu/nodejs/iconv
'./build' -> '/usr/lib/aarch64-linux-gnu/nodejs/iconv/build'
'./lib' -> '/usr/lib/aarch64-linux-gnu/nodejs/iconv/lib'
'./package.json' -> '/usr/lib/aarch64-linux-gnu/nodejs/iconv/package.json'
# Searching module in /usr/share/nodejs/iconv
# Launch debian/tests/pkg-js/test with sh -ex
+ mkdir -p test/tmp
+ node test/run-tests.js
assert.js:101
  throw new AssertionError(obj);
  ^

AssertionError [ERR_ASSERTION]: Missing expected exception.
at Object.
(/tmp/autopkgtest-lxc.djwhve30/downtmp/autopkgtest_tmp/smokeZDEmls/test/test-basic.js:136:8)
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)
at Function.Module._load (internal/modules/cjs/loader.js:724:14)
at Module.require (internal/modules/cjs/loader.js:903:19)
at require (internal/modules/cjs/helpers.js:74:18)
at Object.
(/tmp/autopkgtest-lxc.djwhve30/downtmp/autopkgtest_tmp/smokeZDEmls/test/run-tests.js:2:1)
at Module._compile (internal/modules/cjs/loader.js:1015:30)
at Object.Module._extensions..js
(internal/modules/cjs/loader.js:1035:10) {
  generatedMessage: false,
  code: 'ERR_ASSERTION',
  actual: undefined,
  expected: undefined,
  operator: 'throws'
}
autopkgtest [07:11:25]: test pkg-js-autopkgtest: ---]




OpenPGP_signature
Description: OpenPGP digital signature