crankd doesn't handle nsdistributednotifications, now it does

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

I have GeekTool installed(v3 is applescriptable. In particular, you can
tell it to refresh certain geeklets, allowing one to update a geeklet on an
event supported by crankd) and I wanted a way to avoid polling for iTunes
track information because that drained my macbook's battery pretty quickly
so I took a look at crankd.  I quickly realized that it didn't support
NSDistributedNotifications so I wrote in support for that particular
notification type.
When the initial patch was finished, I discovered that there might be a
quirk(maybe this "quirk" is by design, I don't know) with the way the
NSDistributedNotificationCenter handles an application launch/quit.  I
found that if you're subscribed to an event and the application emitting
that event quits and subsequently relaunches, the handler will never be
called again.
I wrote another patch to fix this problem.  It adds a config option for
NSDistrbutedNotifications, process, that tells crankd to subscribe to
ApplicationDidLaunch events with a special handler, do_reload.  When an
application is launched, the do_reload handler is called and checks the
process name against the config option.  If they match, the
DistNotification is reloaded and hence the handler is valid once again. 
This handler won't replace any existing appdidlaunch events.

The patched version of crankd is attached to this issue with changes marked
by the comment "# distnot patch: change" (where change is what was added).
 FYI: All spaces were converted to tabs.

btw, I wanted to take this opportunity to thank you for crankd, it's been
working amazingly well for me :D.

Original issue reported on code.google.com by [email protected] on 10 Dec 2009 at 9:34

Attachments:

• crankd.py

Copied from original issue: #2

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

edit: remove the line containing "del RELAUNCH_IDS[event]" otherwise the 
appdidlaunch
event will be lost

Original comment by [email protected] on 10 Dec 2009 at 9:52

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

I applied the changes in the original attachment to my GitHub fork[1] of this 
project.  The PATCH is attached (and I'd like to rap the author's knuckles with 
a hammer for not providing a patch AND for changing spaces to tabs).  Hopefully 
this is a little more useful.

[1]; https://github.com/blalor/pymacadmin

Original comment by [email protected] on 14 Mar 2011 at 2:55

Attachments:

• 0001-Adding-NSDistributedNotifications-support.patch
• 0002-Adding-sample-NSDistributed-events.patch

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

ouch!  Knuckles are a bit sore :( Sorry 'bout that, I think this was only my 
second FOSS contribution ever.

Original comment by [email protected] on 14 Mar 2011 at 4:16

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

I guess I forgot to be gracious about you doing the work in the first place.  
Sorry about that; it works very well! :-)  I extended it a bit by allowing an 
event key of "*" to be translated to None, which allows *all* events to be 
captured.  The dispatcher needs a little more work to provide access to the 
entire notification object, but that's trivial.

Original comment by [email protected] on 14 Mar 2011 at 5:04

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

no need to apologize :)  I'm glad it was merged into trunk and that other 
people have found it useful.

Original comment by [email protected] on 15 Mar 2011 at 9:45

[acdha]

From @GoogleCodeExporter on May 20, 2015 16:39

It's not been merged into the trunk, it's just in my GitHub fork.  But I did 
find it useful!

Original comment by [email protected] on 15 Mar 2011 at 10:16