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

Set EmbedUntrackedSources=false to prevent generated WPF files from being gathered during SourceLink #4123

Merged
merged 1 commit into from
Oct 16, 2019

Conversation

vatsan-madhavan
Copy link
Member

@vatsan-madhavan vatsan-madhavan commented Oct 15, 2019

Set EmbedUntrackedSources=false in Workarounds.props since Microsoft.SourceLink.Common.targets gathers these files for embedding in _SetEmbeddedFilesFromSourceControlManagerUntrackedFiles otherwise

/cc @rladuca
/cc @chcosta, @tmat

…ets gethers these files for embedding in _SetEmbeddedFilesFromSourceControlManagerUntrackedFiles otherwise
@vatsan-madhavan
Copy link
Member Author

Note: We found this by investigating the following failure:

https://github.com/dotnet/wpf/pull/2051/checks?check_run_id=261380012

@tmat
Copy link
Member

tmat commented Oct 16, 2019

You meant to write EmbedUntrackedSources=false, not true, correct?

@vatsan-madhavan
Copy link
Member Author

You meant to write EmbedUntrackedSources=false, not true, correct?

🤦‍♂

@vatsan-madhavan vatsan-madhavan changed the title Set EmbedUntrackedSources=true to prevent generated WPF files from being gathered during SourceLink Set EmbedUntrackedSources=false to prevent generated WPF files from being gathered during SourceLink Oct 16, 2019
@vatsan-madhavan vatsan-madhavan merged commit cbfa29d into dotnet:master Oct 16, 2019
@vatsan-madhavan vatsan-madhavan deleted the fixup-wpftmp-workaround branch October 16, 2019 04:33
@tmat
Copy link
Member

tmat commented Nov 3, 2019

@vatsan-madhavan This workaround is masking a product bug.

The root cause is dotnet/wpf#1718 - WPF generates incorrect relative paths in #pragma checksum and #line directives in .g.cs files when BaseIntermediateOutputPath is not under the containing directory of the project file.

@vatsan-madhavan
Copy link
Member Author

You’re right.

dotnet/wpf#1718 may not be addressed before .NET 5, so this workaround is a good thing to have for the time being.

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.

2 participants