[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