Compiler Support
Just a heads up: sometime in the near future we will be dropping support for VS2010 and by extension building on Windows XP.
|
Assuming this is due to C++11 requirements it might help people to know the minimum version of gcc required as well.
|
GCC 4.6 is the minimum. (Currently)
Clang, unsupported, although it might work with Perl 5.20+ or not building with Perl support ... EDIT: Code:
buildclang % ls bin |
It's unfortunate Perl didn't compile with clang for so long, will be a while before 5.20 filters down =/
|
VS2012 supports targeting Windows XP afaik.
|
Quote:
|
I've managed to get it compiled with clang and perl.
There are a lot of patches that need to be in place for it to actually run though. There's a difference in the way gcc and clang process function arguments(basically they are reversed, left to right vs right to left). For example the line is fine with GCC, but creates a segfault when compiled with clang: Code:
if (RunQuery(query, MakeAnyLenString(&query, "SELECT count(slot) from aa_effects where aaid=%i", skill_id), errbuf, &result)) { Code:
uint32 len_query = MakeAnyLenString(&query, "SELECT count(slot) from aa_effects where aaid=%i", skill_id); As far as what I did to get Perl working correctly, I don't remember at this point, but I remember fighting with it, so I'm pretty sure I have that worked out. It's just been a while since I got it all working last. Might be just a simple change to the cmake files. I can share my work if there is any interest. |
Thanks for bringing that to my attention, we're slowly phasing out that style of query anyway at least.
Fixing perl appears to be needing to either update to a newer version or actually modifying the perl headers in some key spots to add spaces between identifiers. |
Thank heavens for Lua!
|
I'm glad to hear those are going away... There are a lot of those queries I had to track down.
This is my fix for perl on FreeBSD. I'm pretty sure that's all I needed. I know I didn't change the headers, and I couldn't seem to find a version that it worked without that. Code:
IF(UNIX) |
Aye the fix is recent (like 3 months ago recent) so it's unlikely to be in many repos.
Thanks for the flag fix, I'll be sure we get that in there. |
All times are GMT -4. The time now is 03:12 AM. |
Powered by vBulletin®, Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.