Bug 186380 - [Xcode] Opt out of the New Build System
Summary: [Xcode] Opt out of the New Build System
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-06-06 19:58 PDT by mitz
Modified: 2018-06-07 10:36 PDT (History)
10 users (show)

See Also:


Attachments
Use the Legacy Build System in the workspace and in command-line builds (2.42 KB, patch)
2018-06-06 20:05 PDT, mitz
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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).