Merge pull request #16 from raducotescu/master

rombert · rombert · commit 3d14705b6501 · 2013-11-19T03:23:16.000-08:00
fix #14: Notifications are not dispatched on Mac OS X 10.9 with terminal-notifier installed
diff --git a/src/main/java/ro/lmn/maven/dmn/impl/MacOSXNotifier.java b/src/main/java/ro/lmn/maven/dmn/impl/MacOSXNotifier.java
@@ -28,7 +28,8 @@ public class MacOSXNotifier extends AbstractNotifier {
 
     @Override
     public void notify(String title, String details, NotificationType notificationType) throws IOException {
-        ProcessBuilder builder = new ProcessBuilder(locator.getPath(TERMINAL_NOTIFIER), "-title", title, "-message", details);
+        ProcessBuilder builder = new ProcessBuilder(locator.getPath(TERMINAL_NOTIFIER), "-title", title, "-message", details, "-sender",
+                "com.apple.Terminal");
         executeProcess(builder);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,8 @@ public class MacOSXNotifier extends AbstractNotifier {`
`28`	`28`
`29`	`29`	`@Override`
`30`	`30`	`public void notify(String title, String details, NotificationType notificationType) throws IOException {`
`31`		`- ProcessBuilder builder = new ProcessBuilder(locator.getPath(TERMINAL_NOTIFIER), "-title", title, "-message", details);`
	`31`	`+ ProcessBuilder builder = new ProcessBuilder(locator.getPath(TERMINAL_NOTIFIER), "-title", title, "-message", details, "-sender",`
	`32`	`+ "com.apple.Terminal");`
`32`	`33`	`executeProcess(builder);`
`33`	`34`	`}`
`34`	`35`