[webkit-reviews] review granted: [Bug 190537] [Tools][webkitpy] fix handling of JSCTESTS_OPTIONS : [Attachment 352204] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 12 15:09:20 PDT 2018


Alexey Proskuryakov <ap at webkit.org> has granted Guillaume Emont
<guijemont at igalia.com>'s request for review:
Bug 190537: [Tools][webkitpy] fix handling of JSCTESTS_OPTIONS
https://bugs.webkit.org/show_bug.cgi?id=190537

Attachment 352204: Patch

https://bugs.webkit.org/attachment.cgi?id=352204&action=review




--- Comment #4 from Alexey Proskuryakov <ap at webkit.org> ---
Comment on attachment 352204
  --> https://bugs.webkit.org/attachment.cgi?id=352204
Patch

The patch looks good.

I’m not so sure about the idea of passing options via environment. The port
object is supposed to implement the knowledge of how to run tools, and passing
arguments verbatim violates incapsulation. 

Why was this approach chosen?


More information about the webkit-reviews mailing list