MFH: r551537 r551607

mail/fetchmail: avoid rc.conf overriding daemon interval, many other fixes

Authors: CH = Corey Halpin, MA = Matthias Andree

• fetchmail's rc script now queries the daemon interval from the configuration, and falls back to the rc.conf value if given. [CH]
• Similarly, the logging facility will be taken from the configuration [MA]
• Add documentation to the rcfile's header comments. [MA]
• Drop support for fetchmail_home_prefix in rc.conf, and query the respective users' home directories with getent instead. [MA]
• In the rc scripts, redirect input from /dev/null so it will not ask for passwords. [MA]
• Add support for the typical 12.1 rc.conf ${name}_... keywords. [MA]
• Make script execution easier to follow by simplifying if...else logic. [CH]
• Fix rcscript's exit code to be 1 if one of the per-user calls fails. [CH]
• Add relevant notes to UPDATING. [MA]

PR: 249860
Submitted by: Corey Halpin (maintainer)
Reported by: Chris James (on fetchmail-users mailing list)
Approved by: Corey Halpin (maintainer)

mail/fetchmail: fix rcscript regression from _1 that broke global mode

In a situation where fetchmail is to be started globally with the
configuration in $LOCALBASE/etc, the rc.d file would try to run
fetchmail for the wrong user.

Simplify script more, avoiding recursive call in single-user mode.

Submitted by: Corey Halpin (maintainer, direct mail to mandree@)
Reported by: Armin Tüting

Approved by: Corey Halpin (maintainer on MFH, direct mail to mandree@)
Approved by: ports-secteam@ (fluffy@)