py3: catch specific OSError subclasses instead of checking errno

Contrary to the previous changesets in this series, this covers cases where
errno was checked for multiple values.

EACCES -> PermissionError
ENOENT -> FileNotFoundError
ENOTDIR -> NotADirectoryError
EISDIR -> IsADirectoryError

mercurial/dirstate.py

@@ -8,7 +8,6 b''
 
 import collections
 import contextlib
-import errno
 import os
 import stat
 import uuid
@@ -1035,13 +1034,11 b' class dirstate:'
                 try:
                     with tracing.log('dirstate.walk.traverse listdir %s', nd):
                         entries = listdir(join(nd), stat=True, skip=skip)
-                except OSError as inst:
-                    if inst.errno in (errno.EACCES, errno.ENOENT):
+                except (PermissionError, FileNotFoundError) as inst:
                         match.bad(
                             self.pathto(nd), encoding.strtolocal(inst.strerror)
                         )
                         continue
-                    raise
                 for f, kind, st in entries:
                     # Some matchers may return files in the visitentries set,
                     # instead of 'this', if the matcher explicitly mentions them

mercurial/fileset.py

@@ -6,7 +6,6 b''
 # GNU General Public License version 2 or any later version.
 
 
-import errno
 import re
 
 from .i18n import _
@@ -575,16 +574,14 b' class matchctx:'
                     return False
                 try:
                     return predfn(fctx)
-                except (IOError, OSError) as e:
-                    # open()-ing a directory fails with EACCES on Windows
-                    if e.errno in (
-                        errno.ENOENT,
-                        errno.EACCES,
-                        errno.ENOTDIR,
-                        errno.EISDIR,
+                # open()-ing a directory fails with PermissionError on Windows
+                except (
+                    FileNotFoundError,
+                    PermissionError,
+                    NotADirectoryError,
+                    IsADirectoryError,
                     ):
                         return False
-                    raise
 
         else:
 

mercurial/lock.py

@@ -38,9 +38,8 b' def _getlockprefix():'
     if pycompat.sysplatform.startswith(b'linux'):
         try:
             result += b'/%x' % os.stat(b'/proc/self/ns/pid').st_ino
-        except OSError as ex:
-            if ex.errno not in (errno.ENOENT, errno.EACCES, errno.ENOTDIR):
-                raise
+        except (FileNotFoundError, PermissionError, NotADirectoryError):
+            pass
     return result
 
 

mercurial/posix.py

@@ -581,9 +581,7 b' def statfiles(files):'
             st = lstat(nf)
             if getkind(st.st_mode) not in _wantedkinds:
                 st = None
-        except OSError as err:
-            if err.errno not in (errno.ENOENT, errno.ENOTDIR):
-                raise
+        except (FileNotFoundError, NotADirectoryError):
             st = None
         yield st
 

setup.py

@@ -95,7 +95,6 b' else:'
 ispypy = "PyPy" in sys.version
 
 import ctypes
-import errno
 import stat, subprocess, time
 import re
 import shutil
@@ -1422,15 +1421,12 b' class RustExtension(Extension):'
             )
         try:
             subprocess.check_call(cargocmd, env=env, cwd=self.rustsrcdir)
-        except OSError as exc:
-            if exc.errno == errno.ENOENT:
+        except FileNotFoundError:
                 raise RustCompilationError("Cargo not found")
-            elif exc.errno == errno.EACCES:
+        except PermissionError:
                 raise RustCompilationError(
                     "Cargo found, but permission to execute it is denied"
                 )
-            else:
-                raise
         except subprocess.CalledProcessError:
             raise RustCompilationError(
                 "Cargo failed. Working directory: %r, "