-
Notifications
You must be signed in to change notification settings - Fork 790
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
Enable Net Sdk to write project properties #4430
Conversation
94424d3
to
37973e5
Compare
25a8969
to
d3c745f
Compare
</PropertyGroup> | ||
|
||
<Target Name="CoreGenerateAssemblyInfo" | ||
Condition="'$(Language)'=='F#'" |
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.
Update the condition to '$(Language)' == 'F#' AND '$(GenerateAssemblyInfo)' != 'false'
.
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 believe this is handled by the DotNET Sdk here: https://github.com/dotnet/sdk/blob/6945e3694c918eea4c8c4fb6217e1485b179994b/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.GenerateAssemblyInfo.targets
Note that dotnet/sdk#2021 was closed in favor of dotnet/sdk#2023, which appears (based on the description) to be the same fix. I mention this in case anyone follows the link from #3113 (which requires this bugfix in order to work) and feels concerned that the linked PR was closed. |
Thanks mate :-) |
Fixes: #3113
This requires a Net Sdk fix: dotnet/sdk#2021
Adds a new target file to handle the overrides. This was necessary, because the FSharp.NetSdk target files occurs before the NetSdk targets are defined.
Once the infra is done, the fix is trivial, just override CoreGenerateAssemblyInfo and run it for F# files only. Include the output in CompileBefore.
The remainder is just deployment nonsense.
Kevin
/cc @nguerrera, @enricosada, @TIHan @dsyme