flycast icon indicating copy to clipboard operation
flycast copied to clipboard

Add `USE_HOST_X` options

Open wongandj opened this issue 11 months ago • 3 comments

wongandj avatar Feb 12 '25 02:02 wongandj

I don't understand what is the point of these changes: vulkan-headers, xxhash, glm and vulkan-memory_allocator are headers-only libraries. So using the host/OS version will not reduce resource usage, will not decrease the executable size and won't allow for easy security updates. It's only adding complexity to the build process and increasing the likelihood of a build failure. Google-test is only used when building and running tests, and isn't part of the final flycast executable. So again, using the host/OS version won't make it easier to package the app.

flyinghead avatar Feb 13 '25 10:02 flyinghead

TBH, I just poked around and did the ones that were trivial. I’ll do more significant ones soon.

wongandj avatar Feb 13 '25 15:02 wongandj

Please limit the changes to what is strictly necessary. The build process is already far too complicated for my taste.

flyinghead avatar Feb 13 '25 17:02 flyinghead