Delete duplicate logrotate config for mail.log

[sushain97]

Fixes #212. This appears to be a common strategy: https://github.com/docker-mailserver/docker-mailserver/blob/c29fe3ff0bc888016720536228878b3f19131056/Dockerfile#L259.

Before

root@postfix-mail-0:/tmp# /etc/cron.daily/logrotate
error: rsyslog:1 duplicate log entry for /var/log/mail.log
error: found error in file rsyslog, skipping
root@postfix-mail-0:/tmp# echo $?
1
root@postfix-mail-0:/tmp# cat /etc/logrotate.d/logrotate.conf
/var/log/mail.log {
    copytruncate
    rotate 1
    monthly
    minsize 1M
    compress
    missingok
    notifempty
    dateext
    olddir /var/log/
    maxage 90
}

After

root@postfix-mail-0:/tmp# if egrep -q '^/var/log/mail.log' /etc/logrotate.d/logrotate.conf; then
        echo "Removing /var/log/mail.log from /etc/logrotate.d/rsyslog"
        sed -i -E '/^\/var\/log\/mail.log/d' /etc/logrotate.d/rsyslog
fi
Removing /var/log/mail.log from /etc/logrotate.d/rsyslog
root@postfix-mail-0:/tmp# /etc/cron.daily/logrotate
root@postfix-mail-0:/tmp# echo $?
0
root@postfix-mail-0:/tmp# cat /etc/logrotate.d/rsyslog
/var/log/syslog
/var/log/kern.log
/var/log/auth.log
/var/log/user.log
/var/log/cron.log
{
        rotate 4
        weekly
        missingok
        notifempty
        compress
        delaycompress
        sharedscripts
        postrotate
                /usr/lib/rsyslog/rsyslog-rotate
        endscript
}

[bokysan]

Looks good!

All tests pass? I will go over it tonight.

B

[sushain97]

@bokysan gentle bump on this

[bokysan]

I'll merge this to master and test it out. If it works, I will push out a new version soon.

[sushain97]

@bokysan thanks!

[sushain97]

@bokysan Any word on a new version?

[sushain97]

@bokysan just checking in - could this be released?

[bokysan]

In a couple of days. Traveling ATM.