target_files_diff: Fix the issue with file type change.

When a file is switched from a regular file to a symlink, we should skip
the comparasion.

Bug: 28373409
Change-Id: I8fef00ab9f2d6f811fde1cadf595e8bd46af2dfd
This commit is contained in:
Tao Bao
2016-04-25 10:03:38 -07:00
parent b5658074c3
commit f31a6deeee

View File

@@ -172,16 +172,15 @@ def recursiveDiff(prefix, dir1, dir2, out_file):
continue continue
if entry in list2: if entry in list2:
if os.path.islink(name1): if os.path.islink(name1) and os.path.islink(name2):
if os.path.islink(name2):
link1 = os.readlink(name1) link1 = os.readlink(name1)
link2 = os.readlink(name2) link2 = os.readlink(name2)
if link1 != link2: if link1 != link2:
print("%s: Symlinks differ: %s vs %s" % (name, link1, link2), print("%s: Symlinks differ: %s vs %s" % (name, link1, link2),
file=out_file) file=out_file)
else: continue
print("%s: File types differ, skipping compare" % name, elif os.path.islink(name1) or os.path.islink(name2):
file=out_file) print("%s: File types differ, skipping compare" % name, file=out_file)
continue continue
stat1 = os.stat(name1) stat1 = os.stat(name1)