-
Notifications
You must be signed in to change notification settings - Fork 133
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
EventKeywords enum in checked-in netstandard.dll/2.0.3 IL source is missing long base type #4328
Comments
@mthalman this isn't related to GenAPI but to the checked-in IL source in SBRP. Here's the source code in netstandard.dll/2.0.3 for the When you look at the checked-in IL source in SBRP, you can see that the enum is missing the long base type: https://github.com/dotnet/source-build-reference-packages/blob/main/src/targetPacks/ILsrc/netstandard.library/2.0.3/build/netstandard2.0/ref/netstandard.il -> L333394 Would you know how that IL source got generated? ILDasm? Moving this issue to source-build. |
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. |
It was generated by some internal tooling located here(Microsoft internal link) The tooling is ildasm based. The current version of the tooling may be newer than this was originally generated from. I would first try regenerating to see if the problem is resolved. Also make sure the latest ildasm is used as noted in the readme |
FWIW, I regenerated netstandard.library 2.0.3 with the 9.0.0 IlDasm and the resulting IL didn't change. Possibly another reason to invest in using GenAPI on the targeting packs. |
When running GenAPI on NuGet.Common.6.8.1, it produces the following snippet in the
NuGetEventSource.Keywords
nested class:This fails to compile:
This is fixed by explicitly casting to the enum type:
See dotnet/source-build-reference-packages@008bdd5 from dotnet/source-build-reference-packages#896 for an example of this occurrence.
The text was updated successfully, but these errors were encountered: