Hey,

On Sun, Mar 12, 2017 at 05:42:34PM +0100, Wolfgang Schweer wrote:
> On Sun, Mar 12, 2017 at 09:16:46AM +0100, Evgeni Golov wrote:
> > I went a bit further, droped that code, and migrated the setting
> > to update-rc.d on upgrade. See attached patchset.
> > 
> > What do you think?
> 
> Looks awesome.
> 
> Tested all possible variants using both systemd and sysv related 
> commands. Seems to work like expected in all cases.

Thanks for testing!

> One point:
> For the d/NEWS file to be actually shipped, at least for me this 
> additional patch was needed:

Uh? It should be automatically installed by dh_installchangelogs.
And happens here (it gets installed as /usr/share/doc/thinkfan/NEWS.Debian.gz).
Wonder why this did not work for you.

Btw, are you interested in co-maintaining thinkfan? The package could certainly
use another pair of eyes :)

Gruesse
Evgeni

Reply via email to