<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-11-30 13:03 GMT+01:00 Paolo Greppi <span dir="ltr"><<a href="mailto:paolo.greppi@libpf.com" target="_blank">paolo.greppi@libpf.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Il 30/11/2017 10:59, Jérémy Lal ha scritto:<br>
> ...<br>
<span class="gmail-">>     Jérémy would you only remove the --shared-zlib ?<br>
>     Since I have no access to any MIPS hardware, I am trying that on amd64, on top of branch master-6.x, at least we'll see if it also breaks the build there.<br>
><br>
>     Anyway is only that blocking nodejs 6 out of testing ?<br>
>     Because if that's so, probably it could be OK to just skip the test and postpone the investigation ... just add a TODO in the patch ?<br>
><br>
> No there's also a segfault #878674 which is not blocking right now,<br>
> but might become again severe.<br>
> Also #859297 the openssl 1.1 "migration" (a bad name since it requires<br>
> heavy changes to nodejs core). On that front, hopefully nodejs 8 will fix it.<br>
> So the goal is more or less upload nodejs 8 asap instead of coping with a<br>
> sinking ship.<br>
><br>
> Jérémy<br>
><br>
<br>
</span>OK so nodejs 6 while in unstable triggered the build failures on mips and mipsel and #878674<br>
<br>
Testing is used by more people than unstable, so it's likely to trigger more trouble if it gets into testing.<br>
<br>
On the other hand if testing were to go from nodejs 4 straight to 8 or 9 it would trigger yet more & all at once.<br>
IMHO better have it introduced in degrees, so that we address the trouble from 4 to 6 separately from that from 6 to 8/9.<br>
<br>
So just send nodejs 6 into testing and let it rain cats & dogs (hey, it's called testing !).<br>
Maybe while 6 is in testing we can move 8/9 to unstable.</blockquote><div><br></div><div>From what i've seen, updating to latest nodejs LTS is usually more of a solution than a problem,</div><div>considering most modules have released versions compatible with node LTS, and sometimes</div><div>incompatible with previous LTS.</div><div>It is true however that more recent v8 has tendency to drop support for legacy cpu (as for < armv7),</div><div>so updating could mean dropping support for one of those architectures.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
In any case I agree that for buster, due to the openssl "migration" nodejs 6 is unacceptable.<br>
BTW according to:<br>
<a href="https://github.com/nodejs/Release#release-schedule" rel="noreferrer" target="_blank">https://github.com/nodejs/<wbr>Release#release-schedule</a><br>
nodejs 8.x will still support OpenSSL-1.0.2.<br>
<br>
In here ("Discussion: OpenSSL 1.1.0 planning"):<br>
<a href="https://github.com/nodejs/node/issues/4270" rel="noreferrer" target="_blank">https://github.com/nodejs/<wbr>node/issues/4270</a><br>
it says "Taking this off the 8.0.0 milestone"<br>
and in here ("Support both OpenSSL 1.1.0 and 1.0.2"):<br>
<a href="https://github.com/nodejs/node/pull/16130" rel="noreferrer" target="_blank">https://github.com/nodejs/<wbr>node/pull/16130</a><br>
it says "The plan would be to land this in ~9.1.0 or thereabouts"<br></blockquote><div><br></div><div>The "build with shared openssl 1.1" switch PR has been merged, so it is available on node 9,</div><div>maybe only on master for now, soon to be released i guess.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
Back to the attempted build using embedded (latest) zlib, in fact deps/zlib is missing from master-6.x because it's removed by the repack.<br>
I'll update you as soon as my attempts give some useful result.<br></blockquote><div><br></div><div>Let me know if you need some help with including zlib again in nodejs source tarball.</div><div>I can do it quickly. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Finally, I noticed that the zlib version embedded in nodejs has been 1.2.11 from at least 4.8 up to current upstream master.<br>
In sid we have zlib1g-dev 1.2.8.dfsg-5 and ubuntu has 1.2.11.dfsg-0ubuntu2.<br>
<br>
So I just reported: <a href="https://bugs.debian.org/883180" rel="noreferrer" target="_blank">https://bugs.debian.org/883180</a></blockquote><div><br></div><div><br></div><div>Thanks for that one.</div><div><br></div><div>Jérémy</div></div></div></div>