[FFmpeg-devel] [FATESERVER/PATCH v2 0/8] Categorization improvements
Timothy Gu
timothygu99 at gmail.com
Wed Jun 11 03:07:46 CEST 2014
New version of the patch, taking account of Derek's comments
Timothy Gu (8):
fate.css: wrap long lines in log
Unchanged
index: use one parameter "sort" for both ascending and descending
sorting
Changed to use /^sort/
index: move the declaration of $sort to the beginning of the file
New patch, mostly for cosmetics
FATE.pm: use $ntests instead of $ntest for consistency
Ditto
Move $nfail calculation to FATE.pm
Makes it possible to use `/?sort=descnfail`
index: allow multiple sorting criteria
index: lsort(): add support for sorting with '//'
Should these two patches be stashed together? The first one
permits requests like `/?sort=descstatus//descnfail`, while the
second one permits `descstatus//descnfail` to be a default value.
index: improve sorting
This patch depends on the "[PATCH] tests/fate.sh: report different
status for different errors" patch for it to work.
As explained in the commit message, if only $nfail is used, build
failures will be as behind as all tests passes; if only $status
is used, 1 test failed will be as behind as 1 test passed, and
Derek is gonna get mad.
The changes try to address Derek's concern, but whether of the two
orders: `new failed -> old failed -> new OK` or
`new failed -> new OK -> old failed`
is debatable. Currently, the patch uses the former algorithm.
FATE.pm | 8 +++++---
fate.css | 2 +-
index.cgi | 57 +++++++++++++++++++++++++++++++++++++--------------------
3 files changed, 43 insertions(+), 24 deletions(-)
--
1.9.1
More information about the ffmpeg-devel
mailing list