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