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 "buildDotnetModule: enable RestoreUseStaticGraphEvaluation" #162476

Merged
merged 1 commit into from
Mar 2, 2022

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Mar 2, 2022

This reverts commit 0b1856b.

This broke one of our clients projects where a local libary could no
longer be found. Since there is no easy way to disable it's better
if this flag is set per project using dotnetRestoreFlags.

This reverts commit 0b1856b.

This broke one of our clients projects where a local libary could no
longer be found. Since there is no easy way to disable it's better
if this flag is set per project using `dotnetRestoreFlags`.
@IvarWithoutBones
Copy link
Member

IvarWithoutBones commented Mar 2, 2022

I think this has gotten fixed with dotnet/msbuild#7361, in my opinion we should re-enable the flag once we confirm it fixes these problems and the pull hits a tagged release. In the meantime I'll merge this, thanks for the fix!

It's also planned to enable the flag by default, see NuGet/Home#9803. Progress on that looks a bit stale, but seeing how there aren't any open issues left I imagine we won't find more regressions from it.

@IvarWithoutBones IvarWithoutBones merged commit f90f533 into NixOS:master Mar 2, 2022
@Mic92 Mic92 deleted the dotnet-fix branch March 4, 2022 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants