<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi,<br>
<br>
I addressed all your comments, thanks very much for the feedback.
More details bellow.<br>
<br>
<div class="moz-cite-prefix">On 02/28/2018 04:05 PM, Michael
Stapelberg wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr"><br>
<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">
<br>
#### containerd<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/containerd"
rel="noreferrer" target="_blank" moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/containerd</a>><br>
<span class="gmail-">- As for testing the thing really, I
didn't go far yet, I just launched<br>
the binary, the log messages that appeared looked
healthy enough,<br>
and that's all for now.<br>
</span></blockquote>
<div><br>
</div>
<div>Some of the comments below apply
(unnecessary-team-upload, DEP-3 headers for patches).</div>
</div>
</div>
</div>
</blockquote>
<br>
Fixed.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div>Also, this requires
golang-github-containerd-console-dev to build, which is
not in the archive and not in this list of packages?</div>
</div>
</div>
</div>
</blockquote>
<br>
golang-github-containerd-console is available at<br>
<a class="moz-txt-link-rfc2396E" href="https://salsa.debian.org/go-team/packages/golang-github-containerd-console"><https://salsa.debian.org/go-team/packages/golang-github-containerd-console></a><br>
<br>
It was part of the very first packages I made, that's why it's in
`salsa.debian.org/go-team/packages`.<br>
<br>
The last time I heard from it it Debian FTP Masters, it said:<br>
<br>
Your package has been put into the NEW queue, which requires
manual action<br>
from the ftpteam to process. The upload was otherwise valid (it
had a good<br>
OpenPGP signature and file hashes are valid), so please be
patient.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex"><span class="gmail-">
<br>
#### golang-github-docker-go-events<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-docker-go-events"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>github-docker-go-events</a>><br>
</span></blockquote>
<div><br>
</div>
<div>Can you address these please?</div>
<div>
<div>P: golang-github-docker-go-events source:
file-contains-trailing-whitespace debian/control (line
25)</div>
<div>W: golang-github-docker-go-events source:
unnecessary-team-upload</div>
</div>
</div>
</div>
</div>
</blockquote>
<br>
Fixed.<br>
<br>
<blockquote type="cite">
<div>I: golang-github-docker-go-events source:
vcs-field-uses-insecure-uri vcs-git git://<a
href="http://anonscm.debian.org/pkg-go/packages/golang-github-docker-go-events.git">anonscm.debian.org/pkg-go/packages/golang-github-docker-go-events.git</a></div>
</blockquote>
<br>
I don't have this warning, neither from lintian Stretch, neither
from lintian backported from Buster. What do I miss ?<br>
<br>
Anyway, I compared with other packages and fixed it.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex"><span class="gmail-">
<br>
#### golang-github-docker-go-<wbr>metrics<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-docker-go-metrics"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>github-docker-go-metrics</a>><br>
</span></blockquote>
<div><br>
</div>
<div>Can’t build this until
golang-github-prometheus-client-golang is fixed. Some of
the lintian warnings above seem to apply here, too,
though.</div>
</div>
</div>
</div>
</blockquote>
<br>
Lintian warnings fixed.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<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">
<span class="gmail-"><br>
#### golang-github-prometheus-<wbr>client-golang<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-prometheus-client-golang"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>github-prometheus-client-<wbr>golang</a>><br>
</span></blockquote>
<div><br>
</div>
<div>This package cannot be cloned with gbp as-is:</div>
<div>
<div><br>
</div>
<div>% gbp clone <a
href="https://salsa.debian.org/elboulangero-guest/golang-github-prometheus-client-golang"
moz-do-not-send="true">https://salsa.debian.org/elboulangero-guest/golang-github-prometheus-client-golang</a></div>
<div>gbp:info: Cloning from '<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-prometheus-client-golang"
moz-do-not-send="true">https://salsa.debian.org/elboulangero-guest/golang-github-prometheus-client-golang</a>'</div>
<div>gbp:error: Git command failed: Error running git
checkout: error: pathspec 'master' did not match any
file(s) known to git.</div>
</div>
<div><br>
</div>
<div>I think you’ll need to set debian-branch in
debian/gbp.conf.</div>
</div>
</div>
</div>
</blockquote>
<br>
OK, I just changed the default branch in the GitLab settings, and
set it to `debian/sid`, which is the branch I worked on, and which
comes with a proper gbp.conf. The default branch before was
`debian/jessie-backports`, and it doesn't have a gpb.conf.<br>
<br>
So now gbp cone works.<br>
<br>
This package has no `master` branch, only a
`debian/jessie-backports` and `debian/sid`. I think it's what DEP-14
recommends so I prefer not to change anything here, even more
because I'm bumping someone else package.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><span class="gmail-"></span><br>
<span class="gmail-"></span>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex"><span class="gmail-">
</span><span class="gmail-">#### continuity<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/continuity"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/continuity</a>><br>
<<a
href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=890983"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://bugs.debian.org/cgi-<wbr>bin/bugreport.cgi?bug=890983</a>><br>
</span><span class="gmail-">- I ran the series of test
described at<br>
<<a href="https://github.com/containerd/continuity"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://github.com/<wbr>containerd/continuity</a>>,<br>
everything went fine.<br>
</span></blockquote>
<div><br>
</div>
<div>It would be good to add a manpage for the continuity
binary. Otherwise, the package looks fine.</div>
</div>
</div>
</div>
</blockquote>
<br>
Agreed, well, I'm running out of time on that project, so I skip for
now, if it's ok with you.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex"><span class="gmail-">
<br>
#### golang-gogottrpc<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-gogottrpc"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>gogottrpc</a>><br>
<<a
href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=890958"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://bugs.debian.org/cgi-<wbr>bin/bugreport.cgi?bug=890958</a>><br>
</span><span class="gmail-">- Patch and issues submitted
upstream.<br>
</span></blockquote>
<div><br>
</div>
<div>Can you add DEP-3 tags to the patches please? See <a
href="http://dep.debian.net/deps/dep3/"
moz-do-not-send="true">http://dep.debian.net/deps/dep3/</a>.
No need to go overboard, just the
last-updated/author/description/forwarded ones.</div>
</div>
</div>
</div>
</blockquote>
<br>
Fixed.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div><br>
</div>
<div>Also, there seems to be a lintian false-positive: it
reports statically-linked-binary despite the Build-Depends
on golang-go. Can you file a lintian bug about that
please?</div>
</div>
</div>
</div>
</blockquote>
<br>
Ok, I'm doing that right now.<br>
<br>
BTW, in this package I use the following Build-Depends for
`golang-go`:<br>
<br>
golang-go (>= 2:1.9~) | golang-1.9-go,<br>
<br>
Using the epoch `2` in the version here. Is it actually needed ? I'm
not 100% sure, but I think I saw other packages that don't mention
epoch and just do something like `>1.9`, which is more concise.
I'm also not sure about the tilde at then end, I just copy-pasted
that from somewhere :)<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex"><span class="gmail-">
<br>
</span><span class="gmail-">####
golang-github-containerd-btrfs<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-containerd-btrfs"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>github-containerd-btrfs</a>><br>
<<a
href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=890989"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://bugs.debian.org/cgi-<wbr>bin/bugreport.cgi?bug=890989</a>><br>
</span>- I disabled the test to avoid installing a
btrfs-test binary. I don't<br>
think it makes sense to create an additional binary
package<br>
just for shipping this test.<br>
</blockquote>
<div><br>
</div>
<div>Sounds good. Maybe change cmd to cmd/btrfs-test so that
we’ll notice newly introduced binaries upon updates?
Otherwise, the package looks good.</div>
</div>
</div>
</div>
</blockquote>
<br>
Indeed, good catch ! Done.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">
<span class="gmail-"><br>
#### golang-github-containerd-<wbr>typeurl<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-containerd-typeurl"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>github-containerd-typeurl</a>><br>
<<a
href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=891181"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://bugs.debian.org/cgi-<wbr>bin/bugreport.cgi?bug=891181</a>><br>
- Patch submitted upstream<br>
</span></blockquote>
<div><br>
</div>
<div>Please add DEP-3 tags to the patch. Looks good
otherwise.</div>
<div> </div>
</div>
</div>
</div>
</blockquote>
<br>
Done.<br>
<br>
<blockquote type="cite"
cite="mid:CANnVG6=dASXYrL0eyTLbHYnkaX9+eW4xyzYhYB-GsEtK9NgoiA@mail.gmail.com">
<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"><span class="gmail-">
<br>
#### golang-github-dmcgowan-go-tar<br>
<<a
href="https://salsa.debian.org/elboulangero-guest/golang-github-dmcgowan-go-tar"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://salsa.debian.org/<wbr>elboulangero-guest/golang-<wbr>github-dmcgowan-go-tar</a>><br>
<<a
href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=890960"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://bugs.debian.org/cgi-<wbr>bin/bugreport.cgi?bug=890960</a>><br>
- This one was a tricky one, please read the commit
messages for details.<br>
</span></blockquote>
<div><br>
</div>
<div>Does the package which needs this require support for
sparse tar files? If not, I think we can do away with this
package in favor of patching the consumer to use
archive/tar instead. Doing a diff -ur shows that this
package is identical to archive/tar as of Go 1.10, except
for the sparse support, which is scheduled to land in Go
1.11.</div>
</div>
</div>
</div>
</blockquote>
<br>
You're right, and there's no need for sparse support. I patched as
you suggested to use archive/tar.<br>
<br>
Best regards,<br>
<br>
Arnaud<br>
<br>
</body>
</html>