Make 0ad build with musl libc.
Compiles on Alpine linux.
I have seen the linked patch, but I have not seen any change in it related the isnanf issue. nvtt git HEAD still uses isnanf() on linux, so upgrading nvtt will not be enough IMO.
Like Vlad said, I have just uploaded a NVTT upgrade, so it would be better if you'd test the branch (and I would happily include your patch in my upgrade if it's still needed).
It is available at https://github.com/na-Itms/0ad/tree/nvtt. Please let me know if you have trouble and thanks for your interest!
Whoops we commented at the same time. I will include this patch in the NVTT upgrade then.
I have tried your nvtt branch, it fails:
"nvmath.h:193:16: error: 'isnanf' was not declared in this scope; did you mean 'isNan'?"
Anyway, You are right, I will try to get it upstream. Thanks for the heads up!
@Stan: Yes, it does, but I had to add this patch too from pull-request linked above: https://github.com/castano/nvidia-texture-tools/pull/294/commits/34c85d642d1478c7ad008eb3af66eca0a19bc202
Thanks for your patch, I have included the first part (with the other one you needed) in our upgraded NVTT.
For cppformat, we must upgrade that library (see #3190 and #4148). Your patch doesn't apply there so I'm going to reference this diff in the ticket and I hope to perform that upgrade soon. Is it OK with you if I abandon this diff, since we are not going to include it as-is?
Thanks again for the work.
I genuinely apologize. I used friend because I wanted to express that we were competitors without hard feelings, so I tried to use a friendlier term, which was apparently interpreted as sarcasm. My hate of sarcasm is at the heart of the issue so I am really sorry for my poor choice of terms.
I think "competitor" is too harsh, "former friend/colleague" too sad, so I'll just use "leper" next time and stop worrying about double meanings. Sorry again.