Merging upstream version 2.1.2.

Signed-off-by: Daniel Baumann <daniel@debian.org>

identify/extensions.py

@@ -183,6 +183,7 @@ EXTENSIONS = {
     'txsprofile': {'text', 'ini', 'txsprofile'},
     'txt': {'text', 'plain-text'},
     'v': {'text', 'verilog'},
+    'vb': {'text', 'vb'},
     'vbproj': {'text', 'xml', 'vbproj'},
     'vcxproj': {'text', 'xml', 'vcxproj'},
     'vdx': {'text', 'vdx'},

identify/identify.py

@@ -1,3 +1,4 @@
+import errno
 import os.path
 import re
 import shlex
@@ -205,8 +206,14 @@ def parse_shebang_from_file(path: str) -> Tuple[str, ...]:
     if not os.access(path, os.X_OK):
         return ()
 
-    with open(path, 'rb') as f:
+    try:
-        return parse_shebang(f)
+        with open(path, 'rb') as f:
+            return parse_shebang(f)
+    except OSError as e:
+        if e.errno == errno.EINVAL:
+            return ()
+        else:
+            raise
 
 
 COPYRIGHT_RE = re.compile(r'^\s*(Copyright|\(C\)) .*$', re.I | re.MULTILINE)

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = identify
-version = 2.1.0
+version = 2.1.2
 description = File identification library for Python
 long_description = file: README.md
 long_description_content_type = text/markdown

tests/identify_test.py

@@ -1,8 +1,11 @@
+import builtins
+import errno
 import io
 import os
 import socket
 import stat
 from tempfile import TemporaryDirectory
+from unittest import mock
 
 import pytest
 
@@ -330,6 +333,15 @@ def test_parse_shebang_from_file_simple(tmpdir):
     assert identify.parse_shebang_from_file(x.strpath) == ('python',)
 
 
+def test_parse_shebang_open_raises_einval(tmpdir):
+    x = tmpdir.join('f')
+    x.write('#!/usr/bin/env not-expected\n')
+    make_executable(x)
+    error = OSError(errno.EINVAL, f'Invalid argument {x}')
+    with mock.patch.object(builtins, 'open', side_effect=error):
+        assert identify.parse_shebang_from_file(x.strpath) == ()
+
+
 def make_executable(filename):
     original_mode = os.stat(filename).st_mode
     os.chmod(