Page MenuHomeFreeBSD

tarfs: Silence warnings when no supported compression is enabled.
ClosedPublic

Authored by des on Feb 7 2023, 1:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 11, 11:30 PM
Unknown Object (File)
Tue, Nov 25, 2:51 AM
Unknown Object (File)
Nov 20 2025, 6:18 AM
Unknown Object (File)
Nov 19 2025, 3:46 AM
Unknown Object (File)
Nov 11 2025, 9:07 PM
Unknown Object (File)
Nov 11 2025, 9:07 PM
Unknown Object (File)
Nov 11 2025, 9:07 PM
Unknown Object (File)
Nov 11 2025, 5:13 PM
Subscribers

Details

Summary

Sponsored by: Juniper Networks, Inc.
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des requested review of this revision.Feb 7 2023, 1:27 PM
sys/fs/tarfs/tarfs_io.c
43

Why not write #ifdef ZSTDIO, same as in other places?

And why do you need TARFS_ZIO, instead of reusing ZSTDIO?

des marked an inline comment as done.Feb 7 2023, 3:03 PM
des added inline comments.
sys/fs/tarfs/tarfs_io.c
43

Because I have branches with support for zlib and xz compression as well. The code which is #ifdef TARFS_ZIO is common to all three, not specific to zstd. I didn't include the other two in the initial code drop because they don't support seekable compression without upstream changes.

kib added inline comments.
sys/fs/tarfs/tarfs_io.c
43

Then at least add this explanation to the commit message. Otherwise it would cause the same question from other people, I am sure.

This revision is now accepted and ready to land.Feb 7 2023, 4:25 PM
This revision was automatically updated to reflect the committed changes.
des marked an inline comment as done.