Olav Reinert schrieb:
On Thursday 04 September 2014 10.28:49 Ludwig Nussel wrote:
Olav Reinert schrieb:
Cronic is a small shim shell script for wrapping cron jobs so that cron only sends email when an error has occurred. Cronic defines an error as any non- trace error output or a non-zero result code. Cronic filters Bash execution traces (or anything matching PS4) from the error output, so jobs can be run with execution tracing to aid forensic debugging. Cronic has no options, it simply executes its arguments.
Sounds like a feature that could be implemented in cron itself instead.
Yes, it does. And it's annoying that it isn't there already.
I guess you are suggesting that the script should be added to the cronie package instead of living in its own package? I'm fine with that. (Alternatively, a cronie-util subpackage could be made.)
Or are you suggesting that a patch should be submitted to upstream to add cronic (or something like it) to cronie itself? I could try to do that, but it would take longer.
I'm not suggesting anything. I'm was just thinking aloud :-) Having this feature in cronie directly would be really cool indeed.
There are obvious tmp races ...
The script could set the noclobber flag. Or use mktemp to allocate the temporary files. Would that address your concerns?
Using mktemp would be better, yes. cu Ludwig -- (o_ Ludwig Nussel //\ V_/_ http://www.suse.de/ SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg) -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org