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
namecollision on macOS #77
Comments
It's the same name as for Teeworlds. Any reason to get this name collision in this case ? |
After fixing this compilation went fine. I renamed pi to tw_pi to be sure there are no further collisions, if you want I can create a Pullrequest with this fix. Here is the build-log after fixing it: As you see it got quite big because of some other warnings. For some reason I can not see the cursor in teeuniverse_editor. If there are some other things you want me to test tell me. Edit: |
Nice :) Happy to see that Harfbuzz is working. I will fix all errors and warnings. Your compiler seems more sensitive, that's nice. |
Yes, the cursor disappears in both cases, about the compiler: clang is default on macOS, you might consider also installing it on your system (and if you got a little too much time you could also try their static analyzer: scan-build), so you get most of its warnings too. Installing all the dependencies was actually pretty easy as they all are available via https://www.macports.org/ |
Can you update the Readme with instructions for OSX ? |
My last commit should fix all warnings and the "pi" thing |
Nice, it indeed fixed most of the warnings:
|
Building on macOS is pretty easy actually. First you need to install macports: https://www.macports.org/install.php And then you can install all required dependencies like this: |
What a shame. I will take care of those errors and enable warning in g++. |
Solution: Find a better name for pi that is not pi.
The text was updated successfully, but these errors were encountered: