On Wed, 02 Oct 2013 11:25:47 -0400, <[email protected]> wrote: > Given that the patch affects several files, a simpler way to > accomplish the same (particularly so long as the patch is not formally > merged) is to just add something like /bin/true to the end of each > command that you want to always succeed (as per Craig's note a few > weeks back, a sequence of commands can be given so this can be > appended to the end). Seems much cleaner and both simpler to > understand and implement...
IIUC, a sequence of commands can *not* be given, since a shell is not spawned. If one wants multiple commands, a script file containing all of them has to be created, and then backuppc told to run that file. And anyway, one may want to know and be notified if a script fails. For instance, I have a post-dump script that sends an email and updates an entry in a database; if that fails, I want it to really fail-fail (not pretend that it worked because I put "|| /bin/true" at the end), but at the same time I'd like backuppc to keep the backup, since the data that was transferred is good and in my case it's pointless to throw it away only because the post processing script failed. Note that there may be situations where it *does* makes sense to fail the whole backup instead; I'm just saying that one should be able to tell backuppc what to do, instead of blindly assuming "failure == delete dump" (or "failure == keep backup") for everything. -- D. ------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60134791&iu=/4140/ostg.clktrk _______________________________________________ BackupPC-devel mailing list [email protected] List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel Wiki: http://backuppc.wiki.sourceforge.net Project: http://backuppc.sourceforge.net/
