Skip to content

Conversation

@wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Apr 3, 2023

Based on a request from the Office team - they pass /norestart to the repair command of the hosting bundle, but still have to restart because args don't get passed through to inner bundles. We don't want to blindly pass through args, but I think it's reasonable to always pass /norestart to the inner bundles (If the user doesn't pass /norestart to the hosting bundle, we'll restart, and if they do pass it, we won't).

To be backported to 6 & 7

Part of #47544

@wtgodbe wtgodbe requested a review from joeloff April 3, 2023 16:59
@wtgodbe wtgodbe requested a review from a team as a code owner April 3, 2023 16:59
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 3, 2023
@ghost
Copy link

ghost commented Apr 3, 2023

Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at.

@wtgodbe
Copy link
Member Author

wtgodbe commented Apr 3, 2023

/backport to release/7.0

@github-actions
Copy link
Contributor

github-actions bot commented Apr 3, 2023

Started backporting to release/7.0: https://github.com/dotnet/aspnetcore/actions/runs/4599333447

@wtgodbe
Copy link
Member Author

wtgodbe commented Apr 3, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@wtgodbe wtgodbe merged commit 6686303 into dotnet:main Apr 3, 2023
@ghost ghost added this to the 8.0-preview4 milestone Apr 3, 2023
@wtgodbe wtgodbe deleted the wtgodbe/NoRestart branch April 3, 2023 21:41
@wtgodbe wtgodbe restored the wtgodbe/NoRestart branch November 15, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants