results.webkit.org: Support Internal in run-javascriptcore-tests
authorjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Feb 2020 01:47:51 +0000 (01:47 +0000)
committerjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Feb 2020 01:47:51 +0000 (01:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=204095

Rubber-stamped by Aakash Jain.

* Scripts/VCSUtils.pm:
(gitBisectStartBranchForDirectory): Renamed from gitBisectStartBranch.
(gitBranchForDirectory): Compute branch for directory.
(gitBranch): Compute branch for the current git directory.
(commitForDirectory): Add git support.
(gitBisectStartBranch): Renamed to gitBisectStartBranchForDirectory.
* Scripts/run-javascriptcore-tests:
(uploadResults): Check for Internal repository.

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

Tools/ChangeLog
Tools/Scripts/VCSUtils.pm
Tools/Scripts/run-javascriptcore-tests

index 8dedbf1..d29d79f 100644 (file)
@@ -1,3 +1,19 @@
+2020-02-04  Jonathan Bedard  <jbedard@apple.com>
+
+        results.webkit.org: Support Internal in run-javascriptcore-tests
+        https://bugs.webkit.org/show_bug.cgi?id=204095
+
+        Rubber-stamped by Aakash Jain.
+
+        * Scripts/VCSUtils.pm:
+        (gitBisectStartBranchForDirectory): Renamed from gitBisectStartBranch.
+        (gitBranchForDirectory): Compute branch for directory.
+        (gitBranch): Compute branch for the current git directory.
+        (commitForDirectory): Add git support.
+        (gitBisectStartBranch): Renamed to gitBisectStartBranchForDirectory.
+        * Scripts/run-javascriptcore-tests:
+        (uploadResults): Check for Internal repository.
+
 2020-02-04  Alex Christensen  <achristensen@webkit.org>
 
         REGRESSION: (r255611) [ Mac ] 3 lldb tests failing related to HashMap
index c3fa424..486d01f 100644 (file)
@@ -62,6 +62,7 @@ BEGIN {
         &fixChangeLogPatch
         &fixSVNPatchForAdditionWithHistory
         &gitBranch
+        &gitBranchForDirectory
         &gitCommitForSVNRevision
         &gitDirectory
         &gitHashForDirectory
@@ -108,7 +109,6 @@ BEGIN {
 
 our @EXPORT_OK;
 
-my $gitBranch;
 my $gitRoot;
 my $isGit;
 my $isGitSVN;
@@ -287,9 +287,10 @@ sub gitTreeDirectory()
     return $result;
 }
 
-sub gitBisectStartBranch()
+sub gitBisectStartBranchForDirectory($)
 {
-    my $bisectStartFile = File::Spec->catfile(gitDirectory(), "BISECT_START");
+    my ($directory) = @_;
+    my $bisectStartFile = File::Spec->catfile($directory, "BISECT_START");
     if (!-f $bisectStartFile) {
         return "";
     }
@@ -299,22 +300,28 @@ sub gitBisectStartBranch()
     return $result;
 }
 
-sub gitBranch()
+sub gitBranchForDirectory($)
 {
-    unless (defined $gitBranch) {
-        chomp($gitBranch = `git symbolic-ref -q HEAD`);
-        my $hasDetachedHead = exitStatus($?);
-        if ($hasDetachedHead) {
-            # We may be in a git bisect session.
-            $gitBranch = gitBisectStartBranch();
-        }
-        $gitBranch =~ s#^refs/heads/##;
-        $gitBranch = "" if $gitBranch eq "master";
+    my ($directory) = @_;
+
+    my $gitBranch;
+    chomp($gitBranch = `git -C \"$directory\" symbolic-ref -q HEAD`);
+    my $hasDetachedHead = exitStatus($?);
+    if ($hasDetachedHead) {
+        # We may be in a git bisect session.
+        $gitBranch = gitBisectStartBranchForDirectory($directory);
     }
+    $gitBranch =~ s#^refs/heads/##;
+    $gitBranch = "" if $gitBranch eq "master";
 
     return $gitBranch;
 }
 
+sub gitBranch()
+{
+    return gitBranchForDirectory(gitDirectory());
+}
+
 sub isGitBranchBuild()
 {
     my $branch = gitBranch();
@@ -385,6 +392,10 @@ sub commitForDirectory($$)
         $info =~ /.*Relative URL: \^\/([a-z]+)\n.*/;
         my $branch = $1;
         $result->{branch} = $branch if ($branch ne 'trunk');
+    } elsif (isGitDirectory($directory)) {
+        $result->{id} = gitHashForDirectory($directory);
+        my $branch = gitBranchForDirectory($directory);
+        $result->{branch} = $branch if ($branch ne '');
     } else {
         die "$directory is not a recognized SCM";
     }
index 5928282..c4cd136 100755 (executable)
@@ -878,11 +878,18 @@ sub uploadResults
         return 0;
     }
 
+    my @commits = [];
+    my $internalCheckout = File::Spec->catdir(dirname(sourceDir()), "Internal");
+    if (-e $internalCheckout and -d $internalCheckout) {
+        @commits = [commitForDirectory(sourceDir(), 'webkit'), commitForDirectory($internalCheckout, 'safari')]
+    } else {
+        @commits = [commitForDirectory(sourceDir(), 'webkit')]
+    }
+
     my %upload = (
         version => 0,
         suite => 'javascriptcore-tests',
-        # FIXME: Integrate the Safari repository, https://bugs.webkit.org/show_bug.cgi?id=204095
-        commits => [commitForDirectory(sourceDir(), 'webkit')],
+        commits => @commits,
         configuration => configurationForUpload(),
         test_results => {
             run_stats => {