Trying to make it possible to share createBuildOperation between multiple implementations of SPMCoreBuild.BuildSystem in preparation to introduce an operation that would build plugin tools.
Modifications:
Rename BuildOperationBuildSystemDelegateHandler into LLBuildProgressTracker which is a more neutral name that could be used by different llbuild operations if necessary.
Integrate commandFailureHandler into the progress tracker
Make BuildOperation.createBuildSystem stateless and use a single member to set both a new build system and its tracker.
Result:
The change it make it much easier to move createBuildSystem out of BuildOperation and into SPMCoreBuild.BuildSystem itself.
xedin
commented
2 weeks ago
Motivation:
Trying to make it possible to share
createBuildOperationbetween multiple implementations ofSPMCoreBuild.BuildSystemin preparation to introduce an operation that would build plugin tools.Modifications:
BuildOperationBuildSystemDelegateHandlerintoLLBuildProgressTrackerwhich is a more neutral name that could be used by different llbuild operations if necessary.commandFailureHandlerinto the progress trackerBuildOperation.createBuildSystemstateless and use a single member to set both a new build system and its tracker.Result:
The change it make it much easier to move
createBuildSystemout ofBuildOperationand intoSPMCoreBuild.BuildSystemitself.