Update C# Runtime dependencies to support NativeAOT-LLVM prerequisites#4601
Draft
rekhoff wants to merge 3 commits intotyler/update-nativeaot-llvm-infrastructurefrom
Draft
Conversation
…re won't pick up the chain of depedencies through the .nupkg
rekhoff
changed the base branch from
master
to
tyler/update-nativeaot-llvm-infrastructure
rekhoff
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of Changes
sdks/csharp/tools~/write-nuget-config.sh, so generated NuGet.Config files include NativeAOT-LLVM prerequisites.SpacetimeDB.Runtimeto ensure restores succeed even when LLVM dependencies are only referenced through the.nupkg.EXPERIMENTAL_WASM_AOT=1to enable NativeAOT build steps without relying on downstream package reference resolution.Context
Changes are required to get
NativeAOT-LLVMin #4515 to build correct, but moving the packages closer to the build, to ensure they get into the Nuget restore successfully.Additional changes where needed to
write-nuget-config.shto allowNuget.Configfiles generated with required changes during regression testing.API and ABI breaking changes
None.
Expected complexity level and risk
2 (Low–moderate). Changes are scoped to build/restore infrastructure and package configuration.
Testing
run-regression-tests.shwithout errors