<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 30, 2017 at 4:43 PM, Holger Levsen <span dir="ltr"><<a href="mailto:holger@layer-acht.org" target="_blank">holger@layer-acht.org</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"><span class="gmail-">On Tue, May 23, 2017 at 09:55:11AM +0200, Michael Stapelberg wrote:<br>
> I’m not subscribed, so please keep me cc'ed :).<br>
<br>
</span>ok!<br>
<span class="gmail-"><br>
> > ok, so only parsing 4 suites… how long did these commands run?<br>
> 294,21s user 212,64s system 30% cpu 27:36,09 total<br>
<br>
</span>ok, so minutes rather than hours…<br>
<span class="gmail-"><br>
> > hehe, nice! got an example?<br>
> Sure:<br>
> <a href="https://manpages.debian.org/unstable/postgresql-client-9.6/pg_dump.1.en.html" rel="noreferrer" target="_blank">https://manpages.debian.org/<wbr>unstable/postgresql-client-9.<wbr>6/pg_dump.1.en.html</a><br>
> <a href="https://manpages.debian.org/unstable/vis/vi.1.en.html" rel="noreferrer" target="_blank">https://manpages.debian.org/<wbr>unstable/vis/vi.1.en.html</a><br>
<br>
</span>those are the links to other version, languages and conflicting commands?<br></blockquote><div><br></div><div>The fact that these manpages are included at all is what the slave-alternative handling got us :)</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">
<span class="gmail-"><br>
> > sounds good, how often would you like to run it?<br>
> Once a day would be sufficient, I think.<br>
<br>
</span>cool.<br>
<span class="gmail-"><br>
> Since piuparts.d.o is running on jessie, we’ll need to use golang-go 1.7<br>
> from jessie-backports, please :)<br>
<br>
</span>ok, I'll request this once we got closer… (see below)<br></blockquote><div><br></div><div>Thanks :).</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">
<span class="gmail-"><br>
> Great. Find a patch attached. Note that it’s untested, as golang-go is not<br>
> yet installed and I lack permission to run as piupartsm. I’m not sure what<br>
> your development process is for changes like this, but feel free to fix up<br>
> the patch as necessary.<br>
<br>
</span>process wise it's fine to just install it and then fix it up…<br>
<br>
but…<br>
<br>
+export GOPATH=$(mktemp -d)<br>
+trap 'rm -rf "${GOPATH}"' TERM INT EXIT QUIT<br>
+go get -u <a href="http://github.com/Debian/debiman/cmd/debiman-piuparts-distill" rel="noreferrer" target="_blank">github.com/Debian/debiman/cmd/<wbr>debiman-piuparts-distill</a><br>
+export PATH=${GOPATH}/bin:$PATH<br>
<br>
from distill_alternatives_log from your patch makes me uneasy… because, for a<br>
start, how is the code authenticated?<br></blockquote><div><br></div><div>The code is authenticated via HTTPS. If you don’t trust GitHub and/or the TLS certificate infrastructure, then we should find an alternative.</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">
<br>
Also, I don't expect debiman-piuparts-distill to change very often, do you?<br></blockquote><div><br></div><div>Agreed, most likely it will not change very often.</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">
<span class="gmail-"><br>
> Sure. I’ll add some commentary, and I’m also happy to explain any details —<br>
> just ask.<br>
<br>
</span>thanks, the code is much nicer to read now! :)<br>
<span class="gmail-"><br>
> It could continue living in <a href="http://github.com/Debian/debiman" rel="noreferrer" target="_blank">github.com/Debian/debiman</a>, or we could move it<br>
> to piuparts. The advantage of keeping it in <a href="http://github.com/Debian/debiman" rel="noreferrer" target="_blank">github.com/Debian/debiman</a> is<br>
> that we don’t need to duplicate code between different git repositories.<br>
<br>
</span>does the code need to live in <a href="http://github.com/Debian/debiman" rel="noreferrer" target="_blank">github.com/Debian/debiman</a> at all? One copy<br>
in the piuparts repo should be enough, or?</blockquote><div><br></div><div>That’s correct; I was referring to code that is shared between different debiman components. Specifically, <a href="https://github.com/Debian/debiman/blob/54dd6050a6ce8a454c14e172a8687d93d0fd241b/internal/write/atomically.go">https://github.com/Debian/debiman/blob/54dd6050a6ce8a454c14e172a8687d93d0fd241b/internal/write/atomically.go</a>.</div><div><br></div><div>But, if the conclusion is that debiman-piuparts-distill should live in the piuparts repo, we can duplicate that code. I don’t expect it to change either.</div></div><div><br></div>-- <br><div class="gmail_signature">Best regards,<br>Michael</div>
</div></div>