Skip to content
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

anotherInstanceStarted don't gets called on MacOS Sierra... #122

Closed
esurharun opened this issue Nov 4, 2016 · 2 comments
Closed

anotherInstanceStarted don't gets called on MacOS Sierra... #122

esurharun opened this issue Nov 4, 2016 · 2 comments

Comments

@esurharun
Copy link

I found a bug on MacOS Sierra x64 on JUCE 4.2.4,

Even though moreThanOneInstanceAllowed returns false, anotherInstanceStarted was getting called.

But on MacOS Sierra it does not work.

@esurharun
Copy link
Author

esurharun commented Nov 5, 2016

Ok. Fixed by updating JUCE sources...

You should change juce_mac_messagemanager.mm line 54 from

 [[NSDistributedNotificationCenter defaultCenter] addObserver: delegate
                                                                selector: @selector (broadcastMessageCallback:)
                                                                    name: getBroadcastEventName()
                                                                  object: nil];

to

[[NSDistributedNotificationCenter defaultCenter] addObserver: delegate
                                                            selector: @selector (broadcastMessageCallback:)
                                                                name: getBroadcastEventName()
                                                              object: nil
                                                        suspensionBehavior:NSNotificationSuspensionBehaviorDeliverImmediately];

otherwise it waits for and event on the other process and sets default to NSNotificationSuspensionBehaviorCoalesce.

@julianstorer
Copy link
Contributor

Thanks! That does seem to work, I'll sort it out. (And sorry - only just saw this report months after you posted it!)

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

No branches or pull requests

2 participants