[GitHub] [buildstream] gtristan commented on issue #1263: Caching buildtrees only works with BuildElement (cannot enter a shell after a failed build)

[GitBox <gi...@apache.org>]

gtristan commented on issue #1263:
URL: https://github.com/apache/buildstream/issues/1263#issuecomment-1008553294


   I think this can be easily solved with some policy though:
   
   * If you think your element should have a build tree that gets cached, then structure your element such that what you want to be cached is under `%{build-root}` (of course the element implementation can also *set* `%{build-root}`)
   
   * Don't have the expectation that all elements under the sun will cache their build trees, instead exit more gracefully with an informative error explaining that the element in question does not provide support for this (then we'd get hopefully the same error when trying to `bst shell` into a `filter` or `compose` element).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@buildstream.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org