<div id="geary-body" dir="auto"><div><br></div></div><div id="geary-quote" dir="auto"><br>On Thu, Sep 19, 2019 at 5:40 PM, Paolo Greppi <paolo.greppi@libpf.com> wrote:<br><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Hi,
it now builds on sid:
<a href="https://salsa.debian.org/js-team/node-jest/pipelines/73174">https://salsa.debian.org/js-team/node-jest/pipelines/73174</a>
but there's still a lot to do:
1) turn on tests of course (it tests itself)
2) this lintian message is worth investigating:
P: node-jest-cli: image-file-in-usr-lib usr/lib/nodejs/jest-cli/build/assets/jest_logo.png
3) it needs a manpage for /usr/bin/jest
4) d/copyright is TODO</div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div>cme update dpkg-copyright should cover most cases.<br></span><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">
5) it should generate 6 additional binaries:
- node-jest-changed-files
- node-jest-environment-jsdom
- node-jest-get-type
- node-jest-resolve-dependencies
- node-jest-runner
- node-jest-snapshot
<br></div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div>I think we can include all of them in node-jest binary (single binary like node-babel).</span><div><span style="white-space: pre-wrap;"><br></span><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">6) there are 12 modules in debian/node_modules/ - some of them are quite large:
SLOC Directory SLOC-by-Language (Sorted)
9422 rollup-plugin-node-builtins javascript=9422
5556 levelup javascript=5547,sh=9
2317 level-js javascript=2317
1610 level-filesystem javascript=1610
1552 buffer-es6 javascript=1552
646 rollup-plugin-node-globals javascript=646
425 babel-plugin-transform-inline-imports-commonjs javascript=425
187 process-es6 javascript=187
18 rollup-plugin-flow javascript=18
13 browserify-fs javascript=13
4 string-length javascript=4
3 astral-regex javascript=3
string-length is also a run-time dep (see below)
BTW rollup-plugin-node-builtins is currently used for building node-mocha 4.1.0:
<a href="https://sources.debian.org/src/node-mocha/4.1.0+ds3-5/debian/node_modules/rollup-plugin-node-builtins/">https://sources.debian.org/src/node-mocha/4.1.0+ds3-5/debian/node_modules/rollup-plugin-node-builtins/</a>
(but that dependency is dropped in 6.2.0 which is in experimental)
<br></div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div>These could be embedded. add-node-component + pkg-js-tool's debian/nodejs/<component>/build this would be easier than before.</span></div><div><span style="white-space: pre-wrap;"><br></span><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">7) probably it should be repackaged to exclude examples/react-native:
P: node-jest source: source-contains-prebuilt-java-object examples/react-native/android/gradle/wrapper/gradle-wrapper.jar
N:
N: The source tarball contains a prebuilt Java class file. These are often
N: included by mistake when developers generate a tarball without cleaning
N: the source directory first. If there is no sign this was intended,
N: consider reporting it as an upstream bug as it may be a DFSG violation.
N:
N: Severity: pedantic, Certainty: possible
N:
N: Check: java, Type: binary, source
<br></div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div>yes, we can exclude.</span></div><div><span style="white-space: pre-wrap;"><br></span><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">8) and finally assuming the 6 additional binaries are generated from this source, it would still be uninstallable because there are 8 missing run-time dependencies:
- node-import-local (>= 1.0.0)
- node-istanbul-api (>= 1.1.14)
- node-istanbul-lib-coverage (>= 1.1.1)
- node-istanbul-lib-instrument (>= 1.8.0)
- node-istanbul-lib-source-maps (>= 1.2.1)
- node-node-notifier (>= 5.2.1)
- node-realpath-native (>= 1.0.0)
- node-string-length (>= 2.0.0)
for none of these there's an ITP
if we update to v22.4.4 (the version required by yarnpkg) then these dependencies are still there
if we update to v24.9.0 (current) istanbul-api disappears (it is deprecated from upstream)
in both cases I haven't checked if there are new deps
<br></div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div>I think these can be embedded as well. Probably istambul* could go in node-istanbul.</span></div><div><span style="white-space: pre-wrap;"><br></span><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Paolo
<div>--
</div>Pkg-javascript-devel mailing list
<a href="mailto:Pkg-javascript-devel@alioth-lists.debian.net">Pkg-javascript-devel@alioth-lists.debian.net</a>
<a href="https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-javascript-devel">https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-javascript-devel</a></div></blockquote></div></div>