[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