patches: Display multiple check marks for multiple signoffs.

André Hentschel nerv at dawncrow.de
Sun Jan 29 14:24:20 CST 2017


Hi,

> --- a/patches/update
> +++ b/patches/update
> @@ -229,7 +229,7 @@ foreach my $file (sort { $patches{$b}->{"order"} <=> $patches{$a}->{"order"} } k
>      }
>      if (defined($patch->{"signoff"}))
>      {
> -        printf INDEX "<td class=\"checkmark\" onclick=\"toggle_display('$file.signoff')\">✓</td>";
> +        printf INDEX "<td class=\"checkmark\" onclick=\"toggle_display('$file.signoff')\">%s</td>", "✓" x @{$patch->{"signoff"}};
>      }


The patch is definitly an improvement, but the problem with the patch is, that right now I can't see that my patch has been signed-off by Aric when looking at it in the patch list
...
129264	New	Aric Stewart	Re: usp10: Fix ScriptRecordDigitSubstitution spec file entry		✓
...
129183	Assigned	André Hentschel	usp10: Fix ScriptRecordDigitSubstitution spec file entry	Aric Stewart	✓
...

I think something like that was what Bruno is looking for



More information about the wine-devel mailing list