diff options
| author | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2015-02-18 11:23:56 +0000 |
|---|---|---|
| committer | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2015-02-18 16:32:50 +0000 |
| commit | 46a8e0de14ce2cca4c93ec53648d7e01333bbf65 (patch) | |
| tree | a0bf88f196cbff5f181d98b801f3c41f98df4338 /distbuild/initiator.py | |
| parent | 51afd2289db20bcf5063ad9512ab1164154ff093 (diff) | |
| download | morph-46a8e0de14ce2cca4c93ec53648d7e01333bbf65.tar.gz | |
distbuild: Remove the build-steps message
This message was hundreds of kilobytes in size, as it contained a
recursive list of dependencies for each artifact in the build graph. It
was used in the initiator only to print this message:
Build steps in total: 592
This message is now gone. The 'Need to build %d artifacts'
build-progress message now indicates the total build steps instead:
Need to build 300 artifacts, of 592 total
This is a compatible change to the distbuild protocol: old initiators
will continue to work as normal with new controllers that don't send
the build-steps message.
Diffstat (limited to 'distbuild/initiator.py')
| -rw-r--r-- | distbuild/initiator.py | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/distbuild/initiator.py b/distbuild/initiator.py index 378c2cae..548ba496 100644 --- a/distbuild/initiator.py +++ b/distbuild/initiator.py @@ -64,7 +64,6 @@ class Initiator(distbuild.StateMachine): self._ref = ref self._morphology = morphology self._original_ref = original_ref - self._steps = None self._step_outputs = {} self.debug_transitions = False @@ -116,7 +115,6 @@ class Initiator(distbuild.StateMachine): 'build-finished': self._handle_build_finished_message, 'build-failed': self._handle_build_failed_message, 'build-progress': self._handle_build_progress_message, - 'build-steps': self._handle_build_steps_message, 'step-started': self._handle_step_started_message, 'step-already-started': self._handle_step_already_started_message, 'step-output': self._handle_step_output_message, @@ -136,12 +134,6 @@ class Initiator(distbuild.StateMachine): def _handle_build_progress_message(self, msg): self._app.status(msg='Progress: %(msgtext)s', msgtext=msg['message']) - def _handle_build_steps_message(self, msg): - self._steps = msg['steps'] - self._app.status( - msg='Build steps in total: %(steps)d', - steps=len(self._steps)) - def _open_output(self, msg): assert msg['step_name'] not in self._step_outputs if self._step_output_dir: |