raduprv / Eternal-Lands

http://www.eternal-lands.com
Other
157 stars 57 forks source link

Xcode project update #152

Closed bendoughty closed 2 years ago

bendoughty commented 2 years ago

Here's an updated Xcode project file which enables the new SSL features for macOS users. I also took the opportunity to do a bit of housekeeping and make the project easier to navigate, since there has been some interest lately.

I anticipate another small change will be required prior to the next release (if only to bump the version number), but for now this seems to be building and running great on both newer (ARM64) and older (x86_64) Macs.

gvissers commented 2 years ago

Merged, thanks.

Quick question though: I see you have C++ standard set to C++17. So far, we have made a deliberate effort to stick to C++11, is there any reason the XCode project sets C++17 (and if so, should we fix it? :smile: )

bendoughty commented 2 years ago

No, no reason (certainly not an intentional one on my part, at least). It looks like that happened this morning after I updated to the latest Xcode beta (to test macOS Monterey support) and found it spitting out various warnings about new things Apple doesn't like this year. Would you prefer for this to be reverted, or does it not matter so much?

gvissers commented 2 years ago

Nah, it's no big deal, I was just wondering if it was a conscious decision, and whether we should act upon it. Just leave it as is, if that is what works best for you.