Bugfix/1827 invalid cross device link

[siemdejong]

Fixes #1827

[siemdejong]

Creating the temporary file next to the target path (making sure it's on the same device) solves this issue. Took inspiration from

filesystem_spec/fsspec/utils.py

Lines 616 to 618 in 6cc68c7

	fd, fn = tempfile.mkstemp(
	dir=os.path.dirname(path), prefix=os.path.basename(path) + "-"
	)

This seems to work as expected with shutil.move.

Then os.chmod may raise PermissionError too, so that error can be catched.

It is difficult to mock the cross-device issue as it requires a temporary file on another device than the target device, so I mock this as an EXDEV at os.rename in shutil.move and then a PermissionError at shutil.move -> shutil.copystat -> lookup('chmod').

[siemdejong]

Why do the tests fail? These tests sporadically fail locally, but not always. And I get these errors also sporadically on master.

[siemdejong]

I'm not sure if this is the cleanest way to do it. I'm also not sure if os.path.exists() creates too much overhead, but I think not doing this check may result in catching permission errors that were not from chmod, but from creating files in shutil.move.