<p dir="ltr"></p>
<p dir="ltr">On 23 Oct 2016 4:41 a.m., "Niels Thykier" <<a href="mailto:niels@thykier.net">niels@thykier.net</a>> wrote:<br>
><br>
> Felipe Sateler:<br>
> > Package: debhelper<br>
> > Version: 10.2.2<br>
> > Severity: wishlist<br>
> > Tags: patch upstream<br>
> ><br>
> > Hi,<br>
> ><br>
> > Please find attached a patch for installing timer units in<br>
> > debian/package.timer, and a small refactor to DRY.<br>
> ><br>
> ><br>
> > [...]<br>
><br>
><br>
> Hi,<br>
><br>
> Thanks for the patches; I am happy to apply them pending a review from<br>
> the systemd maintainers.</p>
<p dir="ltr">Sure.</p>
<p dir="ltr">><br>
> AFAICT, a side-effect of the refactoring is that dh_systemd_enable will<br>
> now *also* install debian/<pkg>@.target (etc.) if present.  Does that<br>
> make sense in general case ?</p>
<p dir="ltr">Well, the only one that is forbidden is for .mount units, as the unit name must match the mount path (but then, installing  <pkg>.mount doesn't make sense either).</p>
<p dir="ltr">For the rest there is nothing forbidding it, although tmpfiles do not make much sense.</p>
<p dir="ltr">If you prefer I can unroll the loop and remove the ones that don't make sense.</p>
<p dir="ltr">><br>
> At the same time, I also noticed that the "FILES" section in<br>
> dh_systemd_enable seems a bit out of date. :)</p>
<p dir="ltr">Yes. I didn't know how to present a list so I refrained from fixing this. Or should we just have a section for each filetype?<br></p>
<p dir="ltr">Saludos </p>