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

Ignore warnings related to deprecated stdext::checked_array_iterator on MSVC #3895

Merged
merged 2 commits into from
Aug 31, 2024

Conversation

nickbianco
Copy link
Member

@nickbianco nickbianco commented Aug 28, 2024

Fixes issue with the Windows 2022 GitHub Actions runner failing because MSVC deprecated stdext::checked_array_iterator, which was used by an older version of fmtlib (dependency of spdlog).

Brief summary of changes

Updated spdlog to v1.14.1.

Update: rather than try to update spdlog, I've changed this PR to ignore the warning on MSVC (per @adamkewley's suggestion).

Testing I've completed

Running the CI suite.

Looking for feedback on...

Should the added target_compile_options be moved outside of the macro (i.e., used only for libraries compiled with spdlog)?

CHANGELOG.md (choose one)

  • no need to update because...CI fix.

This change is Reviewable

@adamkewley
Copy link
Contributor

It's going to be rough to actually rollout the latest spdlog, because it requires going through every printable type (incl. those from simbody) and annotating the relevant datastructures to say "ostream this".

OpenSim Creator's been getting this warning for a couple of years now, because it's compiling with /W4, the way to ignore it is by disabling the warning:

Not exactly a great solution but if you want CI back up-and-running then I'd maybe consider first disabling those warnings and then separately try to sort out the spdlog upgrade, so that you aren't getting pressure from PRs etc. to fix CI

@nickbianco
Copy link
Member Author

Thanks @adamkewley, I think patching the issue by disabling warnings is better for now. I'll switch this PR to do that instead.

@nickbianco nickbianco changed the title Update spdlog to v1.14.1 Ignore warnings related to deprecated stdext::checked_array_iterator on MSVC Aug 30, 2024
@nickbianco nickbianco requested a review from adamkewley August 30, 2024 22:28
@adamkewley
Copy link
Contributor

adamkewley commented Aug 31, 2024

Seems to do the trick! - sucks to ignore warnings but it gets the project's wheels turning while evaluating what to do (imo, I'd de-integrate spdlog+fmt around the time OpenSim upgrades to C++20 on target platforms that include std::format - drops a dependency but means you can still use the "{}" syntax

@adamkewley adamkewley merged commit 7466c6d into main Aug 31, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants