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
Dev remove roslyn nupkg ref #30
Conversation
</NuGetContent> | ||
<NuGetContent Include="build\net46\*"> | ||
<Destination>build\net46</Destination> | ||
</NuGetContent> | ||
<NuGetContent Include="tools\*.ps1"> | ||
<Destination>tools\net45</Destination> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason that this is the same as net45?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
anything under net45 folder will apply to project targeting 4.5+, if there is no other netxx folder in nupkg. The install.ps1 works in the same way no matter of the targeting framework.
<Destination>build\net45</Destination> | ||
</NuGetContent> | ||
<NuGetContent Include="build\net46\*"> | ||
<Destination>build\net46</Destination> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am assuming this the latest you are referring later? So this should cover anything after 4.6, correct? If so, do you want to use latest here as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
net46 folder name has special meaning in Nupkg. Check this.
Remove the dependency of Microsoft.Net.Compilers nupkg, instead embed the Roslyn bits inside CodeDomProvider nupkg. This will ensure VS always use the Roslyn compiler shipped with msbuild. Developer can also specify a path through the following msbuild property from which CodeDomProvider copies the Roslyn compilers to bin\roslyn folder.
[some path]