Skip to content

Manual: update checksum command - #61687

Closed
gschwaer wants to merge 1 commit into
void-linux:masterfrom
gschwaer:fix_manual_checksum_cmd
Closed

Manual: update checksum command#61687
gschwaer wants to merge 1 commit into
void-linux:masterfrom
gschwaer:fix_manual_checksum_cmd

Conversation

@gschwaer

Copy link
Copy Markdown
Contributor

According to the manual the tarballs content is checksumed. But xbps-src uses xbps-digest, which checksums the whole tarball.

Testing the changes

  • I tested the changes in this PR: YES
    • I tested checksumming the tarball and the tarball content of a known package (neovim), the tarball checksum matched, the tarball content checksum did not match.

Local build testing

  • I did no built testing of this PR.

@sgn sgn closed this Jul 28, 2026
@gschwaer

Copy link
Copy Markdown
Contributor Author

@sgn Please let me know why this PR was closed so I can improve. Thx

@sgn

sgn commented Jul 29, 2026

Copy link
Copy Markdown
Member

Current content is correct, yours is wrong!

@gschwaer

Copy link
Copy Markdown
Contributor Author

Apologies, I did not read carefully. I skimmed through the docs to find the command how to generate a checksum and missed that this command only applies to @ marked content checksums.

Sorry for wasting your time. 🙇

I applied the boy scout rule: #61715

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