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

chore(man): Don't compress man pages during build #317

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

Antiz96
Copy link
Owner

@Antiz96 Antiz96 commented Feb 17, 2025

Description

Compressing man pages is a distribution / user preference that should be handled on the packaging side of things.

Indeed, compressing man pages by default imposes a compression algorithm / level for users whereas most package managers handle that already and allow users to choose / configure them (if wanted at all). Additionally, it can lead to unexpected side effects. For instance, gzip recording timestamps in metadata by default, which prevents reproducible builds (see 905f641).

See https://cmpct.info/~sam/blog/posts/automatic-manpage-compression/ for detailed rational.

Compressing man pages is a distribution / user preference that should be handled on the packaging side of things.

Indeed, compressing man pages by default imposes a compression algorithm / level for users whereas most package managers handle that already and allow users to choose / configure them (if wanted at all). Additionally, it can lead to unexpected side effects. For instance, gzip recording timestamps in metadata by default, which prevents reproducible builds (see 905f641).

See https://cmpct.info/~sam/blog/posts/automatic-manpage-compression/ for detailed rational.
@Antiz96 Antiz96 merged commit b60600e into main Feb 17, 2025
1 check passed
@Antiz96 Antiz96 deleted the do_not_compress_man_pages branch February 17, 2025 09:09
@Antiz96 Antiz96 added this to the v3.7.2 milestone Feb 17, 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.

1 participant