<div dir="ltr">All done. Find attached the updated patch series (I hope that was what we needed, I’m losing track over what is already merged and what isn’t).<div><br></div><div>Thanks!</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 3, 2017 at 7:09 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Aug 03, 2017 at 03:35:48PM +0200, Andreas Beckmann wrote:<br>
> > Hmm, so the issue is that /srv/<a href="http://piuparts.debian.org/sbin" rel="noreferrer" target="_blank">piuparts.debian.org/sbin</a> is not in the PATH,<br>
> > hence it doesn’t find the binary. Looking at /etc/piuparts/piuparts.conf, I<br>
> > can’t find a config variable which points to sbin. Do we need to introduce<br>
> > one, or should we use some other approach entirely?<br>
><br>
> Just install the binary into the master binary directory<br>
> $(DESTDIR)$(sharedir)/<wbr>piuparts/master/<br>
> That is in the path for master's cronjob.<br>
<br>
</span>yup, indeed. Michael, please provice a patch which does that.<br>
<br>
> BTW, why sbin?<br>
<br>
good catch! should be bin indeed.<br>
<span class=""><br>
> >> Finally, please do provide a debian/changelog entry for your work, you<br>
> >> can much better describe it than I and I'm lazy ;-)<br>
> Copyright+License statements are missing as well in the new files.<br>
<br>
</span>indeed. Michael, please! :)<br>
<br>
Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
cheers,<br>
        Holger<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Best regards,<br>Michael</div>
</div>