Bug 212428 – Update MobileMiniBrowser project settings to current Xcode

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Kenneth Russell]

Attempting to build the MobileMiniBrowser inside Xcode, after having built with:

Tools/Scripts/build-webkit --release --ios-simulator

and switching the "Run" scheme for MobileMiniBrowser to "Release", fails primarily because the legacy build system is still enabled.

Upgrading the project as recommended by Xcode resulted in some undesirable edits, such as changing the "LastUpgradeCheck" from 1000 to 1140, but there are some which look like they should be checked in.

[Kenneth Russell]

Created attachment 400371 [details]
Patch

[Kenneth Russell]

Without these changes, attempting to build MobileMiniBrowser in Xcode 11.4 yielded over a thousand errors trying to build JavaScriptCore:

---
Build target JavaScriptCore of project JavaScriptCore with configuration Release
Check dependencies
Processing header files with build rules is not supported by the legacy build system.
---

This is a subset of the Xcode project changes that were necessary locally to make this build successfully (had to change the SDK from "macosx.internal" to "macosx"  in a few places), but it seems these should probably be made because WebKit doesn't build in Xcode without them.

[Dean Jackson]

Comment on attachment 400371 [details]
Patch

I think we can do all the changes to the .pbxproj file by editing the Tools/MobileMiniBrowser/Configurations/Base.xcconfig file

Add the lines

CODE_SIGN_IDENTITY = "-";
GCC_WARN_64_TO_32_BIT_CONVERSION = YES;


Hmmm... CODE_SIGN_IDENTITY is already there. I wonder why you needed it.

[Dean Jackson]

I also think the Base.xcconfig should have:

SDKROOT = $(SDKROOT_$(USE_INTERNAL_SDK));
SDKROOT_ = macosx;
SDKROOT_YES = macosx.internal;

[Darin Adler]

Comment on attachment 400371 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=400371&action=review

> Tools/ChangeLog:10
> +        Take some of Xcode's recommended project changes, specifically
> +        enabling code signing, and enabling a couple of warnings and
> +        features.

Since all other build settings are in the .xcconfig file it’s a bit strange to have these in the project file. I know Xcode puts them there.

Especially the GCC_WARN_64_TO_32_BIT_CONVERSION one, which is quite like many others in Base.xcconfig.

[Dean Jackson]

Although that SDKROOT doesn't make sense anyway, since this target is iOS only.

[Dean Jackson]

(In reply to Darin Adler from comment #5)
> Comment on attachment 400371 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=400371&action=review
> 
> > Tools/ChangeLog:10
> > +        Take some of Xcode's recommended project changes, specifically
> > +        enabling code signing, and enabling a couple of warnings and
> > +        features.
> 
> Since all other build settings are in the .xcconfig file it’s a bit strange
> to have these in the project file. I know Xcode puts them there.
> 
> Especially the GCC_WARN_64_TO_32_BIT_CONVERSION one, which is quite like
> many others in Base.xcconfig.

Yeah. I think the important change is in the workspace settings (to use the new build system). The changes in the pbxproj that Xcode provided should all be made in Base.xcconfig.

(Although I'm still confused as to why it changed the CODE_SIGNING_IDENTITY)

[Kenneth Russell]

Created attachment 400378 [details]
Patch

[Kenneth Russell]

Thanks for your suggestions. Revised the patch to modify Base.xcconfig explicitly. It seems to be working - the build's proceeding on my machine.

[EWS]

Committed r262220: <https://trac.webkit.org/changeset/262220>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 400378 [details].

[Radar WebKit Bug Importer]

<rdar://problem/63692812>