Bug 26704 - bugzilla-tool needs to invoke a specific build-webkit
Summary: bugzilla-tool needs to invoke a specific build-webkit
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-24 19:08 PDT by Eric Seidel (no email)
Modified: 2009-06-25 14:52 PDT (History)
2 users (show)

See Also:


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+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 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.
Comment 1 Eric Seidel (no email) 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(-)
Comment 2 Mark Rowe (bdash) 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
Comment 3 Eric Seidel (no email) 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