Display notification when build is done.

dart/tools/build.py

 #!/usr/bin/env python
 #
 # Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 #
 
 import optparse
 import os
 import re
@@ skipping 283 matching lines @@
         has_interesting_info = True
       if has_interesting_info:
         print '\n'.join(chunk)
         chunk = []
       else:
         chunk.append(line)
   if not is_empty_chunk(chunk):
     print '\n'.join(chunk)
 
 
+def NotifyBuildDone(build_config, success):
+  if not success:
+    print "BUILD FAILED"
+

[ricow1, 2014/04/01 07:59:49]

this may be a good place to do a
sys.stdout.flush()
to make sure we get this in the right location for annotated steps based builds
on windows (I am not a 100% sure this is necessary, since this script will exit
right after)

[ahe, 2014/04/01 12:11:07]

Done. I agree this is a good place, just as a precaution.

+  if success:
+    message = 'Build succeeded.'
+  else:
+    message = 'Build failed.'
+  title = build_config

[ricow1, 2014/04/01 07:59:49]

why do we put this into a local variable?

[ahe, 2014/04/01 12:11:07]

Because it more clearly describes that this will be used as the title of the
dialog. I don't think that it is obvious that the title should be a string like
"ReleaseIA32".

+
+  command = None
+  if HOST_OS == 'macos':
+    # Use AppleScript to display a UI non-modal notification.
+    script = 'display notification  "%s" with title "%s" sound name "Glass"' % (
+      message, title)
+    command = "osascript -e '%s' &" % script

[ricow1, 2014/04/01 07:59:49]

I would just remove the command variable and branch at the end and run os.system
(or subprocess.call) here directly, easier to follow flow then IMHO (and below
for linux)

[ahe, 2014/04/01 12:11:07]

I had that originally, but then I had to repeat the comment about ignoring the
return code.

+  elif HOST_OS == 'linux':
+    if success:
+      icon = 'dialog-information'
+    else:
+      icon = 'dialog-error'
+    command = "notify-send -i '%s' '%s' '%s' &" % (icon, message, title)
+
+  if command:
+    # Ignore return code, if this command fails, it doesn't matter.
+    os.system(command)
+
+
 def Main():
   utils.ConfigureJava()
   # Parse the options.
   parser = BuildOptions()
   (options, args) = parser.parse_args()
   if not ProcessOptions(options, args):
     parser.print_help()
     return 1
   # Determine which targets to build. By default we build the "all" target.
   if len(args) == 0:
@@ skipping 83 matching lines @@
             process = subprocess.Popen(args,
                                        stdin=None,
                                        bufsize=1, # Line buffered.
                                        stdout=subprocess.PIPE,
                                        stderr=subprocess.STDOUT)
             FilterEmptyXcodebuildSections(process)
           else:
             process = subprocess.Popen(args, stdin=None)
           process.wait()
           if process.returncode != 0:
-            print "BUILD FAILED"
+            NotifyBuildDone(build_config, success=False)
             return 1
+          else:
+            NotifyBuildDone(build_config, success=True)
 
   return 0
 
 
 if __name__ == '__main__':
   sys.exit(Main())