SSIA
Created attachment 199775 [details] Patch
Comment on attachment 199775 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199775&action=review > Tools/Scripts/update-webkit:58 > 'gyp' => \$useGYP, Don't you need to remove 'gyp' as well ? As far as I know, gyp was for chromium
It seems that isAppleMacWebKit is using it. elsif (isAppleMacWebKit()) { my @local_options = @options; push @local_options, XcodeCoverageSupportOptions() if $coverageSupport && $project ne "ANGLE"; my $useGYPProject = $useGYP && ($project =~ "WebCore|JavaScriptCore"); my $projectPath = $useGYPProject ? "gyp/$project" : $project; $projectPath = $project =~ /gtest/ ? "xcode/gtest" : $project; $result = buildXCodeProject($projectPath, $clean, @local_options, @ARGV); }
Comment on attachment 199775 [details] Patch Clearing flags on attachment: 199775 Committed r149166: <http://trac.webkit.org/changeset/149166>
All reviewed patches have been landed. Closing bug.