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 #490 by using Directory.Build.props #496

Merged
merged 1 commit into from
Mar 31, 2017

Conversation

pharring
Copy link
Member

@pharring pharring commented Mar 30, 2017

Fixes #490
Rename Global.props to Directory.Build.props so it will automatically be included in every project.
Thanks to @karolz-ms for the tip.

You'll notice there's now a new workaround for what appears to be a gitlink issue in common.props. Gitlink builds using MSBuild version 4.0 to parse the solution and, if you don't have MSBuild 4.0 installed, then it can't find Microsoft.Common.props. I'll investigate separately whether using a newer version of gitlink will solve that.

Rename Global.props to Directory.Build.props so it will automatically be included in every project.
@msftclas
Copy link

@pharring,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@SergeyKanzhelev SergeyKanzhelev merged commit acd9afd into microsoft:2.4.0-beta1 Mar 31, 2017
@pharring pharring deleted the Fix490 branch March 31, 2017 16:18
TimothyMothra pushed a commit that referenced this pull request Oct 25, 2019
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.

3 participants