Bug 31536

Summary: GitTest.test_create_binary_patch fails if /tmp is symlink
Product: WebKit Reporter: Shinichiro Hamaji <hamaji>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch v1 none

Description Shinichiro Hamaji 2009-11-16 02:10:14 PST
It seems git add /tmp/.../foo fails if /tmp is a symbolic link because git considers this operation is done from outside the repository. For example, typical Mac's /tmp is /private/tmp. In this case, this operation is trying to modify /tmp_git_test_checkout/foo from /private/tmp/tmp_git_test_checkout/, which git dislikes.

> ./WebKitTools/Scripts/run-webkit-unittests GitTest.test_create_binary_patch
E
======================================================================
ERROR: test_create_binary_patch (modules.scm_unittest.GitTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/hamaji/GitKit/WebKitTools/Scripts/modules/scm_unittest.py", line 569, in test_create_binary_patch
    run(['git', 'add', test_file_path])
  File "/Users/hamaji/GitKit/WebKitTools/Scripts/modules/scm_unittest.py", line 48, in run
    return SCM.run_command(args, cwd=cwd)
  File "/Users/hamaji/GitKit/WebKitTools/Scripts/modules/scm.py", line 138, in run_command
    error_handler(script_error)
  File "/Users/hamaji/GitKit/WebKitTools/Scripts/modules/scm.py", line 109, in default_error_handler
    raise error
ScriptError: Failed to run "['git', 'add', '/tmp/tmpM4wgJGgit_test_checkout/binary_file']" exit_code: 128

----------------------------------------------------------------------
Ran 1 test in 6.193s

FAILED (errors=1)
Comment 1 Shinichiro Hamaji 2009-11-16 02:11:14 PST
Created attachment 43281 [details]
Patch v1
Comment 2 Eric Seidel (no email) 2009-11-18 15:41:09 PST
Comment on attachment 43281 [details]
Patch v1

LGTM.  Thanks again for all your recent work at making our tools work better with git!
Comment 3 Shinichiro Hamaji 2009-11-18 21:33:25 PST
Comment on attachment 43281 [details]
Patch v1

Rejecting patch 43281 from commit-queue.

Failed to run "['git', 'svn', 'rebase']" exit_code: 1
Last 500 characters of output:
8f90067 (trunk)
	M	WebCore/ChangeLog
	M	WebCore/html/HTMLInputElement.cpp
r51170 = d6a819cc491fa9514d5186c71dbb3090f3beac63 (trunk)
	M	WebCore/ChangeLog
	M	WebCore/platform/qt/QWebPageClient.h
r51171 = bc68e0d81f5427b13321581e6528732cf6ed4288 (trunk)
First, rewinding head to replay your work on top of it...
error: Untracked working tree file 'LayoutTests/fast/ruby/ruby-remove-expected.txt' would be overwritten by merge.
could not detach HEAD
rebase refs/remotes/trunk: command returned error: 1
Comment 4 Shinichiro Hamaji 2009-11-18 21:47:48 PST
Comment on attachment 43281 [details]
Patch v1

Clearing flags on attachment: 43281

Committed r51173: <http://trac.webkit.org/changeset/51173>
Comment 5 Shinichiro Hamaji 2009-11-18 21:47:57 PST
All reviewed patches have been landed.  Closing bug.