On 9/16/19 10:43 AM, Alex Bennée wrote: > Philippe Mathieu-Daudé <f4...@amsat.org> writes: > >> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> >> --- >> .travis.yml | 18 ++++++++++++++++-- >> 1 file changed, 16 insertions(+), 2 deletions(-) >> >> diff --git a/.travis.yml b/.travis.yml >> index 69a37f7387..753276eb33 100644 >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -265,9 +265,23 @@ matrix: >> - "3.6" >> >> >> - # Acceptance (Functional) tests >> + # Acceptance (Functional) tests [enterprise] >> - env: >> - - CONFIG="--python=/usr/bin/python3 >> --target-list=x86_64-softmmu,mips-softmmu,mips64el-softmmu,aarch64-softmmu,arm-softmmu,s390x-softmmu,alpha-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu" >> + - CONFIG="--python=/usr/bin/python3 >> --target-list=x86_64-softmmu,mips64el-softmmu,aarch64-softmmu,s390x-softmmu,ppc64-softmmu" > > You could use: > > --target-list=${MAIN_SOFTMMU_TARGETS} > > and > > --disable-user --target-list-exclude=${MAIN_SOFTMMU_TARGETS}
I like the idea, but this variable is slighly different: MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu" Are you suggesting we use MAIN_SOFTMMU_TARGETS for 'Enterprise' targets? As long as mips64-softmmu is present, I think we can remove mips-softmmu from MAIN_TARGETS. Maybe i386-softmmu can be removed too. That would give the following hobbyist list: i386-softmmu,mips-softmmu,alpha-softmmu,ppc-softmmu,m68k-softmmu (I plan to add hppa-softmmu there too). >> + - TEST_CMD="make check-acceptance" >> + after_failure: >> + - cat tests/results/latest/job.log >> + addons: >> + apt: >> + packages: >> + - python3-pil >> + - python3-pip >> + - python3.5-venv >> + >> + >> + # Acceptance (Functional) tests [hobbyist] >> + - env: >> + - CONFIG="--python=/usr/bin/python3 >> --target-list=mips-softmmu,arm-softmmu,alpha-softmmu,ppc-softmmu,m68k-softmmu" >> - TEST_CMD="make check-acceptance" >> after_failure: >> - cat tests/results/latest/job.log > > > -- > Alex Bennée >