Bug 75153 - Add Production configuration to gtest Xcode project
Summary: Add Production configuration to gtest Xcode project
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords:
Depends on:
Blocks: 66521
  Show dependency tree / graph
 
Reported: 2011-12-22 16:46 PST by David Kilzer (:ddkilzer)
Modified: 2012-01-03 08:25 PST (History)
5 users (show)

See Also:

Attachments
Patch v1 (10.02 KB, patch)
2011-12-22 17:10 PST, David Kilzer (:ddkilzer) 		no flags Details | Formatted Diff | Diff
Patch v2 (11.59 KB, patch)
2011-12-22 21:57 PST, David Kilzer (:ddkilzer) 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2011-12-22 16:46:40 PST 
The fix for Bug 66521 requires a header search path be added to WebCore.framework.  In order to make gtest build properly internally at Apple, we need a separate Production configuration that will allow us to use a different path for WebCore.framework.

Initially, the Production configuration will simply be based on the Release configuration.
Comment 1 David Kilzer (:ddkilzer) 2011-12-22 17:10:41 PST 
Created attachment 120416 [details]
Patch v1
Comment 2 David Kilzer (:ddkilzer) 2011-12-22 21:57:23 PST 
Created attachment 120433 [details]
Patch v2
Comment 3 David Kilzer (:ddkilzer) 2011-12-22 21:58:17 PST 
(In reply to comment #2)
> Created an attachment (id=120433) [details]
> Patch v2

Made "Production" the default configuration for xcodebuild.

Tested Tools/Scripts/build-api-test and it still works as expected.
Comment 4 WebKit Review Bot 2012-01-03 08:25:27 PST 
Comment on attachment 120433 [details]
Patch v2

Clearing flags on attachment: 120433

Committed r103941: <http://trac.webkit.org/changeset/103941>
Comment 5 WebKit Review Bot 2012-01-03 08:25:31 PST 
All reviewed patches have been landed.  Closing bug.