Terminal-notifier hangs #46

Closed
drewcrawford opened this Issue Mar 27, 2013 · 3 comments

2 participants

@drewcrawford

I've noticed that terminal-notifier has started hanging recently. Maybe it's related to 10.8.3 which came down the pipe recently?

I put a sample from the hang into a gist:

https://gist.github.com/drewcrawford/5257395

@drewcrawford

I am mysteriously no longer able to reproduce this issue, although I have no idea what changed.

I'm gonna close this out since without reproduction it will be hard to track down...

@drewcrawford

I was able to reproduce this. It turns out, for reasons that are a mystery to me, the Info.plist for terminal-notifier.app was bad. Why exactly this causes the issue I'm not sure, nor am I sure how the plist (and version number!) changed without any update to any of the binaries etc. But there you go--applying and reverting the patch below will toggle the issue on and off.

I'm wondering if maybe the notification center framework internally is looking at version numbers of apps or something.

shadowfax:ConstructPM drew$ diff -u old-list.txt new-list.txt 
--- old-list.txt    2013-04-05 01:56:42.000000000 -0500
+++ new-list.txt    2013-04-05 01:56:51.000000000 -0500
@@ -1,17 +1,17 @@
 Dict {
+    CFBundleName = terminal-notifier
     DTXcode = 0440
-    NSHumanReadableCopyright = Copyright © 2012 Eloy Durán. All rights reserved.
     DTSDKName = macosx10.8
-    CFBundleName = terminal-notifier
+    NSHumanReadableCopyright = Copyright © 2012 Eloy Durán. All rights reserved.
     DTSDKBuild = 12A264
     CFBundleDevelopmentRegion = en
-    CFBundleVersion = 4e43c0
+    CFBundleVersion = 7
     BuildMachineOSBuild = 12A269
     NSPrincipalClass = NSApplication
     NSMainNibFile = MainMenu
-    CFBundleShortVersionString = 1.4.207
     CFBundlePackageType = APPL
     CFBundleIconFile = Terminal
+    CFBundleShortVersionString = 1.4.2
     CFBundleInfoDictionaryVersion = 6.0
     CFBundleExecutable = terminal-notifier
     DTCompiler = 
`
@alloy
Collaborator

Wow, that’s some nice investigative work 👍

The notification center stuff is very much bound to Launch Services, so I can easily see that getting messed up when the Info.plist changes.

At least it’s nice to have a ticket that people can find to help them out in case they run into the same issue. Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment