phosit

An AI was roman.

Indeed, thanks for the ping. It should be fixed with rP27994.

After updating ICU I also had linkage errors. It was fixed by cleaning the workspace.

Looking at the linked ticket a future 2.12.3 should no longer have any issue, still technically it's a downstream bug, just that the fallout was unexpectedly large and the missing headers where injected back upstream. So I'd add the parser header regardless as this also allows the use of all 2.12.x releases. Right i used 2.12.1 Can you upload a patch? (I'm shy of patching libraries) To address the readability issue we could use a temporary/using. using MaybeConstXmlError = std::conditional_t<LIBXML_VERSION >= 21200, const xmlError, xmlError>; void errorHandler(void* UNUSED(userData), MaybeConstXmlError* error)

Thank you for the work. I don't like preprocessor directives. We could go for: void errorHandler(void* UNUSED(userData), std::conditional_t<LIBXML_VERSION >= 21200, const xmlError, xmlError>* error) Also that wouldn't introduce duplication. @hyperion Does Fcollada also have an issue with newer lxbxml2 version? I get this errors: FCollada/FUtils/FUXmlDocument.cpp: In constructor ‘FUXmlDocument::FUXmlDocument(FUFileManager*, const fchar*, bool)’: FCollada/FUtils/FUXmlDocument.cpp:39:39: error: ‘xmlParseMemory’ was not declared in this scope 39 | xmlDocument = xmlParseMemory((const char*) fileData, (int)fileLength); | ^~~~~~~~~~~~~~ FCollada/FUtils/FUXmlDocument.cpp: In constructor ‘FUXmlDocument::FUXmlDocument(const char*, size_t)’: FCollada/FUtils/FUXmlDocument.cpp:67:23: error: ‘xmlParseMemory’ was not declared in this scope 67 | xmlDocument = xmlParseMemory(data, (int)length); | ^~~~~~~~~~~~~~

Indeed. While writing that prompt it also got to my mind: In the winter there aren't that much less animals. Instead there is less gain on the fields.

In days of old, when winter's chill did reign, Meat was a luxury, scarce and plain. The frozen fields lay barren, void of life, Leaving hunger pangs to plague our strife. But fear not, for in those frozen lands, There were still ways to feed our hands. We turned to fields of grain and green, Where other forms of sustenance were seen. The wheat and oats we harvested with care, Baked into bread, our bellies could bear. And from the gardens, vegetables galore, Stews and soups we made, and more. In winter's grip, we found a way, To survive each day, come what may. For though meat was a rarity, Our resourcefulness kept us in unity. So let us cherish these fields of gold, That give us life, both young and old. For in their bounty, we find a way, To thrive, even in winter's gray.

Ok i reinstaled the git repo, but still don't get the xml error. If there is no objection i'll commit it as is.

No, I also use GCC 13.2.1 from the arch repo I'll reinstall the complete git repo this weekend to see if that yields more errors. (Probably ./clean-workspace.sh didn't clean everything)

Curiously i don't get an error about the xml-validator stuff. I made a patch about thous errors I get. @Riesi I added you as "Riesi" to the credits. Is this ok for you? Do you also want your name?

sure

I also use Linux. I didn't encounter this error untill you mentioned it. I don't get it all the times. Propably it's because of the precompiled header. We already use CI.

There is work done towards a alternating placement. But that isn't commited yet Can you upload the replay?

The OPEX cost would make the popcap a less hard limit. Which is good IMO.

I like how the cities integrate in to the terrain. In 0 A.D. the maps who have cities protected by cliffs are mainly acropolises. Which look unnaturally exposed.