MacOS error reappearing again

[reubenharry]

Running on a Mac (recently updated), when I run the rhine-bayes executable via cabal run rhine-bayes-gloss I get

2024-01-27 18:24:27.834 rhine-bayes-gloss[13639:2049511] GLUT Fatal Error: internal error: NSInternalInconsistencyException, reason: NSWindow drag regions should only be invalidated on the Main Thread!

(after selecting any of the 3 options in the list).

Curiously, I wasn't having this problem earlier today, and am on the master branch, and did cabal clean. Any ideas? (I'm on ghc 9.4.8)

(I've previously been able to resolve this issue, e.g. in my rhine-bayes-examples repo, but I can't presently remember how).

[reubenharry]

OK, closing this issue, as I remembered the fix, which is: use flowGlossIO and removed -threaded in the compilation options.

[turion]

Ah, great you remembered! Shouldn't we change the code then?

[turion]

Using flowGlossIO shouldn't have an effect, it's defined as launchInGlossThread settings . flow. But I guess -threaded makes a huge effect. Maybe it's possible to conditionally disable -threaded on mac?

[reubenharry]

Hmm interested. Just removing -threaded was not sufficient (although certainly necessary), so then it's unclear what I did to make it work...

[turion]

This is tricky: When you change the GHC options in the cabal files, it doesn't necessarily recompile (although it morally should). Can you maybe add a line of comment, recompile, and check whether changing -threaded maybe in fact is the relevant change?

[reubenharry]

I did cabal clean first. Would that cause recompilation?

[turion]

I would have thought it should. This is confusing. Can you try the following:

1. Go to master
2. cabal clean or git clean
3. cabal run rhine-bayes-gloss (this should throw the error)
4. Remove the flag
5. cabal run rhine-bayes-gloss (this should still throw the error, right?)
6. Use flowGlossIO directly (now the error is gone?)
7. Remove flowGlossIO again (now the error returns?)

If the last steps are as predicted, then using flowGlossIO is indeed necessary.

[reubenharry]

I stand corrected! After 4, the bug is resolved. Sorry for the confusion.

[reubenharry]

So yeah, maybe we can disable -threaded conditionally on mac.

[turion]

Fantastic, thanks!