From e69d6c2e7bef9e8821423cb476d1a727b73ca1a4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jonathan=20Neusch=C3=A4fer?= Date: Wed, 13 Apr 2016 02:47:24 +0200 Subject: romcc: Rewrite the test system MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Reviewed-on: https://review.coreboot.org/14291 Reviewed-by: Martin Roth Tested-by: build bot (Jenkins) --- .gitignore | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to '.gitignore') diff --git a/.gitignore b/.gitignore index 912aa85eb2..07a48358a3 100644 --- a/.gitignore +++ b/.gitignore @@ -96,8 +96,7 @@ util/nvramtool/.dependencies util/nvramtool/nvramtool util/optionlist/Options.wiki util/romcc/romcc -util/romcc/tests/fail_test*.S -util/romcc/tests/*.S-O2-mmmx +util/romcc/build util/runfw/googlesnow util/superiotool/superiotool util/vgabios/testbios -- cgit v1.2.3