_apache_config_file_name_for_platform always returns a config file name for php5 on Cygwin Python, one for php7 on Win32 Python.
Created attachment 379434 [details] Patch
Created attachment 379437 [details] Patch
Comment on attachment 379437 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379437&action=review > Tools/Scripts/webkitpy/port/base.py:1301 > + conf = self._filesystem.join(prefix, "php%sts.dll" % version) Can we use format() string since % won't work in Python 3 (I recognize that we use % all over this file, I'm working on the migration....I figure it's good not to add more, though)
Created attachment 379522 [details] Patch * Addressed the review feedback.
Comment on attachment 379522 [details] Patch Clearing flags on attachment: 379522 Committed r250330: <https://trac.webkit.org/changeset/250330>
All reviewed patches have been landed. Closing bug.
<rdar://problem/55692444>