-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
ArrayMarshaller.cs file doesn't get included when a project sets DisableImplicitAssemblyReferences to false but depends on System.Memory #67425
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
Affects
And DirectoryServices* projects |
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries Issue DetailsRepro:
Error:
Workaround: Set
|
@am11 unsure if that's actually infra libraries, depends on who owns the generators.targets file. I think area-Infrastructure would be better in this case :) |
Tagging subscribers to this area: @dotnet/runtime-infrastructure Issue DetailsRepro:
Error:
Workaround: Set
|
@ViktorHofer, my thinking was if it is related to infra && src/libraries, area-Infrastructure-libraries would be more accurate; but I will defer to your assessment. 😉 |
Given that this will go away after #66623 is approved and ArrayMarshaller is approved, I'm not inclined to fix this now if it's not blocking. |
Works for me, I can work around it for now. |
Fixed by #68173 |
Repro:
<DisableImplicitAssemblyReferences>false</DisableImplicitAssemblyReferences>
to the Odbc.csproj<Reference Include="..." />
statements at the bottom of the project file..\dotnet.cmd build src\libraries\System.Data.Odbc\src\System.Data.Odbc.csproj /bl -f net7.0-windows
Error:
Workaround: Set
EnableLibraryImportGenerator
to true.cc @jkoritzinsky
The text was updated successfully, but these errors were encountered: