[webkit-reviews] review denied: [Bug 229080] Build ANGLE as intermediate static library : [Attachment 435495] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Aug 13 12:39:08 PDT 2021
Alex Christensen <achristensen at apple.com> has denied Kimmo Kinnunen
<kkinnunen at apple.com>'s request for review:
Bug 229080: Build ANGLE as intermediate static library
https://bugs.webkit.org/show_bug.cgi?id=229080
Attachment 435495: Patch
https://bugs.webkit.org/attachment.cgi?id=435495&action=review
--- Comment #7 from Alex Christensen <achristensen at apple.com> ---
Comment on attachment 435495
--> https://bugs.webkit.org/attachment.cgi?id=435495
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=435495&action=review
This effectively undoes r276734 which was done for a good reason. I've also
been dealing with and fixing internal fallout from that change for several
months since then. I very very strongly oppose doing this right now, and only
one very strongly oppose doing this in the future. The stated reason has
something to do with unit testing ANGLE. Is there no other way to do that?
Can we not make the dylib able to be used in unit tests instead? That would
have the added benefit of testing a closer configuration to what customers see.
> Source/ThirdParty/ANGLE/ChangeLog:21
> + Remove stale scheme that was left when this work was originally
removed in
> + bug 225162.
This sounds like a good thing to do.
More information about the webkit-reviews
mailing list