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

phase2: signpack: fix sha256sums signatures of apk packages.adb index #54

Merged
merged 3 commits into from
Oct 12, 2024

Conversation

ynezz
Copy link
Member

@ynezz ynezz commented Oct 12, 2024

Currently the sha256sum verification of apk's packages.adb index
fails as the file is modified with apk adbsign, but we currently don't
send the sha256sums file to the master, thus it can't be fixed during
signing.

So lets pack sha256sums files and ship them to master for proper
signing.

Fixes: a94d4e1 ("add APK signing logic")

@ynezz ynezz force-pushed the ynezz/phase2-signpack-fixes branch from e6ee788 to c1d6f19 Compare October 12, 2024 08:00
So its more readable and diffs of changes are more clear, thus easier to
review.

Signed-off-by: Petr Štetiar <[email protected]>
Currently the sha256sum verification of apk's packages.adb index
fails as the file is modified with `apk adbsign`, but we currently don't
send the sha256sums file to the master, thus it can't be fixed during
signing.

So lets pack sha256sums files and ship them to master for proper
signing.

Fixes: a94d4e1 ("add APK signing logic")
Signed-off-by: Petr Štetiar <[email protected]>
@ynezz ynezz force-pushed the ynezz/phase2-signpack-fixes branch from c1d6f19 to 49c07ef Compare October 12, 2024 08:01
Fixes following warning:

 gitpoller.py:103: buildbot.warnings.DeprecatedApiWarning: [3.11.3 and later] pollinterval has been deprecated: please use pollInterval

Signed-off-by: Petr Štetiar <[email protected]>
@openwrt-bot openwrt-bot merged commit 3915bd8 into openwrt:main Oct 12, 2024
4 checks passed
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.

2 participants