Skip to content

Conversation

ricardobranco777
Copy link

The --zero option is useless if not supported by --check.

This PR fixes it.

@pixelb
Copy link
Member

pixelb commented Feb 24, 2024

I'm not strongly against this, but the thinking was that md5sum itself doesn't need the --zero option when veryifying, whereas other programs could benefit from the simplified processing.

I.e. by disabling --zero working with --check, we restrict checksum files to text format, without restricting any functionality

So the explicit disadvantages of not restricting checksum files to text format are:

  • checksum files are no longer textual, and so harder to handle with other tools
  • older versions of md5sum can not parse these binary checksum files

@ricardobranco777 ricardobranco777 changed the title md5: Allow --zero with --check md5sum: Allow --zero with --check Feb 25, 2024
@ricardobranco777
Copy link
Author

  • older versions of md5sum can not parse these binary checksum files

Newer versions of md5sum should support checking files generated with --zero. It should have been done when introducing the flag. Anyway the patch is trivial and works.

@pixelb
Copy link
Member

pixelb commented Oct 2, 2025

Closing on the basis of minimizing checksum file format variants

@pixelb pixelb closed this Oct 2, 2025
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