Community
Participate
Working Groups
Bug 329338 introduces new API. We started a discussion whether BuildContext should be available via IncrementalProjectBuilder#getContext or maybe, like a build kind, it should be passed as an argument to #build. The context lives at the build execution time only, so it does not make sense to call IPB#getContext outside the build operation. Even, if only Build Framework can operate on builders, we should discuss the issue.
We need to address it during M6.
I actually think the current API is OK. Personally I don't think this warrants changing existing API as that will just make the API more complex . Also, there's precedent for methods which provide metadata on the build: #getCommand, #getDelta, #getProject, #isInterrupted, #hasBeenBuilt. All of these are only valid in the context of a build. (In reply to comment #0) > The context lives at the build execution time only, so it does not make sense > to call IPB#getContext outside the build operation. Even, if only Build > Framework can operate on builders, we should discuss the issue. The IncProjBuilder is a call-back based API. Builders don't make sense outside of a core.resources driven #build / #clean call. I don't think clients have, or expect to be able to, use builders outside of an actual build.
Sounds fair. We need to remove EXPERIMENTAL tags then during M6 and we are done :-)
I removed the experimental tags.
