[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: qemu test-qga failure on mergebuild after VERSION file change: depen
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: qemu test-qga failure on mergebuild after VERSION file change: dependency issues?? |
Date: |
Thu, 16 Jul 2020 14:11:41 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 7/16/20 12:53 PM, Peter Maydell wrote:
> The first merge I tried to process after bumping VERSION for rc0
> failed on test-qga like this:
>
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-qga -m=quick -k --tap < /dev/null | ./scripts/tap-driv
> er.pl --test-name="test-qga"
> PASS 1 test-qga /qga/sync-delimited
> PASS 2 test-qga /qga/sync
> PASS 3 test-qga /qga/ping
> **
> ERROR:/home/petmay01/linaro/qemu-for-merges/tests/test-qga.c:303:test_qga_info:
> assertion failed (version == QEMU_VERSION): ("5.0.9
> 0" == "5.0.50")
> ERROR test-qga - Bail out!
> ERROR:/home/petmay01/linaro/qemu-for-merges/tests/test-qga.c:303:test_qga_info:
> assertion failed (versio
> n == QEMU_VERSION): ("5.0.90" == "5.0.50")
> Aborted (core dumped)
> /home/petmay01/linaro/qemu-for-merges/tests/Makefile.include:659:
> recipe for target 'check-unit' failed
>
> Looking at timestamps on files, tests/test-qga.o never got rebuilt,
> even though config-host.h has been updated (and so has the new
> QEMU_VERSION). Any idea what's gone wrong here?
>
> Also weird: this build tree has no .d files in it.
Yes. We chatted about that yesterday on IRC, I ran this test:
- checkout master
- build after distclean
- checkout mytest
- change a function prototype in qdev-properties.c
(simply because I had it opened in my editor)
- build
make: *** [rules.mak:69: hw/core/qdev-properties.o] Error 1
But the file is still here from the previous 'master' build:
$ ls -ln hw/core/qdev-properties.o
-rw-rw-r--. 1 1000 1000 140624 Jul 15 16:27 hw/core/qdev-properties.o
$ touch hw/core/qdev-properties.o
$ make -> link success... but prototype doesn't match...
>
> thanks
> -- PMM
>