<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Sep 21, 2017, at 13:25, Michael Catanzaro <<a href="mailto:mcatanzaro@igalia.com" class="">mcatanzaro@igalia.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On Thu, Sep 21, 2017 at 2:26 PM, Tim Horton <<a href="mailto:timothy_horton@apple.com" class="">timothy_horton@apple.com</a>> wrote:<br class=""><blockquote type="cite" class="">Sure. And someday I think we should go down that path, but generating the xcconfigs will be trickier (but also a positive because people already have trouble with them) because they involve more bizarre conditions. But not today.<br class=""></blockquote><br class="">I strongly support removing FeatureList.pm since it's useless for keeping build definitions in sync between XCode and CMake, and it complicates things unnecessarily for both XCode and CMake.<br class=""></div></div></blockquote><div><br class=""></div><div>I don’t intend to remove it <i class="">entirely</i> yet, because it enables the build-webkit arguments to enable/disable features, just the default values. But it’s a start.</div><br class=""><blockquote type="cite" class=""><div class=""><div class="">There is one very minor complication. GTK and WPE currently use the FeatureList.pm features as "developer features" since our bots all use the build-webkit script, so we have some features enabled there that we don't have enabled in WebKitFeatures.cmake. I think right now that is just MSE and WebRTC. This means that we'll need to adjust our buildbot configurations to manually enable these features at the same time that FeatureList.pm is removed to avoid breaking all these tests.<br class=""></div></div></blockquote><div><br class=""></div><div>OK. Is that something that you could prepare a patch for? I think technically that could land <i class="">before</i> the FeatureList.pm change, right? That would be quite helpful.</div><br class=""><blockquote type="cite" class=""><div class=""><div class="">Michael<br class=""><br class=""></div></div></blockquote></div><br class=""></body></html>