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

Snow leopard Compatibility Fixes #125

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Snow leopard Compatibility Fixes #125

wants to merge 9 commits into from

Conversation

bazz1tv
Copy link
Owner

@bazz1tv bazz1tv commented Jul 25, 2020

With these fixes, compiling SNES Tracker on Mac OSX 10.6.8 should be similar to the following model:
CC=clang++-mp-9.0 CPP=clang++-mp-9.0 OBJCC=clang++-mp-9.0 CPPFLAGS=-stdlib=libc++ OBJCFLAGS=-stdlib=libc++ make

Special Thanks to @KungFuFurby for figuring out the missing link on getting SNES Tracker to run on Snow Leopard!!

Helps with #32, and fixes fatal runtime issues on Snow Leopard.
Now compiling on Snow Leopard should be as simple as, for example:
CC=clang++-mp-9.0 CPP=clang++-mp-9.0 OBJCC=clang++-mp-9.0 CPPFLAGS=-stdlib=libc++ OBJCFLAGS=-stdlib=libc++ make
@bazz1tv
Copy link
Owner Author

bazz1tv commented Jul 25, 2020

I need this branch (SnowLeopard2) to be tested for good operation at least in a general sense from people not only on OSX Snow Leopard, but modern OSX, Windows, and Linux before I can merge it in with confidence.

@bazz1tv
Copy link
Owner Author

bazz1tv commented Jul 27, 2020

OSX 10.11

  • compiled fine
  • loaded and played a song in ST
  • loaded SPC, rar, and 7z files fine in STD
  • Seems fine

@KungFuFurby
Copy link

This fix is KungFuFurby approved! ...with a caveat.

I tested this with clang++ versions 3.7, 5.0, 8.0, and 9.0, all from MacPorts. Currently, 5.0 is the earliest version of the compiler that can be used with this program.

@KungFuFurby
Copy link

Tested with a manual merge of 0.1.1 (albeit without me committing it). Had to fix merge conflicts (usually related to my original fixes to the control-related warnings) and fix a few warnings related to control reaching the end of a non-void function, as without doing so I was crashing again.

iconutils is not compatible with my OS due to being distributed with a later version of XCode than is supported on my computer, so I temporarily disabled the related code for this occasion. A workaround exists, but for me, would require libicns, which I can get through MacPorts if I haven't already done so.

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 this pull request may close these issues.

2 participants