Skip to content

Update for OS X Mavericks #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 16, 2014
Merged

Update for OS X Mavericks #4

merged 1 commit into from
Apr 16, 2014

Conversation

rgov
Copy link

@rgov rgov commented Apr 15, 2014

I updated the event handler script to support Messages on OS X Mavericks. The changes include:

  • Change from binary-format .scpt files to plain-text .applescript files
  • Rename the AppleScripts to follow more typical naming conventions
  • Rewrite installation instructions according to new Messages UI
  • Add unimplemented functions to the event handler script to avoid errors

@lazerwalker
Copy link
Owner

Awesome, thanks!

Do you know if these changes will still allow it to work on Mountain Lion, or are the plaintext .applescript files Mavericks-only?

(I think Mavericks-only is fine, but if that's the case I'll add a note about that in the README and perhaps tag a previous SHA as having legacy support)

@rgov
Copy link
Author

rgov commented Apr 15, 2014

I don't believe that .applescript files are new to Mavericks, but the Messages event handler system might be different (such as now using a single script file, instead of many).

@lazerwalker
Copy link
Owner

Alright. I don't have a good way to test for earlier versions of compatibility, so I might just make a note anyway, since that's pretty low-impact.

Thanks again for doing this! Clearly I haven't been using this lib lately, so it's greatly welcomed to see someone else help whip it into shape :).

lazerwalker added a commit that referenced this pull request Apr 16, 2014
Update for OS X Mavericks
@lazerwalker lazerwalker merged commit 1c1e713 into lazerwalker:master Apr 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants