On Thursday, 22 June 2017 12:55:15 PM CEST Robert Yang wrote: > On 06/21/2017 10:31 PM, Paul Eggleton wrote: > > On Tuesday, 13 June 2017 4:36:51 AM CEST Robert Yang wrote: > >> Note, we have to move the location of tinfiol's code to make it can be > >> shutdown correctly. > > > > How does this relate to the rest of this change? Also, can you explain the > > issue you were hitting? The try...finally is supposed to ensure it gets > > shut down. > > I updated the commit message: > > update_layer.py: delete layerbranch for non-existed branch > > The branch is not needed any more when it has been removed from the repo, so > we > also need remove its layerbranch, otherwise it still can be got from the web, > which causes confusions. > > Note, we have to move the location of tinfiol's code to avoid creating tinfoil > when not needed, otherewise, if tinfoil is created (but not needed), and the > program exists earlier before "try ... finally" block, then tinfoil.shutdown() > doesn't run so that it is not shutdown. Move the code back, right before where > it is needed can fix the problem.
OK, that explains the change, thanks. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre -- _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto