diff mbox series

[FFmpeg-devel,PATCHv2] Include slot owner in summary hash

Message ID 20200121101342.14010-1-martin@martin.st
State New
Headers show
Series [FFmpeg-devel,PATCHv2] Include slot owner in summary hash
Related show

Checks

Context Check Description
andriy/ffmpeg-patchwork pending
andriy/ffmpeg-patchwork fail Failed to apply patch

Commit Message

Martin Storsjö Jan. 21, 2020, 10:13 a.m. UTC
From: Mans Rullgard <mans@mansr.com>

Signed-off-by: Martin Storsjö <martin@martin.st>
---
Does this look ok? I removed the [fateserver] disambiguation tag
and added my own Signed-off-by to it.
---
 FATE.pm    | 14 +++++++++++++-
 report.cgi |  8 +-------
 2 files changed, 14 insertions(+), 8 deletions(-)

Comments

Michael Niedermayer Jan. 21, 2020, 10:36 a.m. UTC | #1
On Tue, Jan 21, 2020 at 12:13:42PM +0200, Martin Storsjö wrote:
> From: Mans Rullgard <mans@mansr.com>
> 
> Signed-off-by: Martin Storsjö <martin@martin.st>
> ---
> Does this look ok? I removed the [fateserver] disambiguation tag
> and added my own Signed-off-by to it.

yes, applied

thx

[...]
Martin Storsjö Jan. 21, 2020, 11:10 a.m. UTC | #2
On Tue, 21 Jan 2020, Michael Niedermayer wrote:

> On Tue, Jan 21, 2020 at 12:13:42PM +0200, Martin Storsjö wrote:
>> From: Mans Rullgard <mans@mansr.com>
>>
>> Signed-off-by: Martin Storsjö <martin@martin.st>
>> ---
>> Does this look ok? I removed the [fateserver] disambiguation tag
>> and added my own Signed-off-by to it.
>
> yes, applied

Thanks!

// Martin
diff mbox series

Patch

diff --git a/FATE.pm b/FATE.pm
index 86ffe07..50b5c69 100644
--- a/FATE.pm
+++ b/FATE.pm
@@ -99,6 +99,18 @@  sub split_rec {
     };
 }
 
+sub owner {
+    my ($slot) = @_;
+    my $owner;
+
+    if (open O, "$fatedir/$slot/owner") {
+        chomp($owner = <O>);
+        close O;
+    }
+
+    return $owner;
+}
+
 sub load_summary {
     my ($slot, $date) = @_;
     my $repdir = "$fatedir/$slot/$date";
@@ -109,7 +121,7 @@  sub load_summary {
         my $conf = split_config scalar <S> or return;
         my $st   = split_stats  scalar <S> or return;
         close S;
-        return { %$hdr, %$conf, %$st };
+        return { %$hdr, %$conf, %$st, owner => owner $slot };
     }
 
     return if not -f "$repdir/report.xz";
diff --git a/report.cgi b/report.cgi
index 9ae42fd..a980617 100755
--- a/report.cgi
+++ b/report.cgi
@@ -52,12 +52,6 @@  my $ntest = $npass + $nfail;
 my $rep = load_summary $req_slot, $req_time;
 my $lastpass = load_lastpass $req_slot;
 
-my $owner;
-if (open O, "$slotdir/owner") {
-    chomp($owner = <O>);
-    close O;
-}
-
 # main text
 
 print "Content-type: text/html\r\n";
@@ -100,7 +94,7 @@  trow 'Architecture',  $$conf{arch};
 trow 'Variant',       $$conf{subarch};
 trow 'CPU',           $$conf{cpu};
 trow 'OS',            $$conf{os};
-trow 'Owner',         $owner if $owner;
+trow 'Owner',         $$rep{owner};
 trow 'Compiler',      $$conf{cc};
 trow 'Configuration', '<code>' . $$conf{config} . '</code>';
 trow 'Comment',       $$hdr{comment};