On Wed, Oct 2, 2013 at 10:08 AM, Ben Pfaff <[email protected]> wrote: > On Wed, Oct 02, 2013 at 09:32:25AM -0700, Gurucharan Shetty wrote: >> From: Duffie Cooley <[email protected]> >> >> commit 46528f78e5c(debian, rhel, xenserver: Ability to collect ovs-ctl logs) >> made changes in the startup scripts such that the o/p of ovs-ctl is logged >> into ovs-ctl.log. But it had an unintended consequence that the exit status >> of ovs-ctl was no longer returned. We would always return success(the exit >> status of tee). >> >> With this commit, we return the exit status of ovs-ctl instead of tee. >> Code referenced from: (line wrapped). >> http://unix.stackexchange.com/questions/14270/\ >> get-exit-status-of-process-thats-piped-to-another/70675#70675) >> >> Signed-off-by: Gurucharan Shetty <[email protected]> >> Signed-off-by: Duffie Cooley <[email protected]> > > Assuming you tested this and it works, > Acked-by: Ben Pfaff <[email protected]> Thanks. I applied this on master, branch 2.0, 1.11 and 1.10
> > Thanks for working out a portable solution that doesn't involve > temporary files or named pipes or other kluges. This solution is > nasty in its own way, but I think it's better. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
