On Tue, 2017-06-27 at 15:25 -0500, Aníbal Limón wrote: > In order to avoid corrupt local.conf and bblayers.conf adds > signal handler for SIGTERM and use try/finally (KeyboardIntrrupt) block > to restore previously backuped configuration.
Looks good to me now, with one minor nitpick which I should have noticed earlier: the second patch fixes a problem introduced by the first one (right?), so when applied in this order, bisecting gets broken. I think they should be merged the other way around to avoid that (calling the parent's setUpClass was fine before, it just wasn't required). -- Best Regards, Patrick Ohly The content of this message is my personal opinion only and although I am an employee of Intel, the statements I make here in no way represent Intel's position on the issue, nor am I authorized to speak on behalf of Intel on this matter. -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core