Bug 18077

Summary:

Integrate testapi.c into the Windows build

Product:

WebKit

Reporter:

Adam Roben (:aroben) <aroben>

Component:

Tools / Tests

Assignee:

Adam Roben (:aroben) <aroben>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

Windows XP

Attachments:

Description	Flags
Step 1: Get testapi.c compiling under MSVC	aroben: review+
Step 2: Add testapi to the build	none
Step 2 (updated): Add testapi to the build	aroben: review+

Adam Roben (:aroben)

Reported 2008-03-25 14:24:29 PDT

On Windows we never build testapi.c. It would be good to start building this so that we can test the JSC API.

Attachments
Step 1: Get testapi.c compiling under MSVC (5.62 KB, patch) 2008-03-25 15:30 PDT, Adam Roben (:aroben)	aroben: review+	Details Formatted Diff Diff
Step 2: Add testapi to the build (28.95 KB, patch) 2008-03-25 15:41 PDT, Adam Roben (:aroben)	no flags	Details Formatted Diff Diff
Step 2 (updated): Add testapi to the build (13.15 KB, patch) 2008-03-25 15:47 PDT, Adam Roben (:aroben)	aroben: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Adam Roben (:aroben)

Comment 1 2008-03-25 15:30:14 PDT

Created attachment 20046 [details] Step 1: Get testapi.c compiling under MSVC

Adam Roben (:aroben)

Comment 2 2008-03-25 15:41:18 PDT

Created attachment 20050 [details] Step 2: Add testapi to the build

Adam Roben (:aroben)

Comment 3 2008-03-25 15:47:58 PDT

Created attachment 20052 [details] Step 2 (updated): Add testapi to the build This patch ensures that our build order remains linear.

Adam Roben (:aroben)

Comment 4 2008-03-25 16:17:36 PDT

Comment on attachment 20046 [details] Step 1: Get testapi.c compiling under MSVC r=sfalken

Adam Roben (:aroben)

Comment 5 2008-03-25 16:17:44 PDT

Comment on attachment 20052 [details] Step 2 (updated): Add testapi to the build r=sfalken

Adam Roben (:aroben)

Comment 6 2008-03-25 16:26:06 PDT

Committed in r31296 and r31297.

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