<div dir="ltr"><div>Hi<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> 
* install: good, just to set a minimal version for pkg-js-tools to help<br>
            backports (0.9 at least)<br>
<br>
 * tests: you didn't use pkg-js-tools for tests, then your autopkgtest<br>
          tests are bad. Todo:<br>
           $ mkdir -p debian/tests/pkg-js<br>
           $ git rm debian/tests/{control,require}<br>
           $ echo 'tap test/*.js' > debian/tests/pkg-js/test<br>
           $ git add debian/tests/pkg-js/test<br>
   Then remove override_dh_auto_test entry from debian/rules, add<br>
   missing "Testsuite: autopkgtest-pkg-nodejs" in debian/control and<br>
   commit<br>
<br>
 * clean: you used an override_dh_auto_clean to remove on file, then<br>
          dh_auto_test is disabled. This is not a good practice. Prefer<br>
          to use a debian/clean file:<br>
           $ echo 'debian/semver.1' > debian/clean<br>
           $ git add debian/clean<br>
   Then remove override_dh_auto_test from debian/rules and commit<br>
<br>
 * links: you didn't take a look at your package (using debc), you have<br>
          2 bad symlinks. To fix that (only one is needed):<br>
        $ echo 'semver/bin/semver /usr/bin/semver' > debian/nodejs/links<br>
<br>
Then commit your changes in js-team/node-semver<br></blockquote><div><br></div><div>I have done the changes. Could you please review, if it looks okay? I also don't have access to js-team/node-semver, please grant me so that I may push.</div><div>I also had a doubt: <br></div><div>I read the man pages, I kind of get an idea, but could you explain how debc works, and how I can check bad symlinks (I am not sure about this one, since doing debc on changes file didn't show bad links before)  ?</div><div>That would be great.</div><div><br></div><div>Thanks and Regards</div><div>Nilesh<br></div></div></div>