test_shutil fails on Windows after r86733

[briancurtin]

BPO	10540
Nosy	@tarekziade, @briancurtin
Files	issue10540.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/briancurtin'
closed_at = <Date 2011-04-20.00:55:10.933>
created_at = <Date 2010-11-26.16:00:33.820>
labels = ['type-bug', 'library', 'OS-windows']
title = 'test_shutil fails on Windows after r86733'
updated_at = <Date 2011-04-20.00:55:10.932>
user = 'https://github.com/briancurtin'

bugs.python.org fields:

activity = <Date 2011-04-20.00:55:10.932>
actor = 'brian.curtin'
assignee = 'brian.curtin'
closed = True
closed_date = <Date 2011-04-20.00:55:10.933>
closer = 'brian.curtin'
components = ['Library (Lib)', 'Windows']
creation = <Date 2010-11-26.16:00:33.820>
creator = 'brian.curtin'
dependencies = []
files = ['19826']
hgrepos = []
issue_num = 10540
keywords = ['patch']
message_count = 3.0
messages = ['122462', '122466', '134115']
nosy_count = 2.0
nosy_names = ['tarek', 'brian.curtin']
pr_nums = []
priority = 'normal'
resolution = 'works for me'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue10540'
versions = ['Python 3.2']

[briancurtin]

My build slave shows a test failure at test_dont_copy_file_onto_link_to_itself. This happens because the implementation of _samefile in Lib/shutil.py (line 70) doesn't work for Windows hard links.

Patch on the way.

[briancurtin]

Here is a patch.

os.path.samefile and hard links don't work for Windows the same way they do for Mac/Linux. In the case where we are on Windows and a link comes into the _samefile function, check that it's a link and then use os.path.sameopenfile. If it's not a link but a regular file, or it's some other OS, do the os.path.samefile check.

[briancurtin]

This hasn't been an issue for quite some time, and I suspect the follow-up work on symbolic and hard links and their supporting functions corrected any problems in this area.