* since it's too easy to use a hyphen--which is invalid

Fix support for analyze_logs, which wasn't working due to the
immutable variable problems with how Puppet uses ERB.  Restructure
the template to not precalculate the results, group lines for a
particular log together, and format everything a bit better.

newsyslog::config now supports a new analyze_logs parameter, which
specifies the list of logs to run through the analyze action (when
different than the list in logs).  analyze_logs defaults to logs if
not given.

Also update the coding style, simplify the define drastically, and
clean up indentation and comments.

This is the (old) master branch along with the fixes to the
cron file permissions that Russ made.