Skip to content
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

Set emsdk path for freetype in emscripten builds #5857

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

ptosco
Copy link
Contributor

@ptosco ptosco commented Dec 10, 2022

Working on something completely different I realized what was wrong with libfreetype: emscripten compiles its own version of libfreetype and caches it within emsdk. Pointing cmake to the emsdk version of libfreetype solves all problems and it is possible to build again with the latest emsdk.
Since the latest emsdk is based on clang16 the -Wno-enum-constexpr-conversion flag is needed to build boost.

  • restore latest emsdk
  • set emsdk path for freetype
  • add compilation flag to enable building with latest clang compilers

- set emsdk path for freetype
- add compilation flag to enable building with latest clang compilers
Copy link
Member

@greglandrum greglandrum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks for identifying and fixing the problem!

@greglandrum greglandrum added bug infrastructure build infrastructure and the like labels Dec 12, 2022
@greglandrum greglandrum added this to the 2022_09_4 milestone Dec 12, 2022
@greglandrum greglandrum merged commit 161b65a into rdkit:master Dec 12, 2022
greglandrum pushed a commit that referenced this pull request Feb 23, 2023
- set emsdk path for freetype
- add compilation flag to enable building with latest clang compilers

Co-authored-by: Tosco, Paolo <paolo.tosco@novartis.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug infrastructure build infrastructure and the like
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants