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

[smart_holder] Another super simple experiment trying to get to the bottom of the type_caster_odr_guard test failures with windows-2022 #4914

Closed
wants to merge 1 commit into from

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Nov 1, 2023

Description

Starting point: #4850 (comment)

Conclusion: #4918

Suggested changelog entry:

@rwgk
Copy link
Collaborator Author

rwgk commented Nov 2, 2023

Suspicion, based on one datapoint:

The type_caster_odr_guard failures are related to the change from

C:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.35.32215/bin/HostX64/x64/cl.exe

to

C:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.37.32822/bin/Hostx64/x64/cl.exe

@rwgk
Copy link
Collaborator Author

rwgk commented Nov 3, 2023

Problem solved with PR #4918.

@rwgk rwgk closed this Nov 3, 2023
@rwgk rwgk deleted the sh_with_noop_change branch November 3, 2023 06:05
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.

1 participant