Merge pull request #2320 from ajkerrigan/ajk/fix/guess-tar-open-bytes

[tar-] use open_bytes() for guessing filetype
saulpw · Feb 19, 2024 · be295d8 · be295d8
2 parents c4f0280 + e05e87c
commit be295d8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -71,3 +71,10 @@ jobs:
 
     - name: Ensure VisiData can create completions
       run: python -v dev/zsh-completion.py
+
+    - name: Ensure VisiData does not drop lines from RepeatFiles
+      run: |
+        NUMLINES=$(seq 10000 | vd -b | wc -l)
+        if ! [[ $NUMLINES -eq 10000 ]]; then
+            exit 1
+        fi
diff --git a/visidata/loaders/archive.py b/visidata/loaders/archive.py
@@ -15,7 +15,7 @@ def guess_zip(vd, p):
 
 @VisiData.api
 def guess_tar(vd, p):
-    if tarfile.is_tarfile(p.fp):
+    if tarfile.is_tarfile(p.open_bytes()):
         return dict(filetype='tar')
 
 @VisiData.api