[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 11/20] docker: filter out linux-user builds for m
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PULL 11/20] docker: filter out linux-user builds for mingw |
Date: |
Tue, 03 Jul 2018 20:45:23 +0100 |
User-agent: |
mu4e 1.1.0; emacs 26.1.50 |
Paolo Bonzini <address@hidden> writes:
> On 03/07/2018 12:14, Alex Bennée wrote:
>> The recent change from TARGET_DIRS to TARGET_LIST (208ecb3e1) had the
>> effect of defaulting all docker builds to the current configured set
>> of targets. This is actually reasonable behaviour but does run into
>> problems if you have linux-user builds configured and you want to test
>> the windows cross builds. This commit fixes that by adding a
>> DOCKER_FILTER_TARGETS variable which is special-cased for mingw builds
>> so we don't pass the whole set down.
>
>> +# Special cases
>> +# mingw/windows builds cannot build linux-user
>> +docker-%-win32-cross: DOCKER_FILTER_TARGETS = %-linux-user
>> +docker-%-win64-cross: DOCKER_FILTER_TARGETS = %-linux-user
>> address@hidden: DOCKER_FILTER_TARGETS = %-linux-user
>
> Some questions:
>
> 1) Any reason to keep all three?
We could prune but it doesn't hurt.
>
> 2) Can we do it in the test script instead?
>
> 3) DEF_TARGET_LIST in the test script becomes meaningless.
Good point.
>
> 4) You now have to override TARGET_LIST with "make
> TARGET_LIST=foo-softmmu address@hidden". Does this interact
> badly with other uses of TARGET_LIST in the Makefile?
Yeah I originally thought this made sense from the "docker builds the
same as my configured setup" sense but your right that's overkill.
>
> Thanks,
>
> Paolo
--
Alex Bennée
- [Qemu-devel] [PULL 12/20] docker: ubuntu: Update the package list before installing new ones, (continued)
- [Qemu-devel] [PULL 12/20] docker: ubuntu: Update the package list before installing new ones, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 09/20] linux-user: introduce preexit_cleanup, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 19/20] docker: add special rule for deboostrapped images, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 18/20] docker: add special handling for FROM:debian-%-user targets, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 20/20] docker: add linux-user powered cross builder for QEMU, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 14/20] docker: Clean the MXE base image, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 15/20] docker: Do not run tests in 'intermediate' images, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 13/20] docker: ubuntu: Use SDL2, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 11/20] docker: filter out linux-user builds for mingw, Alex Bennée, 2018/07/03
- [Qemu-devel] [PULL 16/20] docker: drop QEMU build-dep from bootstrap, Alex Bennée, 2018/07/03
- Re: [Qemu-devel] [PULL 00/20] Travis, Code Coverage and Cross Build updates, Paolo Bonzini, 2018/07/03
- Re: [Qemu-devel] [PULL 00/20] Travis, Code Coverage and Cross Build updates, Alex Bennée, 2018/07/04