Skip to content
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

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

bitsandfoxes
Copy link
Contributor

@bitsandfoxes bitsandfoxes commented Nov 21, 2023

Apparently, this is the way to go: dotnet/maui#9573 (comment)
#skip-changelog

@bitsandfoxes bitsandfoxes changed the title fix: Skipping package signing during workload installation fix: Skipping package signing check during workload installation Nov 21, 2023
@@ -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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
--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 ?

Copy link
Contributor Author

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

Copy link
Collaborator

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.

Copy link
Member

@bruno-garcia bruno-garcia left a 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

@jamescrosswell jamescrosswell merged commit 2177b05 into main Nov 21, 2023
16 checks passed
@jamescrosswell jamescrosswell deleted the fix/package-signing-issue branch November 21, 2023 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants