[Problem building FCollada]

Was the signature for xmlDocDumpFormatMemory changed at some point? On my system, its third parameter is declared as int* and changing XMLFix.cpp to intptr_t size = -1; causes this not to compile on my machine.

[Building on OS X]

Hi Arnaud,

The patch has since been checked into the Subversion repository, so you shouldn't need to apply it yourself.

I don't know if there are differences that will affect compiling on Tiger versus Leopard, but you can always post any errors you get here, and we can try to help figure them out.

[Building on OS X]

Thanks olsner, I am glad to see the changes made it in.

• fcollada: pragmas (I didn't see any issues with this when compiling on my mac - was this only warnings?)
  
• fcollada: use of snprintf (should be fixed, especially if long is more than 32 bits, but didn't cause any compile errors)
  

Both of these only caused warnings. In the case of snprintf, I don't know why it issued a warning (long is 32 bits on both 32-bit and 64-bit Macs, as far as I can tell), but changing from %ul to %u eliminates the warning.

• wxWidgets regexp stuff: I assume that passing the "advanced" argument enables some regexp extensions - but do these regexpes really mean the same thing if we use default rather than advanced regexpes? If so, we should change them to actually use the default-flag for everyone, else we should probably rewrite the regexpes to only use "default"-features.
  

I had thought both the affected regexps were extended, but on reading the docs, it looks like the \d character class has to be written as [[:digit:]] instead. Other than that, it looks like it's okay, but I suppose they should be tested to make sure they still work.

• atlas/ScenarioEditor: Apple-ifdeffed _UINTxx defines. I didn't see any related compile errors on my mac, but if it's a problem with another version of something (os x, for instance) it looks like we would like to find a cleaner solution for the problem anyway.
  

I ran into a conflict between SpiderMonkey (jsotypes.h) and Security.framework (cssmconfig.h), which are both included by ScenarioEditor.cpp, because they both provide typedefs for uint32 and the like. This might be specific to the version of SpiderMonkey (I've got the MacPorts 1.7 version).

Thanks again for going through that change set, it's great to see such a quick response.

[Building on OS X]

I tried my best to avoid patching anything outside the 0 A.D. source code, since I don't have control over those and it increases the complexity for a new person to get the source working.

There are a few changes in there to reduce warnings from the compiler, and I use the wxWidgets included with Leopard, which doesn't have wxRE_ADVANCED, so I had to include a patch for that sa well.

[Building on OS X]

I just got my first complete build done, and finally have the game running on OS X. It took a while, figuring out dependencies and correcting issues, so I thought I'd document what I did, and invite others to share their own workarounds. These changes were applied to revision 7008.

The first thing I did was use MacPorts to install the following packages: libsdl, boost, spidermonkey, libvorbis, jpeg, libpng, libdevil, libxml2, i.e.,

sudo port install libsdl, boost, spidermonkey, libvorbis, jpeg, libpng, libdevil, libxml2

Next, I downloaded, built, and installed the stable 1.2 release of enet.

Then, I checked out, built, and installed valgrind. The 3.4.1 release does not work, you must get the repository version.

Now to the changes to the source...

I deleted the directory libraries/fcollada/include/LibXML because it overrides my libxml headers (my drive is formatted case insensitive, so LibXML matches libxml). There might be a better way to work around this, but it's only there for Windows users anyway.

I deleted the two library files in libraries/fcollada/lib, called libFColladaSD.a and libFColladaSR.a, because these are files that need to be rebuilt.

Then I these changes (pastebin): http://pastebin.ca/1498286

[Problem building FCollada]

Philip checked in a patch (rev 6991) that should let you compile libraries/fcollada/src.