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)
Tue, Apr 8, 7:35 PM
Unknown Object (File)
Mar 4 2025, 9:21 PM
Unknown Object (File)
Feb 19 2025, 2:06 PM
Unknown Object (File)
Feb 1 2025, 12:30 AM
Unknown Object (File)
Dec 23 2024, 4:15 PM
Unknown Object (File)
Oct 3 2024, 10:09 PM
Unknown Object (File)
Oct 2 2024, 4:52 AM
Unknown Object (File)
Oct 1 2024, 12:42 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.