Bug 56671 – Add ObjC bindings to the WebCore GYP build

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Adam Barth]

Add ObjC bindings to the WebCore GYP build

[Adam Barth]

Created attachment 86204 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 86204 [details]
Patch

Seems we should have an exclusion list instead of removing them from teh gypi, but OK.

[Adam Barth]

> Seems we should have an exclusion list instead of removing them from teh gypi, but OK.

I think we just blindly run all the IDL files through the ObjC code generator.  Many of those files don't even build.

[Eric Seidel (no email)]

Yes, we do just run them all through.  I just thought it was more gyp-like to have a black list instead of a white list.

[Adam Barth]

(In reply to comment #4)
> Yes, we do just run them all through.  I just thought it was more gyp-like to have a black list instead of a white list.

My thought is that the extra files don't really "exist" in the sense that no one wants to build them.  It probably depends on whether the default policy for new IDL files is to include them in the ObjC bindings.  My sense is "no" given how these files have been treated previously.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 86204 [details]:

webarchive/test-link-rel-icon.html bug 56663 (author: [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 86204 [details]
Patch

Clearing flags on attachment: 86204

Committed r81517: <http://trac.webkit.org/changeset/81517>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.