Ok, I took another look and I now see that my first understanding of the logic of the script was off. My changes now boil down to the following:

• Removal of the undocumented variable LOGWATCH_TRIGGER and condition that always evaluates true in the start-mailserver.sh script.
• Addition of appropriate warning messages when reports are turned off, not misleading errors.
• Removal of PFLOGSUMM_TRIGGER condition that also always evaluates to true (the variable is set in a later function, but cannot be "none" when the condition is evaluated).

Ok, I took another look and I now see that my first understanding of the logic of the script was off. My changes now boil down to the following:

* Removal of the undocumented variable LOGWATCH_TRIGGER and condition that always evaluates true in the start-mailserver.sh script.

* Addition of appropriate warning messages when reports are turned off, not misleading errors.

* Removal of PFLOGSUMM_TRIGGER condition that also always evaluates to true (the variable **is** set in a later function, but cannot be "none" when the condition is evaluated).

I like this. Nevertheless, I will request a review more. But I will already approve of this :)

Perhaps the message should be info rather than warning as having the reports disabled is perfectly normal? Other than that it looks fine!

That's a good idea @erik-wramner. @erdos4d I will just do this, as this is the fastest and easiest way. Then I will merge it.