-
-
Notifications
You must be signed in to change notification settings - Fork 207
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Skipping package signing check during workload installation #2878
Conversation
@@ -42,7 +42,8 @@ runs: | |||
maui-android \ | |||
${{ runner.os == 'macOS' && 'maui-ios maui-maccatalyst maui-windows maui-tizen' || '' }} \ | |||
${{ runner.os == 'Windows' && 'maui-windows' || '' }} \ | |||
--temp-dir "${{ runner.temp }}" --from-rollback-file rollback.json | |||
--temp-dir "${{ runner.temp }}" --from-rollback-file rollback.json \ | |||
--skip-sign-check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--skip-sign-check | |
--skip-sign-check | |
# Skipping signing check above because: https://github.com/dotnet/maui/issues/9573 |
Unrelated but didnt' we remove rollback.json ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We did on the feat/4.0.0
branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if this has something to do with the --from-rollback-file
parameter. We weren't experiencing these problems on the feat/4.0.0
branch where that parameter had been removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that was on .NET 6, I'm surprised if this "hack" is needed in .NET 8
Apparently, this is the way to go: dotnet/maui#9573 (comment)
#skip-changelog