Skip to content

Fix Roslyn dependency version in source-build - #84366

Merged
jkoritzinsky merged 1 commit into
mainfrom
source-build-version-fix
Apr 6, 2023
Merged

Fix Roslyn dependency version in source-build#84366
jkoritzinsky merged 1 commit into
mainfrom
source-build-version-fix

Conversation

@jkoritzinsky

Copy link
Copy Markdown
Member

Now that Roslyn builds after dotnet/runtime in source-build, we don't want to use the live Roslyn version in our source-build build.

Unblocks dotnet/sdk#31571

Now that Roslyn builds after dotnet/runtime in source-build, we don't want to use the live Roslyn version in our source-build build.
Unblocks dotnet/sdk#31571
@jkoritzinskyjkoritzinsky added blocking Marks issues that we want to fast track in order to unblock other important work area-Infrastructure partner-impact This issue impacts a partner who needs to be kept updated labels Apr 5, 2023
@ghost

ghost commented Apr 5, 2023

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Now that Roslyn builds after dotnet/runtime in source-build, we don't want to use the live Roslyn version in our source-build build.

Unblocks dotnet/sdk#31571

Author:jkoritzinsky
Assignees:-
Labels:

blocking, area-Infrastructure, partner-impact

Milestone:-

@jkoritzinsky

Copy link
Copy Markdown
MemberAuthor

Test failures are unrelated. Merging in to unblock dotnet/sdk.

@jkoritzinsky
jkoritzinsky merged commit cc01464 into mainApr 6, 2023
@jkoritzinsky
jkoritzinsky deleted the source-build-version-fix branch April 6, 2023 06:23
@ghostghost locked as resolved and limited conversation to collaborators May 6, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-InfrastructureblockingMarks issues that we want to fast track in order to unblock other important workpartner-impactThis issue impacts a partner who needs to be kept updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jkoritzinsky@carlossanlop