Added terminal-notifier to the list of packages that should be installed with homebrew. draft
authorNathan Hakkakzadeh <nhakkakzadeh@mozilla.com>
Mon, 23 May 2016 16:13:38 -0700
changeset 369901 fa54bde945c8b8d97a97781e1215d43eb5e49860
parent 369317 16663eb3dcfa759f25b5e27b101bc79270c156f2
child 521646 eac97c81dff3e83aa13138f47cb98fda6422aafc
push id18945
push userbmo:nhakkakzadeh@mozilla.com
push dateMon, 23 May 2016 23:26:38 +0000
milestone49.0a1
Added terminal-notifier to the list of packages that should be installed with homebrew. Before, ./mach build would try to use terminal-notifier after building, but would not be able to since it isn't installed. MozReview-Commit-ID: 4oBAVfOdcNs
python/mozboot/mozboot/osx.py
--- a/python/mozboot/mozboot/osx.py
+++ b/python/mozboot/mozboot/osx.py
@@ -310,17 +310,18 @@ class OSXBootstrapper(BaseBootstrapper):
             # development headers which are missing from OS X (at least on
             # 10.8) and because the build system wants a version newer than
             # what Apple ships.
             ('python', 'python'),
             ('mercurial', 'mercurial'),
             ('git', 'git'),
             ('autoconf213', HOMEBREW_AUTOCONF213),
             ('gnu-tar', 'gnu-tar'),
-            ('watchman', 'watchman',)
+            ('watchman', 'watchman',),
+            ('terminal-notifier', 'terminal-notifier')
         ]
         self._ensure_homebrew_packages(packages)
 
     def ensure_homebrew_browser_packages(self):
         packages = [
             ('yasm', 'yasm'),
         ]
         self._ensure_homebrew_packages(packages)