-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replacing config file with cmake for freetype v2.10.4 port update #14228
Conversation
Freetype doesn't compile using emconfigure and emmake. I rebuilt the configuration using cmake instead based on issue 4 from the freetype port.
If possible I would rather not increase our dependency on external tools like Is there any fundamental reason we can't keep the old method? |
By the way the existing method doesn't use |
When I updated this file to test the building v2.10.4 using the old method, there were 40k lines worth of compiler errors, mostly duplicates spread across files. One example file that covers most of them is below. There's some discussion about cmake in the port's issues: /local/emsdk/upstream/emscripten/cache/ports-builds/freetype/src/psaux/psobjs.c:347:27: error: use of undeclared identifier 'FT_ERR_PREFIXInvalid_File_Format'; did you mean 'FT_Err_Invalid_File_Format'? |
I'm think you will need to figure out where those error are coming from. Switching to relying on cmake + the actual build tool (make on UNIX and mingw-make on windows) is probably not direction we want to go in here.. at least not lightly. |
I still need to un-cmake this.
This issue has been automatically marked as stale because there has been no activity in the past year. It will be closed automatically if no further activity occurs in the next 30 days. Feel free to re-open at any time if this issue is still relevant. |
Freetype 2.10.4 doesn't compile using emconfigure and emmake. I rebuilt the configuration using cmake instead based on issue 4 from the freetype port.
This needs to get edited and merged after the matching emscripten-ports/freetype PR gets merged.