Be a bit more forgiving with copy_file

Author: ocaisa

easybuild/tools/filetools.py

@@ -2550,14 +2550,12 @@ def copy_file(path, target_path, force_in_dry_run=False):
                         if os.path.exists(target_path) and os.stat(target_path).st_mode & stat.S_IWUSR:
                             raise EasyBuildError("Failed to copy file %s to %s: %s", path, target_path, err)
 
-                        pyver = LooseVersion(platform.python_version())
-                        if pyver >= LooseVersion('3.7'):
+                        # If we have anything other than a PermissionError we bail out
+                        if not isinstance(err, PermissionError):
                             raise EasyBuildError("Failed to copy file %s to %s: %s", path, target_path, err)
-                        elif LooseVersion('3.7') > pyver >= LooseVersion('3'):
-                            if not isinstance(err, PermissionError):
-                                raise EasyBuildError("Failed to copy file %s to %s: %s", path, target_path, err)
 
-                        # double-check whether the copy actually succeeded
+                        # double-check whether the copy actually succeeded (or may have already happened
+                        # if the contents are the same)
                         if not os.path.exists(target_path) or not filecmp.cmp(path, target_path, shallow=False):
                             raise EasyBuildError("Failed to copy file %s to %s: %s", path, target_path, err)
 
@@ -2569,8 +2567,10 @@ def copy_file(path, target_path, force_in_dry_run=False):
                         except OSError as err:
                             raise EasyBuildError("Failed to copy file %s to %s: %s", path, target_path, err)
 
-                        msg = ("Failed to copy extended attributes from file %s to %s, due to a bug in shutil (see "
-                               "https://bugs.python.org/issue24538). Copy successful with workaround.")
+                        msg = ("In some cases, we may fail to copy extended attributes from file %s to %s, due to a "
+                               "bug in shutil (see https://bugs.python.org/issue24538). We may also have already "
+                               "copied the file over so we just check the contents and update the xattrs. "
+                               "Copy successful with workaround.")
                         _log.info(msg, path, target_path)
 
                 elif os.path.islink(path):