Include initialization time in the total session time of the build log

This patch refactors the frontend App object and touches a few internals.

  o The scheduler now takes a start time given to it at instantiation time,
    instead of considering the start time to commence in Scheduler.run()

  o The App.initialize() method has been swapped out for an
    App.initialized() context manager.

    This context manager now takes care of the main start/fail/success
    messages. For convenience and ensured consistency, this context manager
    is now responsible for:

      o Printing the startup heading
      o Printing the end of session summary
      o Exiting with a consistent error status in case of errors

  o The Pipeline() object no longer prints the start/fail/success messages,
    as they are now taken care of by the App()

  o The cli.py frontend code is much simplified by using the new context
    manager, also enforcing consistency in how we handle and report errors.

This fixes issue #186

1 files changed, 3 insertions, 1 deletions

diff --git a/buildstream/_exceptions.py b/buildstream/_exceptions.py
index 8a4f13aea..fb42d9fd2 100644
--- a/buildstream/_exceptions.py
+++ b/buildstream/_exceptions.py
@@ -220,7 +220,7 @@ class ArtifactError(BstError):
 #
 class PipelineError(BstError):
 
-    def __init__(self, message=None, *, detail=None, reason=None):
+    def __init__(self, message=None, *, detail=None, reason=None, terminated=False):
 
         # The empty string should never appear to a user,
         # this only allows us to treat this internal error as
@@ -229,3 +229,5 @@ class PipelineError(BstError):
             message = ""
 
         super().__init__(message, detail=detail, domain=ErrorDomain.PIPELINE, reason=reason)
+
+        self.terminated = terminated