Skip to content

Don't use docker for sourcebuild leg#51815

Merged
333fred merged 1 commit intodotnet:mainfrom
333fred:sourcebuild-notdocker
Mar 12, 2021
Merged

Don't use docker for sourcebuild leg#51815
333fred merged 1 commit intodotnet:mainfrom
333fred:sourcebuild-notdocker

Conversation

@333fred
Copy link
Member

@333fred 333fred commented Mar 11, 2021

No description provided.

@333fred 333fred requested a review from a team as a code owner March 11, 2021 23:33
@ghost ghost added the Area-Infrastructure label Mar 11, 2021
@333fred
Copy link
Member Author

333fred commented Mar 12, 2021

@RikkiGibson @jaredpar for review.

@chsienki
Copy link
Member

chsienki commented Mar 12, 2021

@333fred do you know what the rationale was for using docker for source build in the first place? (and conversely is there a reason we're getting rid of it?)

@333fred
Copy link
Member Author

333fred commented Mar 12, 2021

@333fred do you know what the rationale was for using docker for source build in the first place? (and conversely is there a reason we're getting rid of it?)

The rationale was "This is what every other build was doing". The rationale is still "This is what every build is doing."

@333fred 333fred merged commit 38be2dc into dotnet:main Mar 12, 2021
@333fred 333fred deleted the sourcebuild-notdocker branch March 12, 2021 02:22
@allisonchou allisonchou added this to the Next milestone Mar 12, 2021
@allisonchou allisonchou modified the milestones: Next, 16.10.P2 Mar 29, 2021
333fred pushed a commit that referenced this pull request Apr 29, 2021
It doesn't seem to be used anymore after #51815
JoeRobich pushed a commit to JoeRobich/roslyn that referenced this pull request May 12, 2022
It doesn't seem to be used anymore after dotnet#51815
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants