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(waydroid-deb and deps): Update sha256sum of the packages too #6817

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kurtbahartr
Copy link
Contributor

The sums were left out of date despite the uprevs.

This in turn causes problems with source verification.

ARM64 builds might still have the sums out of date, this part definitely
needs some checks to be done.

@github-actions github-actions bot added the Update Pacscript Updates to the pacscript itself label Jan 1, 2025
@oklopfer
Copy link
Member

oklopfer commented Jan 2, 2025

Please ensure you are:

  1. Updating not just the sha256sums, but also the pkgver (or pkgrel if it is only a hash update) so that it pulls correctly
  2. You cannot update just amd64. Arm64 must be updated too.

Copy link
Member

@oklopfer oklopfer left a comment

Choose a reason for hiding this comment

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

See comment above

@kurtbahartr
Copy link
Contributor Author

Roger that, I'll do the changes you asked in a few moments.

@kurtbahartr
Copy link
Contributor Author

Just did a force-push for the changes you requested. Could you please review again when you can?

Also uprev the pkgrel since these are just quick fixups to the
existing versions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Update Pacscript Updates to the pacscript itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants