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

Fix solution restore in libraries #38294

Merged
merged 1 commit into from
Jun 23, 2020
Merged

Conversation

ViktorHofer
Copy link
Member

Fixes #37358

VS doesn't honor the RestoreUseStaticGraphEvaluation set in a Directory.Build.props import as that file isn't imported in a solution. Specifying the property in a Directory.Solution.props file which is imported in a solution uses static graph restore and fixes the restore issue.

@ghost
Copy link

ghost commented Jun 23, 2020

Tagging subscribers to this area: @safern, @ViktorHofer
Notify danmosemsft if you want to be subscribed.

@ViktorHofer
Copy link
Member Author

Thanks for reporting @Anipik 👍

Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

Interesting didn’t know this file existed.

@ViktorHofer
Copy link
Member Author

dotnet/msbuild#4784

@safern
Copy link
Member

safern commented Jun 23, 2020

I see, thanks. Would it make sense to import Directory.Build.props from it? Or no need to do that.

@ViktorHofer
Copy link
Member Author

Probably not as the reason why msbuild introduced the Directory.Solution.* extension point was because importing Directory.Solution.* would have been a breaking change. No need for a solution to get the full msbuild context.

@safern
Copy link
Member

safern commented Jun 23, 2020

Makes sense. Just wondering what we needed to import and share on both kind of environments.

@ViktorHofer ViktorHofer merged commit 69fca92 into master Jun 23, 2020
@ViktorHofer ViktorHofer deleted the ViktorHofer-solutionrestore branch June 23, 2020 20:35
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Restore task failing on our solutions
4 participants