fix path issues on mac and win for skpdiff_server.py
R=djsollen@google.com, epoger@google.com, jvanverth@google.com Author: zachr@google.com Review URL: https://chromiumcodereview.appspot.com/22650009 git-svn-id: http://skia.googlecode.com/svn/trunk@10719 2bbb7eff-a529-9590-31e7-b0007b416f81
This commit is contained in:
parent
7d65dee189
commit
fe87fb8a33
@ -67,8 +67,12 @@ def get_skpdiff_path(user_path=None):
|
||||
else:
|
||||
possible_paths.append(os.path.join(SKIA_ROOT_DIR, 'out',
|
||||
'Release', 'skpdiff'))
|
||||
possible_paths.append(os.path.join(SKIA_ROOT_DIR, 'out',
|
||||
'Release', 'skpdiff.exe'))
|
||||
possible_paths.append(os.path.join(SKIA_ROOT_DIR, 'out',
|
||||
'Debug', 'skpdiff'))
|
||||
possible_paths.append(os.path.join(SKIA_ROOT_DIR, 'out',
|
||||
'Debug', 'skpdiff.exe'))
|
||||
# Use the first path that actually points to the binary
|
||||
for possible_path in possible_paths:
|
||||
if os.path.isfile(possible_path):
|
||||
@ -159,8 +163,9 @@ def get_head_version(path):
|
||||
"""
|
||||
|
||||
# git-show will not work with absolute paths. This ensures we give it a path
|
||||
# relative to the skia root.
|
||||
git_path = os.path.relpath(path, SKIA_ROOT_DIR)
|
||||
# relative to the skia root. This path also has to use forward slashes, even
|
||||
# on windows.
|
||||
git_path = os.path.relpath(path, SKIA_ROOT_DIR).replace('\\', '/')
|
||||
git_show_proc = subprocess.Popen(['git', 'show', 'HEAD:' + git_path],
|
||||
stdout=subprocess.PIPE)
|
||||
|
||||
@ -221,8 +226,10 @@ class ExpectationsManager:
|
||||
|
||||
|
||||
# Create a temporary file tree that makes sense for skpdiff to operate
|
||||
# on.
|
||||
image_output_dir = tempfile.mkdtemp('skpdiff')
|
||||
# on. We take the realpath of the new temp directory because some OSs
|
||||
# (*cough* osx) put the temp directory behind a symlink that gets
|
||||
# resolved later down the pipeline and breaks the image map.
|
||||
image_output_dir = os.path.realpath(tempfile.mkdtemp('skpdiff'))
|
||||
expected_image_dir = os.path.join(image_output_dir, 'expected')
|
||||
actual_image_dir = os.path.join(image_output_dir, 'actual')
|
||||
os.mkdir(expected_image_dir)
|
||||
|
Loading…
Reference in New Issue
Block a user