dovecot - Dec 2020 - fyi: please change /var/run to /run in debian buster repository

hey,

just one minor thing:

if there's any time left, please change

in the repository
  debian/buster

in the package
  dovecot-core

in the file
  /lib/systemd/system/dovecot.service

the line
  PIDFile=/var/run/dovecot/master.pid

to
  PIDFile=/run/dovecot/master.pid

because systemd always shows the following warning:

  Dec 12 15:46:30 mail systemd[1]: /lib/systemd/system/dovecot.service:9: 
PIDFile= references path below legacy directory /var/run/, updating 
/var/run/dovecot/master.pid ? /run/dovecot/master.pid; please update the 
unit file accordingly.

creating a drop-in in /etc/systemd/system/dovecot.service.d/ doesn't do 
anything.

greetings...

On 12/12/20 10:07 AM, Michael wrote:
> 
> in the file
> ?/lib/systemd/system/dovecot.service
> 
> the line
> ?PIDFile=/var/run/dovecot/master.pid
> 
> to
> ?PIDFile=/run/dovecot/master.pid
> 
Not that it matters for systemd, but in general, both /var/run and /run
should be replaced by @runstatedir@ which can then be replaced at
build-time by the appropriate value from ./configure --runstatedir.

That allows a single file to work on both Linux and BSD.

The runstatedir support is part of the official autoconf-2.70 release.