-
Notifications
You must be signed in to change notification settings - Fork 17
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
Reproducible stdlib option #694
Comments
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
alandefreitas
added a commit
to alandefreitas/mrdocs
that referenced
this issue
Oct 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Although results are more reproducible now with libc++, when MrDocs runs CMake to generate the compilation database, compilers have different defaults for stdlib, causing inconsistencies in CI.
In particular, when MSVC is used to generate compile_commands.json, and then each command is converted to clang commands by the MrDocs database, but MSVC defaults to a much older standard than GCC and Clang for
stdlib
, which causes most of these inconsistencies.Thus, if no C++ standard is set in the command line for the compilation command, MrDocs compilation database should make it consistent by setting it to the same level as Clang instead of using MSVC's C++11.
The text was updated successfully, but these errors were encountered: