[yocto] [PATCH V4 10/10] update_layer.py: delete layerbranch for non-existed branch

Paul Eggleton paul.eggleton at linux.intel.com
Wed Jun 21 07:31:38 PDT 2017


On Tuesday, 13 June 2017 4:36:51 AM CEST Robert Yang wrote:
> 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.

I think I'd previously avoided this since I was concerned we'd potentially 
lose data about a branch which was manually gathered (such as maintainer 
information) however I can't actually think of a scenario now where you would 
lose valuable data - if the branch is gone then there's not much point keeping 
a reference to it around, so this should be OK.

> 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.

Cheers,
Paul


-- 

Paul Eggleton
Intel Open Source Technology Centre



More information about the yocto mailing list