-
Notifications
You must be signed in to change notification settings - Fork 16
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
Unable to build on OS X Mojave 10.14.4 #1
Comments
Hey, I think this might (hopefully) be an easy fix. If not: Traditionally (until a very recent go version) all go-projects were located inside a unified location (wherever the $GOPATH points). So try:
Then copy/move the project folder inside $HOME/go and try the If you like, read up on it here: https://golang.org/doc/install and here for GOPATH: https://github.com/golang/go/wiki/SettingGOPATH Good luck and please report back :) |
Hi Maurice, thanks! Unfortunately it's not working. Before, I tried building from
Also with manually setting |
The issue is with the usage of |
Oh, right, thanks, I must have overlooked that... I'll have a look, If I can create a branch without the multisampling for OSX users maybe later today. |
Disabling multisampling (so not using
|
Yes, glMemoryBarrier seems to be OpenGL 4.2 according to the docs. I guess we don't really need it anyway (was just to be sure and avoid possible weird frames). Just remove line 475 in glView.go and give it another try. |
No still the seem issue. Something is going wrong with the ui lib. |
It has something to do with not running ui and gl commands on the main thread but instead in a goroutine. |
Did the linking work when building? It looks like the the ui crashes when trying to first call the C function over ffi. The gl commands should run on the main thread here (at least my ones and all GLFW event handlings). The ui is indeed running inside a go-routine. But was never an issue (at least on Linux and Windows not) |
The ui also needs to run in the main thread. When I do this it works by putting I think to make this work requires quite a rewrite of your main loops. |
... That's really annoying. I'm sorry it seems to be that complicated to get it working (Couldn't really test it with OSX). If you can think about a good design for the both windows (ui + glfw), I am open to suggestions/discussions or pull-requests. Preferably a short discussion before you spend too much work on a pull-request (if you like to help here -- no hard feelings if not). |
I'm afraid I don't have the time to work on this. |
I'm happy to test any new releases on OS X, but my Go knowledge is nonexistent unfortunately 😄 |
Alright, let's keep this open, maybe I'll find some time or have an idea to remodel the main loop and keep both windows in the main thread! |
This might just be me being unexperienced with Go, but I'm having trouble building this on an almost clean machine.
git clone https://github.com/MauriceGit/Voronoi_Image_Manipulation
(into/tmp/Voronoi_Image_Manipulation
)go get ./...
but got an error: go get: no install location for directory /tmp/Voronoi_Image_Manipulation outside GOPATH)ld: warning: object file (/var/folders/96/r2y8ffxs7zz8k9lynb5v_fbh0000gn/T/go-link-704353261/000057.o) was built for newer OSX version (10.14) than being linked (10.8)
Any help would be appreciated. Apologies if I'm overseeing something totally obvious 😄
The text was updated successfully, but these errors were encountered: