Merge cd70c99c67a681581337f8a5e8912a286d296978 into 85a455752586ddec223bcce4f282656ea1f60e4f

core/results.py

@@ -10,7 +10,6 @@ import logging
 import re
 import os
 import os.path as op
-from errno import EISDIR, EACCES
 from xml.etree import ElementTree as ET
 
 from hscommon.jobprogress.job import nulljob
@@ -377,8 +376,8 @@ class Results(Markable):
             do_write(outfile)
         except OSError as e:
             # If our OSError is because dest is already a directory, we want to handle that. 21 is
-            # the code we get on OS X and Linux (EISDIR), 13 is what we get on Windows (EACCES).
+            # the code we get on OS X and Linux, 13 is what we get on Windows.
-            if e.errno in (EISDIR, EACCES):
+            if e.errno in {21, 13}:
                 p = str(outfile)
                 dirname, basename = op.split(p)
                 otherfiles = os.listdir(dirname)

hscommon/conflict.py

@@ -14,7 +14,6 @@ import re
 import os
 import shutil
 
-from errno import EISDIR, EACCES
 from pathlib import Path
 from typing import Callable, List
 
@@ -76,8 +75,10 @@ def smart_copy(source_path: Path, dest_path: Path) -> None:
     try:
         _smart_move_or_copy(shutil.copy, source_path, dest_path)
     except OSError as e:
-        # It's a directory, code is 21 on OS X / Linux (EISDIR) and 13 on Windows (EACCES)
+        if e.errno in {
-        if e.errno in (EISDIR, EACCES):
+            21,
+            13,
+        }:  # it's a directory, code is 21 on OS X / Linux and 13 on Windows
             _smart_move_or_copy(shutil.copytree, source_path, dest_path)
         else:
             raise