Bug 26704

Summary: bugzilla-tool needs to invoke a specific build-webkit
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: levin, mrowe
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Call WebKitTools/Scripts scripts by their absolute paths mrowe: review+

Eric Seidel (no email)
Reported 2009-06-24 19:08:23 PDT
bugzilla-tool needs to invoke a specific build-webkit Currently it uses the one in your path. Which might be in a different webkit repository, which means you'll end up buildnig the wrong copy of WebKit. This is what caused the bad checkins today. :( It's also wrong to use the build-webkit in the path because not everyone will have build-webkit in their path. bugzilla-tool is cwd aware, it passes the cwd off to the SCM system and asks it if its a valid SCM checkout and if so, what the root directory of the checkout is. bugzilla-tool assumes whatever root directory you're in is a WebKit checkout. This allows me to have multiple webkit checkouts and make modifications to bugzilla-tool in one and test the modified version against other checkouts.
Attachments
Call WebKitTools/Scripts scripts by their absolute paths (3.59 KB, patch)
2009-06-25 14:35 PDT, Eric Seidel (no email)
mrowe: review+
Eric Seidel (no email)
Comment 1 2009-06-25 14:35:03 PDT
Created attachment 31876 [details] Call WebKitTools/Scripts scripts by their absolute paths --- 3 files changed, 27 insertions(+), 6 deletions(-)
Mark Rowe (bdash)
Comment 2 2009-06-25 14:49:55 PDT
Comment on attachment 31876 [details] Call WebKitTools/Scripts scripts by their absolute paths As mentioned on IRC, the leading underscore on _script_path is inconsistent with the convention used on other methods in the file. r=me
Eric Seidel (no email)
Comment 3 2009-06-25 14:52:27 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebKitTools/ChangeLog M WebKitTools/Scripts/bugzilla-tool M WebKitTools/Scripts/modules/scm.py Committed r45197 http://trac.webkit.org/changeset/45197
Note You need to log in before you can comment on or make changes to this bug.