<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 2 Feb 2020 at 22:48, Jonas Smedegaard <<a href="mailto:jonas@jones.dk">jonas@jones.dk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Quoting Nilesh Patra (2020-02-02 16:01:57)<br>
> I fixed node-jsonld to build with Node.js >= 12. It builds fine in a clean<br>
> chroot, and autopkgtests pass.<br>
> (Didn't use pkg-js-tools since I'm unsure if I should embed modules only<br>
> for test).<br>
> I have pushed my changed here[1].<br>
> (Didn't push to js-team since I was unsure of the changes)<br>
> Needs review and sponsorship.<br>
<br>
Thanks!<br>
<br>
I have merged the parts I found sensible,</blockquote><div><br></div><div>Thank you, :)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> but it fails for me to build <br>
in unstable (did you perhaps intend for it to be targeted experimental <br>
instead?).<br></blockquote><div><br></div><div>Well, it does seem to build fine for me. Sharing the logs[1]. Not sure why that fails for you.</div><div>Do let me know if it is a mistake at my end.</div><div><br></div><div>[1]: <a href="https://git.fosscommunity.in/gi-boi/build-logs/blob/master/jsonld.txt">https://git.fosscommunity.in/gi-boi/build-logs/blob/master/jsonld.txt</a></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I skipped your bumping debhelper compatibility level: I don't like to <br>
blindly doing that, so please tell which actual feature you find is the <br>
reason for such change.  I recommend to generally mention reasoning when <br>
bumping debhelper compatibility level.<br></blockquote><div><br></div><div>I bumped it because it is the newer suggested compatibility version, and lintian suggested doing that as well.</div><div>I don't have any other reason for doing that.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I skipped your adding metadata which to me seem all duplicates of <br>
information already in debian/copyright.  Please clarify (I may very <br>
well have missed something).<br></blockquote><div><br></div><div>Noted this point(to check d/copyright). I included it because I have seen it in a good number of packages.</div><div>Lintian adviced me to do that as well, so added that in. Apologies if these two changes shouldn't have been done. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I reduced your module resolving patch to only add /usr/share/nodejs - if <br>
the two relative paths ('.' and 'node_modules') are really needed then <br>
please explain why (again, I may very well have missed something, but it <br>
looks to me like a dirty hack which might cause trouble at least on <br>
non-clean build environments).<br></blockquote><div><br></div><div>I have faced issues with webpack failing to resolve modules when they are embedded.</div><div>I added that in to avoid webpack failing to recognize those, if in case modules are embedded in future.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Regards,<br>
<br>
 - Jonas<br>
<br>
-- <br>
 * Jonas Smedegaard - idealist & Internet-arkitekt<br>
 * Tlf.: +45 40843136  Website: <a href="http://dr.jones.dk/" rel="noreferrer" target="_blank">http://dr.jones.dk/</a><br>
<br>
 [x] quote me freely  [ ] ask before reusing  [ ] keep private<br>
</blockquote></div></div>