[iOS] build-dumprendertree does not build ImageDiff
authoraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Oct 2015 00:57:38 +0000 (00:57 +0000)
committeraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Oct 2015 00:57:38 +0000 (00:57 +0000)
https://bugs.webkit.org/show_bug.cgi?id=150152

Reviewed by Tim Horton.

run-webkit-tests attempts to build the tools it requires by calling build-dumprendertree and build-webkittestrunner.
On iOS, build-dumprendertree builds the DumpRenderTree.app target, which does not contain the ImageDiff target
as a dependent. If you haven't built ImageDiff by other means (say, because you built the 'All Source' scheme in
WebKit.xcworkspace), tests that rely on image diffing won't work properly.

There's actually no reason to build the DumpRenderTree.app target on iOS; the default aggregate target works
fine on that platform, and results in ImageDiff being built.

* Scripts/build-dumprendertree:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@191078 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/build-dumprendertree

index eab6c2d..ebaad93 100644 (file)
@@ -1,3 +1,20 @@
+2015-10-14  Andy Estes  <aestes@apple.com>
+
+        [iOS] build-dumprendertree does not build ImageDiff
+        https://bugs.webkit.org/show_bug.cgi?id=150152
+
+        Reviewed by Tim Horton.
+
+        run-webkit-tests attempts to build the tools it requires by calling build-dumprendertree and build-webkittestrunner.
+        On iOS, build-dumprendertree builds the DumpRenderTree.app target, which does not contain the ImageDiff target
+        as a dependent. If you haven't built ImageDiff by other means (say, because you built the 'All Source' scheme in
+        WebKit.xcworkspace), tests that rely on image diffing won't work properly.
+
+        There's actually no reason to build the DumpRenderTree.app target on iOS; the default aggregate target works
+        fine on that platform, and results in ImageDiff being built.
+
+        * Scripts/build-dumprendertree:
+
 2015-10-14  Brent Fulgham  <bfulgham@apple.com>
 
         [Win] Enforce launcher/library naming scheme
index 3e41687..a144055 100755 (executable)
@@ -65,8 +65,7 @@ chdir File::Spec->catdir("Tools", "DumpRenderTree") or die;
 
 my $result;
 if (isAppleMacWebKit()) {
-    my @target = isIOSWebKit() ? ("-target", "DumpRenderTree.app") : ();
-    $result = buildXCodeProject("DumpRenderTree", $clean, XcodeOptions(), (@ARGV, @target));
+    $result = buildXCodeProject("DumpRenderTree", $clean, XcodeOptions(), @ARGV);
 } elsif (isGtk() || isEfl() || isAnyWindows()) {
     # Gtk, EFL, and Windows build everything in one shot. No need to build anything here.
     $result = 0;