<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 10, 2018 at 6:08 PM, Pete Heist <span dir="ltr"><<a href="mailto:pete@eventide.io" target="_blank">pete@eventide.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div><div><blockquote type="cite"><div>On Feb 9, 2018, at 11:43 AM, Pete Heist <<a href="mailto:pete@eventide.io" target="_blank">pete@eventide.io</a>> wrote:</div><div><div style="word-wrap:break-word"><br><div><blockquote type="cite"><span class="gmail-"><div>On Feb 9, 2018, at 8:15 AM, Michael Stapelberg <<a href="mailto:stapelberg@debian.org" target="_blank">stapelberg@debian.org</a>> wrote:</div></span><div><div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><span class="gmail-">On Fri, Feb 9, 2018 at 2:09 AM, Pete Heist <span dir="ltr"><<a href="mailto:pete@eventide.io" target="_blank">pete@eventide.io</a>></span> <wbr>wrote:<br></span><span class="gmail-"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>From the original warnings you sent I know I’ve fixed all except "I: irtt source: testsuite-autopkgtest-missing”<wbr>, but a) I’m not seeing that warning and b) I’m seeing two others. So my questions are:</div><div><br></div><div>1) What are your expectations for 'testsuite-autopkgtest-missing<wbr>', and why am I not seeing it? There are no unit tests to run.</div></div></blockquote><div><br></div><div>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></span></div></div></div></div></blockquote><div><br></div></div></div></div></blockquote><span class="gmail-"><blockquote type="cite"><div style="word-wrap:break-word"><div><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><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"><div style="word-wrap:break-word"><div>2) Do you care about 'debian-watch-does-not-check-g<wbr>pg-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><br></div><div>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><br></div><div>Thanks!</div></div></div></div></blockquote></div></div></blockquote></span></div><br><div>Ok, I’ve updated the irtt package:</div><div><br></div><div>- I added a couple high-value unit tests, which also satisfies lintian.</div><div><br></div><div>- CHANGES.md is split out from README.md to satisfy 'no-upstream-changelog'.</div></div></div></blockquote><div><br></div><div>Great, thanks for both of these! :)</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"><div style="word-wrap:break-word"><div><div><br></div><div>- I tried getting signatures working but punted because GitHub creates a .tar.gz, which I signed with a .tar.gz.asc, but the build process expects a .tar.xz.asc in the upstream, so lintian then warns with 'orig-tarball-missing-<wbr>upstream-signature'. Is there any pkg-go example of signatures working with a GitHub hosted repo? Otherwise I’ll eventually figure it out.</div></div></div></blockquote><div><br></div><div>The process currently is a bit clunky. dh-make-golang hardcodes using xz (via tar J), so either you change that in dh-make-golang’s source, or you manually create the packaging git repository:</div><div><br></div><div># save GitHub .tar.gz as irtt_0.9.0.orig.tar.gz</div><div>% mkdir irtt</div><div>% cd irtt</div><div>% git init</div><div>% gbp import-orig ../irtt_0.9.0.orig.tar.gz</div><div>% cp -r /tmp/dh-make-golang/irtt/debian .</div><div>% git commit -a -m "Initial packaging"</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"><div style="word-wrap:break-word"><div><div><br></div><div>- I blew away the irtt.git repo and started over so I could just stay on the rails of your dh-make-golang.html blog entry, hope that’s fine.</div></div></div></blockquote><div><br></div><div>Sure.</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"><div style="word-wrap:break-word"><div><div><br></div><div>Let me know if anything else. Thanks!</div></div></div></blockquote><div><br></div><div>It would be good to avoid copying the systemd .service file into debian/. I think you can instead install it with a debian/rules override. See e.g. <a href="https://sources.debian.org/src/forked-daapd/25.0-2/debian/rules/?hl=22#L22">https://sources.debian.org/src/forked-daapd/25.0-2/debian/rules/?hl=22#L22</a></div><div><br></div><div>Aside from that, the package looks very clean now.</div><div><br></div><div>Let me know how you’d like to proceed regarding the signatures. It’s not a requirement to sign the releases for now, but note that after the initial upload, we don’t have the luxury of starting with a fresh repository anymore.</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"><div style="word-wrap:break-word"><div><span class="gmail-HOEnZb"><font color="#888888"><div><br></div><div>Pete</div><div><br></div></font></span></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Best regards,<br>Michael</div>
</div></div>