Skip to content

(GH-598) Allow users to disable or override target framework path#607

Merged
gep13 merged 2 commits intocake-contrib:developfrom
AdmiringWorm:feature/gh-598
Aug 4, 2020
Merged

(GH-598) Allow users to disable or override target framework path#607
gep13 merged 2 commits intocake-contrib:developfrom
AdmiringWorm:feature/gh-598

Conversation

@AdmiringWorm
Copy link
Member

In some cases, there is no need to use the TargetFrameworkOverridePath when building .NET Full Framework using .NET Core. This is especially true if the user already references the needed reference assemblies provided by Microsoft (which will also be the default in .NET Core 5.x).

In other cases, we may resolve the wrong path to the mono library directory (especialy in cases where a custom directory is used), so allowing users to override the directory used is also useful.

resolves #598

…meworkOverridePath

In some cases, there is no need to use the TargetFrameworkOverridePath
when building .NET Full Framework using .NET Core.
This is especially true if the user already references the needed
reference assemblies provided by Microsoft (which will also
be the default in .NET Core 5.x).
@AdmiringWorm AdmiringWorm requested a review from gep13 August 4, 2020 18:53
Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gep13 gep13 merged commit 1d3303f into cake-contrib:develop Aug 4, 2020
@gep13
Copy link
Member

gep13 commented Aug 4, 2020

@AdmiringWorm your changes have been merged, thanks for your contribution 👍

@AdmiringWorm AdmiringWorm deleted the feature/gh-598 branch August 4, 2020 19:30
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.

Allow disabling or overriding the framework path override

2 participants