Greetings-
Please find attached patch for cleanups and bug fixes of releng
./tools/catalyst-auto script.
I realize there is more than one issue fixed, and some if not most is
cosmetic. Happy to split into multiple patch files, each diff section is
unique and is in order as described here.

==
update copyright header
Cosmetic :)

==
improve send_mail() because it acts more like the logger
 * output failure events to stdout if verbose set, useful for those without
a mailer
 * do not email if recipient var is undef; allows flexibility for
non-releng use cases

I was trying to troubleshoot why the script kept terminating, I didn't have
a mailer setup on that build system yet.
Figured it would also be handy to disable broadcasting messages to the
gentoo devs if testing or for unofficial user builds

==
do not source catalyst.conf as it's not bash syntax
 * non-bash formatted lines are executed; results in several "command not
found" errors
 * instead, parse the file for the specific var

These errors are cleared
/etc/catalyst/catalyst.conf: line 13: digests: command not found
/etc/catalyst/catalyst.conf: line 19: envscript: command not found
/etc/catalyst/catalyst.conf: line 22: options: command not found
/etc/catalyst/catalyst.conf: line 26: autoresume,: command not found
/etc/catalyst/catalyst.conf: line 33: bindist,: command not found
/etc/catalyst/catalyst.conf: line 39: distcc,: command not found
/etc/catalyst/catalyst.conf: line 50: kerncache,: command not found
/etc/catalyst/catalyst.conf: line 53: pkgcache,: command not found
/etc/catalyst/catalyst.conf: line 57: seedcache,: command not found
/etc/catalyst/catalyst.conf: line 66: ]: command not found
/etc/catalyst/catalyst.conf: line 87: distcc_hosts: command not found


==
make it more apparent that bash time is not used.

This will still output "which: no time", with more info now as how to
correct
which: no time in
(/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/bin:/usr/lib/llvm/13/bin)


==
snapshot_cache was removed Apr 2020 catalyst commit
6493f3bc534f97468c514a12035b10e3107c4ecf
* --preclean would fail as catalyst.conf is invalid

Storedir now becomes the only var extracted from
/etc/catalyst/catalyst.cong using the new catalyst_var() function redefined
in this patch


Hope this helps
Thank you :)

-- 

Nic