<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 19, 2018 at 11:02 AM, Arnaud <span dir="ltr"><<a href="mailto:arnaud.rebillout@collabora.com" target="_blank">arnaud.rebillout@collabora.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dear Debian Go Packaging Team,<br>
<br>
The latest version of containerd requires a recent snapshot of<br>
<a href="http://github.com/urfave/cli" rel="noreferrer" target="_blank">github.com/urfave/cli</a>. This can be seen with a command such as:<br>
   <br>
  wget --quiet \<br>
   <br>
<a href="https://raw.githubusercontent.com/containerd/containerd/master/vendor.conf" rel="noreferrer" target="_blank">https://raw.githubusercontent.<wbr>com/containerd/containerd/<wbr>master/vendor.conf</a> \<br>
    -O - | \<br>
    grep urfave<br>
<br>
At the moment, containerd requires the commit 7bc6a0a, from October 2017.<br></blockquote><div><br></div><div>Are you certain that this commit is actually a requirement, or just happens to be the latest version the containerd people tried?</div><div><br></div><div>I’m asking because the changes to the upstream branch are fairly small. While new APIs are introduced (e.g. FlagNamePrefixFunc), none of these APIs seem to be used by the containerd source.</div><div><br></div><div>Did you test the version currently in Debian and run into issues?</div><div><br></div><div>I’m not saying importing new upstream versions isn’t a good idea, I’m just trying to make sure you don’t do any extra work you don’t want to do.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Currently, the version packages in Debian is the latest released,<br>
1.20.0, from August 2017.<br>
<br>
So I updated the Debian package to the snapshot required by containerd.<br>
You can see my attempt at:<br>
<br>
  <a href="https://salsa.debian.org/go-team/packages/golang-github-urfave-cli" rel="noreferrer" target="_blank">https://salsa.debian.org/go-<wbr>team/packages/golang-github-<wbr>urfave-cli</a><br>
<br>
Please notice that:<br>
<br>
- the current debian package for urfave-cli is on <a href="http://anonscm.debian.org" rel="noreferrer" target="_blank">anonscm.debian.org</a>. So<br>
here I had to create a repository on <a href="http://salsa.debian.org" rel="noreferrer" target="_blank">salsa.debian.org</a>. Maybe you don't<br>
like that and already have a procedure for migrating packages from<br>
anonscm to salsa.<br></blockquote><div><br></div><div>While it is fine to create new repositories on salsa, we should either add this repository to the rewritemap ASAP (+cc aviau) or don’t move it for the time being.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- I didn't touch the gbp.conf file, which works good but is not as<br>
simple as the default gpb.conf file create by dh-make-golang. I don't<br>
know how much you want to enforce a common gbp config, and I don't want<br>
to be rude to the persons maintaining this package.<br></blockquote><div><br></div><div>In the long run, I’d like to converge on common gbp settings, but let’s leave it as-is for now.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- Lintian complained about NMU (as I'm not a DD), so I fixed that, but I<br>
don't remember having to do that for the packages I created these last<br>
days. So I don't know if it was right to care about that particular<br>
Lintian warning.<br></blockquote><div><br></div><div>Instead of doing an NMU, add “Team upload.” to the changelog (gbp-dch’s --team option will do that for you), or add yourself as Uploader: in debian/control.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Any comment appreciated !<br>
<br>
Best regards,<br>
<br>
  Arnaud<br>
<br>
</blockquote></div><div class="gmail_extra"><br></div>Overall, your changes look good to me. Let me know which way you prefer to proceed regarding repository location, fix the changelog, and I can upload it for you.<br><br>Thanks!<br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Best regards,<br>Michael</div>
</div></div>