-
Notifications
You must be signed in to change notification settings - Fork 53
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
Allow disabling or overriding the framework path override #598
Comments
@AdmiringWorm this sounds like a good idea to me. Is this something that you wanted to take a stab at? |
Yeah, most likely, either during the weekend or sometime next week. |
…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).
…argetFrameworkOverridePath
(GH-598) Allow users to disable or override target framework path
A null exception happens when running on windows, under cake .net core and mono not being installed. This commit fixes that issues to only set shouldUseFrameworkPath to true when running on Unix
(GH-598) Fix null exception happening on windows
When running a .NET Core build on Unix system we always add a framework path override.
In some cases, this may be undesirable (like if the projects have the .NET Framework assembly references).
I would suggest allowing this ability to be disabled, and optionally be allowed to pass in a different path to use instead (would be required when cake.recipe support .NET Core only builds).
The text was updated successfully, but these errors were encountered: