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

Revert "ResolveAssemblyReference CPU optimizations (#8916)" #9037

Merged
merged 1 commit into from
Jul 14, 2023

Conversation

JaynieBai
Copy link
Member

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
This reverts commit 1ff019a.
Copy link
Member

@ladipro ladipro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far I know the problem is in 1ff019a#diff-20c3f77300d596d35264f7f9351652e233cb3455c3f7a7262df4842d437fe8efR1407-R1414 but let's revert for now to unblock insertions. Thank you!

@JaynieBai JaynieBai merged commit 4989625 into main Jul 14, 2023
@JaynieBai JaynieBai deleted the exp/revertPR8916 branch July 14, 2023 09:54
@ladipro
Copy link
Member

ladipro commented Jul 14, 2023

Uh-oh, the problem is that the task item implementing IMetadataContainer may be a transparent proxy and Linq enumerators cannot be serialized. Fix coming soon.

ladipro added a commit to ladipro/msbuild that referenced this pull request Jul 17, 2023
@ladipro
Copy link
Member

ladipro commented Jul 17, 2023

#9044 has the fix. I have tested it with an experimental VS insertion.

YuliiaKovalova pushed a commit to YuliiaKovalova/msbuild that referenced this pull request Jul 18, 2023
…tnet#9037)

This reverts commit 1ff019a.

Fixes #
VS Insertion, C++ Project System - PR: VC.ProjectSystem.VCProject.CachedProjectAutomation fails since these changes.
https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_git/VS/pullRequest/483186#1689247828

So far I know the problem is in 1ff019a#diff-20c3f77300d596d35264f7f9351652e233cb3455c3f7a7262df4842d437fe8efR1407-R1414 but let's revert for now to unblock insertions. Thank you!
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.

None yet

3 participants