-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Compilation fails with C++20 on emscripten's toolchain #4218
Comments
@Floris0106 i think it's fixed by #4187. but if you want to build with fmt 11.0.2, the patch might not apply cleanly. i'd suggest use the patch at https://src.fedoraproject.org/rpms/fmt/c/8da7a020a557972c246e4b8c927c3108c492e83c?branch=rawhide which contains a backport to 11.0.2. |
I don't know if I'm doing something wrong, but the fedora repo only gives me some source and patch files, both using FetchContent and when manually cloning. Anyway, I'll close this since it's fixed in 22701d5 (which is what I'm using now). |
@Floris0106 for applying a patch when using FetchContent_Declare(fmt
GIT_REPOSITORY https://github.com/fmtlib/fmt.git
GIT_TAG 11.0.2
PATCH_COMMAND git apply --ignore-whitespace "${CMAKE_CURRENT_LIST_DIR}/path-to-patch.patch"
DOWNLOAD_EXTRACT_TIMESTAMP true) |
Backport fmtlib#4187 to 11.0.2 See: fmtlib#4218
When compiling fmt using emscripten's CMake toolchain file (obtained through the emsdk) with the following errors:
I'm able to reproduce this issue with the latest versions of emsdk (3.1.70) and fmt (11.0.2). I used the following CMakePresets.json:
Here
EMSDK
is an environment variable that points to the root directory of the emsdk. CMakeLists.txt:I'm not sure whether this is an issue with fmt or emscripten, so I've made an issue on the emscripten repository as well. (see here)
The text was updated successfully, but these errors were encountered: