Also look for errno.EACCES to ensure compatibility with Windows

core/results.py

@@ -10,7 +10,7 @@ import logging
 import re
 import os
 import os.path as op
-from errno import EISDIR
+from errno import EISDIR, EACCES
 from xml.etree import ElementTree as ET
 
 from hscommon.jobprogress.job import nulljob
@@ -377,7 +377,7 @@ 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.
-            if e.errno == EISDIR:
+            if e.errno in (EISDIR, EACCES):
                 p = str(outfile)
                 dirname, basename = op.split(p)
                 otherfiles = os.listdir(dirname)

hscommon/conflict.py

@@ -14,7 +14,7 @@ import re
 import os
 import shutil
 
-from errno import EISDIR
+rom errno import EISDIR, EACCES
 from pathlib import Path
 from typing import Callable, List
 
@@ -76,7 +76,7 @@ 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:
-        if e.errno == EISDIR:  # it's a directory
+        if e.errno in (EISDIR, EACCES):  # it's a directory
             _smart_move_or_copy(shutil.copytree, source_path, dest_path)
         else:
             raise