<div dir="auto"><div dir="auto" style="font-family:sans-serif" class="elided-text"><div dir="ltr"><br><br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">>>       * sass-spec is a dev dependency: no need to install it<br>>>      * don't use lintian-overrides to hide some real problems (fixed by ^)<br>>>      * never set an empty debian/tests/pkg-js/test: this produces a bad<br>>>        autopkgtest report (success while no test where played)<br>>>      * enabling test shows that sass-spec isn't compatible (good version is<br>>>        3.5.4-1, not 3.5.4<br>>>      * "src" dir contains only cpp files => not installed<br>>>      * debian/clean was missing<br>>>      * /usr/bin/node-sass was a bad link. I added a feature in pkg-js-tools<br>>>        0.9.8 to help here<br>>><br>>>     I pushed all in js-team/node-node-sass. Please review my changes<br>>><br>>><br>>> Thanks for pushing the changes:)<br>>> I had some doubts, and it would be great if you could clarify them(I<br>>> apologize if they sound like a very newbie doubt):<br>>><br>>>    * How to the exact version of each package that should be used, and<br>>> also what modules should be embedded for the package to be built?.(I do<br>>> it by seeing the build logs, it however seems that they aren't accurate<br>>> to judge the depends)<br>> <br>> I used "npm install" to make test work, then I searched what was the<br>> difference with current failing test<br>> <br>>>    * What exactly are upstream test? and how are they different from<br>>> autopkgtests?<br>> <br>> autopkgtest is a Debian framework that relaunches test on installed<br>> files. Upstream test is the test given in package.json#test (here mocha)<br>> <br>>>    * How is debian/nodejs/files different from debian/install ?<br>> <br>> debian/nodejs/files overrides package.json#files used by<br>> pkg-js-tools/dh_auto_install to know which files have to be installed<br>> <br>> debian/install is not used here since we have a dh_auto_install.<br>> debian/install is a manual install<br>> <br>>>    * `"src" dir contains only cpp files => not installed` : I didn't<br>>> understand the reason completely, would you mind explaining it a little<br>>> more?<br>> <br>> cpp files are useful only for build. A Debian package usually provides<br>> usable files not source ones<br>> <br>>>    * What exactly is a 'bad link'? And how to know if a particular link<br>>> is bad or not?<br>> <br>> /usr/bin/node-sass was linked to a non existent file<br>> <br>>>    * The pristine-tar branch contains the in-publish files, is there a<br>>> way to un-commit them?<br>> <br>> Simply:<br>> $ git checkout pristine-tar<br>> $ ls<br>> $ git rm <unwanted files><br>> $ git commit -a -m 'Remove useless files'<br>> $ git checkout master<br></blockquote></div><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">Thanks a lot! :) </div><div dir="auto" style="font-family:sans-serif">If everything looks fine to you, could you upload the package? (I needed this for node-mermaid) </div><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">Thanks and Regards</div><div dir="auto" style="font-family:sans-serif">Nilesh</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 25 Aug 2019, 14:27 Xavier, <<a href="mailto:yadd@debian.org">yadd@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 25/08/2019 à 10:45, Nilesh Patra a écrit :<br>
> <br>
> <br>
> <br>
>       * sass-spec is a dev dependency: no need to install it<br>
>      * don't use lintian-overrides to hide some real problems (fixed by ^)<br>
>      * never set an empty debian/tests/pkg-js/test: this produces a bad<br>
>        autopkgtest report (success while no test where played)<br>
>      * enabling test shows that sass-spec isn't compatible (good version is<br>
>        3.5.4-1, not 3.5.4<br>
>      * "src" dir contains only cpp files => not installed<br>
>      * debian/clean was missing<br>
>      * /usr/bin/node-sass was a bad link. I added a feature in pkg-js-tools<br>
>        0.9.8 to help here<br>
> <br>
>     I pushed all in js-team/node-node-sass. Please review my changes<br>
> <br>
> <br>
> Thanks for pushing the changes:)<br>
> I had some doubts, and it would be great if you could clarify them(I<br>
> apologize if they sound like a very newbie doubt):<br>
> <br>
>    * How to the exact version of each package that should be used, and<br>
> also what modules should be embedded for the package to be built?.(I do<br>
> it by seeing the build logs, it however seems that they aren't accurate<br>
> to judge the depends)<br>
<br>
I used "npm install" to make test work, then I searched what was the<br>
difference with current failing test<br>
<br>
>    * What exactly are upstream test? and how are they different from<br>
> autopkgtests?<br>
<br>
autopkgtest is a Debian framework that relaunches test on installed<br>
files. Upstream test is the test given in package.json#test (here mocha)<br>
<br>
>    * How is debian/nodejs/files different from debian/install ?<br>
<br>
debian/nodejs/files overrides package.json#files used by<br>
pkg-js-tools/dh_auto_install to know which files have to be installed<br>
<br>
debian/install is not used here since we have a dh_auto_install.<br>
debian/install is a manual install<br>
<br>
>    * `"src" dir contains only cpp files => not installed` : I didn't<br>
> understand the reason completely, would you mind explaining it a little<br>
> more?<br>
<br>
cpp files are useful only for build. A Debian package usually provides<br>
usable files not source ones<br>
<br>
>    * What exactly is a 'bad link'? And how to know if a particular link<br>
> is bad or not?<br>
<br>
/usr/bin/node-sass was linked to a non existent file<br>
<br>
>    * The pristine-tar branch contains the in-publish files, is there a<br>
> way to un-commit them?<br>
<br>
Simply:<br>
$ git checkout pristine-tar<br>
$ ls<br>
$ git rm <unwanted files><br>
$ git commit -a -m 'Remove useless files'<br>
$ git checkout master<br>
<br>
> Thanks and Regards<br>
> Nilesh<br>
</blockquote></div>