[coreboot-gerrit] Patch merged into coreboot/master: romcc: Rewrite the test system

gerrit at coreboot.org gerrit at coreboot.org
Wed Apr 13 17:37:35 CEST 2016


the following patch was just integrated into master:
commit e69d6c2e7bef9e8821423cb476d1a727b73ca1a4
Author: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
Date:   Wed Apr 13 02:47:24 2016 +0200

    romcc: Rewrite the test system
    
    Differences:
     - The test logic is now only implemented in one place (pending the
       deletion of the old parts), whereas it previously was implemented
       both as make rules and as a pair of shell scripts.
     - Tests don't need to be registered anymore. Just adding a new file
       with the correct name is enough to have it tested.
     - The code is hopefully more readable and maintainable.
     - The new test script supports colors (if the standard output is a
       terminal and --nocolor was not passed on the command line).
    
    Things to do in follow-up patches:
     - Remove the old test code
     - Test or remove fail_test*.c, hello_world*.c and raminit_test*.c
     - Fix regressions that have built up over the years, while making sure
       not to introduce new ones
     - Makefile integration
     - Jenkins integration
    
    There are tests in the makefile that specify -fno-always-inline, but
    this option doesn't exist anymore, so I didn't port them over.
    
    Change-Id: Idd6b89368c1e36555cb880c37bbe07035c938cd7
    Signed-off-by: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
    Reviewed-on: https://review.coreboot.org/14291
    Reviewed-by: Martin Roth <martinroth at google.com>
    Tested-by: build bot (Jenkins)


See https://review.coreboot.org/14291 for details.

-gerrit



More information about the coreboot-gerrit mailing list