tar: fail early for bad options

[mknos]

• Add missing getopts() return value check
• This version of tar didn't have a usage string, so add one
• Don't list -c in usage string for now because archive creation isn't implemented
• Unconditionally use Compress::Zlib because it is a core library

%perl tar -x -f yo -L
Unknown option: L
usage: tar {-tx} [-mvZz] [-f archive] [file ...]

[coveralls]

Pull Request Test Coverage Report for Build 13655550118

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 4 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.9%) to 70.088%

---

Files with Coverage Reduction	New Missed Lines	%
bin/glob	1	57.14%
bin/units	3	69.91%

Totals
Change from base Build 13632474012:	-0.9%
Covered Lines:	396
Relevant Lines:	565

---

💛 - Coveralls

[briandfoy]

This is a v5.10 module, so we can't rely on it being there. The conditional check should still be there,.

[mknos]

Thanks for the feedback. New commit added which restores the module-exists check, without conditionally changing the personality of getopts().
As a test I created an archive with GNU tar 1.34, listed the entries with perl tar -tf x.tar, compressed the archive with gzip 1.12 and did a detailed list of compressed archive with perl tar -tvzf x.tar.gz.