tar: raise error for directory argument to -f #337

mknos · 2023-11-21T02:13:10Z

Adding die() for directory argument ensures tar doesn't exit with a success code
Test command: perl tar -tf .
Found when testing against GNU tar

* Adding die() for directory argument ensures tar doesn't exit with a success code * Test command: perl tar -tf . * Found when testing against GNU tar

briandfoy

This can be improved with the more sophisticated exits we've been doing, but this is good enough for now.

tar: raise error for directory argument to -f

cbf437f

* Adding die() for directory argument ensures tar doesn't exit with a success code * Test command: perl tar -tf . * Found when testing against GNU tar

github-actions bot added Type: enhancement improve a feature that already exists Priority: low get to this whenever Program: tar The tar program labels Nov 21, 2023

briandfoy approved these changes Nov 21, 2023

View reviewed changes

briandfoy merged commit 469549e into briandfoy:master Nov 21, 2023
1 check passed

briandfoy temporarily deployed to merge November 21, 2023 06:02 — with GitHub Actions Inactive

briandfoy added Status: accepted The fix is accepted and removed Priority: low get to this whenever labels Nov 21, 2023

briandfoy self-assigned this Nov 21, 2023

briandfoy added Status: released there is a new release with this fix and removed Status: accepted The fix is accepted labels Nov 24, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

tar: raise error for directory argument to -f #337

tar: raise error for directory argument to -f #337

mknos commented Nov 21, 2023

briandfoy left a comment

tar: raise error for directory argument to -f #337

tar: raise error for directory argument to -f #337

Conversation

mknos commented Nov 21, 2023

briandfoy left a comment

Choose a reason for hiding this comment