Bug 186380

Summary: [Xcode] Opt out of the New Build System
Product: WebKit Reporter: mitz
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, commit-queue, darin, dbates, ews-watchlist, jbedard, krollin, lforschler, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Use the Legacy Build System in the workspace and in command-line builds none

Description mitz 2018-06-06 19:58:56 PDT
In Xcode 10 beta 1, the New Build System is used by default. Until all projects in the stack are building successfully with the New Build System, we should opt out of using it.

For command-line builds, I think this can be done by changing webkitdirs.pm to include -UseNewBuildSystem=NO in the xcodebuild command-line.

For building from within Xcode, I think this can be done by changing the WebKit workspace at the root of the source tree.

People opening individual projects in Xcode, or using their own workspaces, will need to change the project or workspace settings to select the Legacy Build System.
Comment 1 mitz 2018-06-06 20:05:53 PDT
Created attachment 342109 [details]
Use the Legacy Build System in the workspace and in command-line builds
Comment 2 WebKit Commit Bot 2018-06-06 22:09:29 PDT
Comment on attachment 342109 [details]
Use the Legacy Build System in the workspace and in command-line builds

Clearing flags on attachment: 342109

Committed r232573: <https://trac.webkit.org/changeset/232573>
Comment 3 WebKit Commit Bot 2018-06-06 22:09:31 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 Radar WebKit Bug Importer 2018-06-06 22:10:23 PDT
<rdar://problem/40881265>
Comment 5 Darin Adler 2018-06-07 09:13:14 PDT
(In reply to mitz from comment #0)
> In Xcode 10 beta 1, the New Build System is used by default. Until all
> projects in the stack are building successfully with the New Build System,
> we should opt out of using it.

Just curious: Which projects don’t build successfully with it?
Comment 6 Jonathan Bedard 2018-06-07 09:23:11 PDT
(In reply to Darin Adler from comment #5)
> (In reply to mitz from comment #0)
> > In Xcode 10 beta 1, the New Build System is used by default. Until all
> > projects in the stack are building successfully with the New Build System,
> > we should opt out of using it.
> 
> Just curious: Which projects don’t build successfully with it?

WebCore and WebKit are the big ones, but I think WebInspector has issues as well.
Comment 7 Darin Adler 2018-06-07 09:27:17 PDT
(In reply to Jonathan Bedard from comment #6)
> WebCore and WebKit are the big ones, but I think WebInspector has issues as
> well.

Is someone working on that, or are we putting it off?
Comment 8 Jonathan Bedard 2018-06-07 09:35:15 PDT
(In reply to Darin Adler from comment #7)
> (In reply to Jonathan Bedard from comment #6)
> > WebCore and WebKit are the big ones, but I think WebInspector has issues as
> > well.
> 
> Is someone working on that, or are we putting it off?

I think we're still determining who will be doing this. For the time being, we are putting it off, but I don't think we will be able to get away with pushing it back much farther than September or October.
Comment 9 mitz 2018-06-07 09:42:05 PDT
(In reply to Darin Adler from comment #5)
> (In reply to mitz from comment #0)
> > In Xcode 10 beta 1, the New Build System is used by default. Until all
> > projects in the stack are building successfully with the New Build System,
> > we should opt out of using it.
> 
> Just curious: Which projects don’t build successfully with it?

Last I tried, JavaScriptCore was the first project in the stack to fail to build.

I should file a bug here to track that (I want it to be linked to the Apple-internal bug and I was too lazy to find that one last night).