Bug 225003 - ANGLE X11 build fixes
Summary: ANGLE X11 build fixes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: ANGLE (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Don Olmstead
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-04-23 15:41 PDT by Don Olmstead
Modified: 2021-04-26 08:05 PDT (History)
8 users (show)

See Also:


Attachments
Patch (1.96 KB, patch)
2021-04-23 15:48 PDT, Don Olmstead
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Don Olmstead 2021-04-23 15:41:43 PDT
See https://bugs.webkit.org/show_bug.cgi?id=223577
Comment 1 Don Olmstead 2021-04-23 15:48:45 PDT
Created attachment 426957 [details]
Patch
Comment 2 Kenneth Russell 2021-04-23 17:56:23 PDT
Comment on attachment 426957 [details]
Patch

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

Seems fine. r+

> Source/WebCore/platform/graphics/angle/ANGLEHeaders.h:52
> +#if defined(None)

Would it be possible to put some guard around this entire block so it only applies to X11 platforms? If not, that's OK.
Comment 3 Don Olmstead 2021-04-23 20:30:24 PDT
(In reply to Kenneth Russell from comment #2)
> Comment on attachment 426957 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=426957&action=review
> 
> Seems fine. r+
> 
> > Source/WebCore/platform/graphics/angle/ANGLEHeaders.h:52
> > +#if defined(None)
> 
> Would it be possible to put some guard around this entire block so it only
> applies to X11 platforms? If not, that's OK.

It seemed to be dragging in X11 headers on Linux even when `angle_use_x11` and `ENABLE_X11_TARGET` were OFF.
Comment 4 Don Olmstead 2021-04-26 08:01:19 PDT
Comment on attachment 426957 [details]
Patch

Talked to mcatanzaro and there doesn't seem to be a good way to guard the set of checks. The other file using it also does the same unconditionally.
Comment 5 EWS 2021-04-26 08:04:49 PDT
Committed r276591 (237027@main): <https://commits.webkit.org/237027@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 426957 [details].
Comment 6 Radar WebKit Bug Importer 2021-04-26 08:05:16 PDT
<rdar://problem/77157618>