Bug 16314

Summary:

Script to launch Drosera

Product:

WebKit

Reporter:

Brent Fulgham <bfulgham>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Normal

CC:

aroben, timothy

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

Windows XP

Bug Depends on:

Bug Blocks:

16315

Attachments:

Description	Flags
Patch adding a new "runDrosera" command to the webkitdirs.pm.	none
Helper DOS command shell to run Drosera.	none
Bash script to run Drosera under Windows	none
Revised patch to support Windows and Mac OS	none
Helper script to run Drosera in debug.	none
Patch containing updates to support building and running Drosera under win32 and Mac OS X.	mjs: review+

Brent Fulgham

Reported 2007-12-05 17:27:11 PST

The attached scripts (and patch) allow the user to launch Drosera.exe on Windows. Without these changes, Drosera cannot find the necessary libraries and support DLLs to run.

Attachments
Patch adding a new "runDrosera" command to the webkitdirs.pm. (1.60 KB, patch) 2007-12-05 17:28 PST, Brent Fulgham	no flags	Details Formatted Diff Diff
Helper DOS command shell to run Drosera. (274 bytes, text/plain) 2007-12-05 17:29 PST, Brent Fulgham	no flags	Details
Bash script to run Drosera under Windows (1.77 KB, text/plain) 2007-12-05 17:30 PST, Brent Fulgham	no flags	Details
Revised patch to support Windows and Mac OS (1.79 KB, patch) 2007-12-05 21:34 PST, Brent Fulgham	no flags	Details Formatted Diff Diff
Helper script to run Drosera in debug. (2.47 KB, text/plain) 2007-12-05 21:36 PST, Brent Fulgham	no flags	Details
Patch containing updates to support building and running Drosera under win32 and Mac OS X. (7.28 KB, patch) 2007-12-16 18:46 PST, Brent Fulgham	mjs: review+	Details Formatted Diff Diff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.

Brent Fulgham

Comment 1 2007-12-05 17:28:37 PST

Created attachment 17729 [details] Patch adding a new "runDrosera" command to the webkitdirs.pm.

Brent Fulgham

Comment 2 2007-12-05 17:29:16 PST

Created attachment 17730 [details] Helper DOS command shell to run Drosera.

Brent Fulgham

Comment 3 2007-12-05 17:30:06 PST

Created attachment 17731 [details] Bash script to run Drosera under Windows

Brent Fulgham

Comment 4 2007-12-05 17:31:25 PST

The "run-drosera-nightly.cmd" and "run-drosera" scripts need to go in the WebKit/WebKitTools/Scripts directory. The patch patches the webkitdirs.pm to support running Drosera. Ideally, the FindSafari.exe program would have an option to output the environment information without forcing the Safari run command. I'll file a separate bug for that.

Brent Fulgham

Comment 5 2007-12-05 17:51:49 PST

Note: You have to exit from the instance of Safari that opens before Drosera will launch. This is because of the way FindSafari outputs its environment text. See Bug #16315 for a thought on how to work around it.

Brent Fulgham

Comment 6 2007-12-05 21:34:22 PST

Created attachment 17736 [details] Revised patch to support Windows and Mac OS Patch adding a new "runDrosera" command to the webkitdirs.pm module.

Brent Fulgham

Comment 7 2007-12-05 21:36:40 PST

Created attachment 17737 [details] Helper script to run Drosera in debug.

Brent Fulgham

Comment 8 2007-12-16 18:46:06 PST

Created attachment 17949 [details] Patch containing updates to support building and running Drosera under win32 and Mac OS X. This patch combines and replaces all the earlier patches.

Maciej Stachowiak

Comment 9 2007-12-16 19:13:55 PST

Comment on attachment 17949 [details] Patch containing updates to support building and running Drosera under win32 and Mac OS X. r=me

Darin Adler

Comment 10 2008-01-13 11:54:01 PST

Comment on attachment 17949 [details] Patch containing updates to support building and running Drosera under win32 and Mac OS X. There were quite a few problems with this patch. For one thing, the ChangeLog didn't describe what was changed or mention this bug number. For another, the patch to webkitdirs.pm was completely wrong for Mac OS X -- presumably a mismerge.

Darin Adler

Comment 11 2008-01-13 11:54:50 PST

Comment on attachment 17949 [details] Patch containing updates to support building and running Drosera under win32 and Mac OS X. Also it introduced tabs into webkitdirs.pm. But I fixed all of that when landing.

Darin Adler

Comment 12 2008-01-13 11:55:20 PST

Committed revision 29455.

Timothy Hatcher

Comment 13 2008-05-17 09:56:07 PDT

Closing since Drosera has been replaced by the new Web Inspector debugger. Moving to the New Bugs component so the Drosera component can be closed and removed.

Note You need to log in before you can comment on or make changes to this bug.