On 16/12/2016 11:29, Paolo Greppi wrote: > I got the same while testing node-cheerio (ITP: > https://bugs.debian.org/848315). > > The domhandler dependency is indeed listed in the current upstream's > package.json. > > Upgrading node-htmlparser2 from 3.7.3 to 3.9.2 would force us to package > domhandler and domutils, and the latter's dependency node-dom-serializer. > > And I assume this bug would be fixed. > > BTW while updating we could also add debian/watch to this one ... > > Paolo
Now that we have those tiny packages is, and because node-cheerio (https://bugs.debian.org/848315) requires htmlparser2 ^3.9.1, I tried packaging the current version of node-htmlparser2 (3.9.2). Since the existing git repo on alioth does not conform to git-buildpackage, I created a brand new one: https://anonscm.debian.org/cgit/pkg-javascript/node-htmlparser2_new.git In there I moved the control and changelog from the old repo, but most of what npm2dep generated I left untouched. So we get the watch file, and of course no patches to bundle node-dom{elementtype,handler,utils}. It works here: all build/DEP8 tests pass. The DEP8 test of the only reverse dependency node-jsdom fail with the same error as on ci.d.n: https://ci.debian.net/data/packages/unstable/amd64/n/node-jsdom/latest-autopkgtest/log.gz Any comment ? Paolo
