Skip to content

Add external antlr4 project#1585

Merged
akoeplinger merged 3 commits intodotnet:mainfrom
jkoritzinsky:antlr4
Mar 6, 2026
Merged

Add external antlr4 project#1585
akoeplinger merged 3 commits intodotnet:mainfrom
jkoritzinsky:antlr4

Conversation

@jkoritzinsky
Copy link
Member

We are adding a dependency on Antlr4.Runtime.Standard in dotnet/runtime#124764 for our new managed ilasm implementation. This PR adds the requisite logic to source-build just the Antlr4 C# runtime.

See dotnet/runtime#124764 for more info

cc: @am11

@jkoritzinsky jkoritzinsky requested a review from akoeplinger March 2, 2026 19:41
@jkoritzinsky jkoritzinsky requested a review from a team as a code owner March 2, 2026 19:41
@jkoritzinsky
Copy link
Member Author

/azp run source-build-reference-packages-unified-build

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

jkoritzinsky added a commit to dotnet/dotnet that referenced this pull request Mar 2, 2026
Added exclusions for various antlr4 runtime files to limit compliance work.

Opening before we merge in dotnet/source-build-reference-packages#1585
Copy link
Member

@ellahathaway ellahathaway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NU1101: Unable to find package Microsoft.NETFramework.ReferenceAssemblies`

On Windows, Antlr4.csproj targets net45;netstandard2.0; restoring net45 implicitly requires Microsoft.NETFramework.ReferenceAssemblies, but this build uses a generated NuGet config that clears all feeds except SBRP, where that package is not available.

@am11
Copy link
Member

am11 commented Mar 5, 2026

restoring net45 implicitly requires Microsoft.NETFramework.ReferenceAssemblies, but this build uses a generated NuGet config that clears all feeds except SBRP, where that package is not available.

I think we should skip net45 as the consumer project is only targeting net11.

Co-authored-by: Adeel Mujahid <3840695+am11@users.noreply.github.com>
@akoeplinger akoeplinger merged commit 126ae71 into dotnet:main Mar 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants