Closed ghost closed 9 years ago
Also adding very useful variable to add rsync options
Do you care merging? There is also extra options for rsync that just must go to master. I can extract it to other pull request or you can cherry-pick.
afaik, this project unfortunately is pretty much dead. no development for years...
Any friends debian maintainers? Don't want to go through long process of sponsored upload.
nope
I'm digging this PR, could you add test case(s) for it?
@kissifrot I don't think I'll have time to do this in near future
"Directly" means that configured command runs without forced redirection, allowing administrator more flexibility. For example, if backup-manager is being run from cron, administrator can setup BM_POST_BACKUP_COMMAND like this: do-some-checking-stuff >/dev/null In this case, if there is errors on stderr, cron will mail it. Anyway, it allows much more things like logging.
The point is to allow admin to manipulate streams the way he/she wants. If pre-command exists with non-zero status backup will not proceed.
This change is not bc in the matter of "false" output.