New submission from Brian Curtin <cur...@acm.org>:

ntpath.samefile is currently implemented using GetFinalPathNameByHandle, which 
doesn't work for hard links.

Since I introduced values for os.stat().st_ino in #8879 (which implemented 
os.link), I suspect we can possibly change ntpath.samefile to be the same as 
posixfile.samefile and leave st_ino.


Hirokazu questioned whether or not that st_ino change was correct, but I think 
it will be fine on NTFS. As FAT does not support links anyways, there would be 
no issue with potential st_ino issues there as we wouldn't be comparing them 
for any reason.


(part of the reason I created this is that we have discussion on st_ino and 
samefile in a few different places and I'm losing track of where we're at)

----------
assignee: brian.curtin
components: Library (Lib), Windows
messages: 123564
nosy: brian.curtin, ocean-city
priority: normal
severity: normal
stage: unit test needed
status: open
title: ntpath.samefile doesn't work for hard links
type: behavior
versions: Python 3.2

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue10646>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to