-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[0.68] MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early #9609
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
Conversation
…r to avoid being evaluated too early (microsoft#9562) * MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early * Change files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to backport this to 0.66 and 0.67 as well? We're suddenly seeing this issue on CI as well, and we're on 0.66. |
we shouldn't see this on older releases if we are using our own VS image (using vs2019), this should only repro with vs2022. Do you have a link to a failure on vs2019 @tido64 ? |
You're right, it looks like I guess a workaround is to explicitly set |
@tido64 yes, we don't "officially" support vs2022 yet since the project can't yet build with the default toolset that ships in 2022 (v143). One can work around it by installing the v142 toolset in vs2022 (in fact that's what I have on my laptop since 2022 is faster). We have workitems to move to v143 but it will take some time (and requires Office to be able to move too...). Ideally, you should have CI run the currently-supported config, which is windows-2019. |
…arameter to avoid being evaluated too early (microsoft#9609) * MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early (microsoft#9562) * MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early * Change files * Change files Co-authored-by: Alexander Sklar <asklar@microsoft.com>
…r to avoid being evaluated too early (microsoft#9609)
…arameter to avoid being evaluated too early (#9749) * [0.68] MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early (#9609) * MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early (#9562) * MustBeNoExceptVoidFunctor should depend on the template type parameter to avoid being evaluated too early * Change files * Change files Co-authored-by: Alexander Sklar <asklar@microsoft.com> * Change files Co-authored-by: Julio César Rocha <JunielKatarn@users.noreply.github.com>
Backport of #9562
Microsoft Reviewers: Open in CodeFlow