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

Issue with High Sierra #17

Closed
andydotxyz opened this issue May 14, 2019 · 1 comment · Fixed by #18
Closed

Issue with High Sierra #17

andydotxyz opened this issue May 14, 2019 · 1 comment · Fixed by #18

Comments

@andydotxyz
Copy link
Contributor

Since merging in #10 the app has been working but today I tested on mac OS High Sierra and saw a lot of

2019-05-14 23:30:17.338 main[50763:1244727] pid(50763)/euid(501) is calling TIS/TSM in non-main thread environment, ERROR : This is NOT allowed. Please call TIS/TSM in main thread!!!

and the app never started.
It seems like the "core.Run()" is required to be called from the main thread on mac OS that is not required on Linux.

I cannot seem to get to the bottom of this so thought I would log this issue instead.
I suspect that there are some OS input handling baked into the core package that I did not find when extracting the last of the graphics code to the "driver" package.

@andydotxyz
Copy link
Contributor Author

It seems that macOS requires that the speaker run on the main thread. The proposed pull request fixes this and should resolve the issues for this gui driver. It unfortunately creates new problems for a Fyne based driver as the render thread blocks. I will try to learn how a speaker loop and a graphics loop could be interpolated...

@HFO4 HFO4 closed this as completed in #18 May 15, 2019
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 a pull request may close this issue.

1 participant