<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 9, 2018, at 8:15 AM, Michael Stapelberg <<a href="mailto:stapelberg@debian.org" class="">stapelberg@debian.org</a>> wrote:</div><div class=""><div dir="ltr" class=""><br class=""><div class="gmail_extra"><div class="gmail_quote">On Fri, Feb 9, 2018 at 2:09 AM, Pete Heist <span dir="ltr" class=""><<a href="mailto:pete@eventide.io" target="_blank" class="">pete@eventide.io</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><br class=""></div><div class="">From the original warnings you sent I know I’ve fixed all except "I: irtt source: testsuite-autopkgtest-missing”<wbr class="">, but a) I’m not seeing that warning and b) I’m seeing two others. So my questions are:</div><div class=""><br class=""></div><div class="">1) What are your expectations for 'testsuite-autopkgtest-<wbr class="">missing', and why am I not seeing it? There are no unit tests to run.</div></div></blockquote><div class=""><br class=""></div><div class="">Did you apply the ~/.lintianrc I provided? Not doing so would explain the difference. autopkgtest is helpful even if it only compiles the package in question. Why are there no unit tests for your code? :)</div></div></div></div></div></blockquote><div><br class=""></div><div>Yes I did, I should have included that...</div><div><br class=""></div><div>$ lintian --version<br class="">Lintian v2.5.74</div><div>$ ls -al ~/.lintianrc <br class="">-rw-r--r-- 1 pete pete 94 Feb  7 10:25 /home/pete/.lintianrc</div><div>$ cat ~/.lintianrc <br class="">display-info = yes<br class="">display-experimental = yes<br class="">pedantic = yes<br class="">show-overrides = no<br class="">color = auto</div><div>$</div><div><br class=""></div><div>What probably explains your seeing testsuite-autopkgtest-missing and my not is that I was previously using dh-make-golang from Debian stable. The current sid version of dh-make-golang has added a "Testsuite: autopkgtest-pkg-go” line to debian/control, which likely satisfies lintian.</div><div><br class=""></div><div>BTW I manually bumped compat-version from 10 to 11 and standards-version from 4.1.1 to 4.1.3 as recent changes to dh-make-golang which do this aren’t in sid yet.</div><div><br class=""></div><div>As for unit tests, in short, I haven’t gotten there yet, though there’s a bit more to it than that. If you’d like, I can add a few basic ones to satisfy, otherwise they’ll go in after one more design issue is sorted out… :)</div><div><br class=""></div><blockquote type="cite" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">2) Do you care about 'debian-watch-does-not-check-<wbr class="">gpg-signature' and 'no-upstream-changelog’? I do not sign the upstream, and the changes are in README.md, in case those should be split out...</div></div></blockquote><div class=""><br class=""></div><div class="">They’re not important for the time being. Of course, if you could start signing upstream and provide a separate changelog file, that would be great.</div><div class=""><br class=""></div><div class="">Thanks!</div></div></div></div></blockquote><br class=""></div><div>Ok, I’ll take a look at these if there’s time.</div><div><br class=""></div><div>Thanks, almost there… :)</div><div><br class=""></div></body></html>