Skip to content

Explicitely pass flag to decompress bzip2 archives in test#518

Merged
Leonidas-from-XIV merged 1 commit intotarides:mainfrom
Leonidas-from-XIV:fix-openbsd-tar
Mar 25, 2026
Merged

Explicitely pass flag to decompress bzip2 archives in test#518
Leonidas-from-XIV merged 1 commit intotarides:mainfrom
Leonidas-from-XIV:fix-openbsd-tar

Conversation

@Leonidas-from-XIV
Copy link
Copy Markdown
Member

While looking at the CI results of #517 the (optional) OpenBSD tests are failing. But it looks that the only thing preventing them from passing is one single cram test.

tar on OpenBSD can decompress bzip2 but it does not do that automatically by default, unlike GNU tar or bsdtar.

However all tar implementations support the j flag so might as well use it in the test - we know it's always going to be a bzip2 compressed file, so the autodetection is not necessary.

This should fix tests on OpenBSD while keeping it working just fine on all other platforms too.

`tar` on OpenBSD can decompress bzip2 but it does not do that
automatically by default, unlike GNU tar or bsdtar.

However all tar implementations support the `j` flag so might as well
use it.

This should fix tests on OpenBSD while keeping it working just fine on
all other platforms too.
@Leonidas-from-XIV Leonidas-from-XIV requested a review from avsm March 25, 2026 10:13
@Leonidas-from-XIV Leonidas-from-XIV added the no changelog Add this to your PRs to disable the changelog update check label Mar 25, 2026
Copy link
Copy Markdown
Member

@Sudha247 Sudha247 left a comment

Choose a reason for hiding this comment

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

LGTM, for what it's worth!

@Leonidas-from-XIV
Copy link
Copy Markdown
Member Author

The OpenBSD tests now all pass; merging.

@Leonidas-from-XIV Leonidas-from-XIV merged commit 7fa2b7d into tarides:main Mar 25, 2026
3 of 4 checks passed
@Leonidas-from-XIV Leonidas-from-XIV deleted the fix-openbsd-tar branch March 31, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog Add this to your PRs to disable the changelog update check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants