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

ca-certificates: reinstall to fix certificates #8

Merged
merged 1 commit into from Jun 4, 2023
Merged

Conversation

dennisameling
Copy link
Collaborator

For some reason, a previous update of the mingw-w64-clang-aarch64-ca-certificates package wiped all contents of various certificate files. Let's reinstall using pacman -S mingw-w64-clang-aarch64-ca-certificates to fix this situation.

Ref: be7c294

For some reason, a previous update of the
`mingw-w64-clang-aarch64-ca-certificates` package wiped all contents of
various certificate files. Let's reinstall using
`pacman -S mingw-w64-clang-aarch64-ca-certificates` to fix this
situation.

Ref: be7c294
Signed-off-by: Dennis Ameling <dennis@dennisameling.com>
@dennisameling dennisameling requested a review from dscho June 4, 2023 11:22
Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

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

Sadly, this will be a recurring situation (and, in fact, I did something similar in the past although I failed to open a PR for it): the post-install script of the ca-certificates package calls p11-kit.exe to (re-)generate the certificate files. Naturally, this executable is an ARM64 one in git-sdk-arm64. But we do run the sync job of git-sdk-arm64 on hosted runners, i.e. on x86_64 CPUs.

For the time being, I think it is best if we keep doing that, but if Windows/ARM64 runners are a long ways off in the future, we may need to hack around this problem.

@dscho dscho merged commit 1490f41 into main Jun 4, 2023
@dscho dscho deleted the fix-ca-certs branch June 4, 2023 19:24
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.

None yet

2 participants