[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/10] Bluetooth removal, and qtest & misc patches
From: |
Thomas Huth |
Subject: |
Re: [PULL 00/10] Bluetooth removal, and qtest & misc patches |
Date: |
Mon, 16 Dec 2019 13:08:02 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 16/12/2019 11.27, Peter Maydell wrote:
> On Thu, 12 Dec 2019 at 18:54, Thomas Huth <address@hidden> wrote:
>>
>> Hi,
>>
>> the following changes since commit 52901abf94477b400cf88c1f70bb305e690ba2de:
>>
>> Update version for v4.2.0-rc5 release (2019-12-10 17:15:21 +0000)
>>
>> are available in the Git repository at:
>>
>> https://gitlab.com/huth/qemu.git tags/pull-request-2019-12-12
>>
>> for you to fetch changes up to e38f04c4db40a9219cc0c516a6c68b9ca0a783d0:
>>
>> tests: use g_test_rand_int (2019-12-12 08:16:24 +0100)
>>
>> ----------------------------------------------------------------
>> - Removal of the deprecated bluetooth code
>> - Some qtest and misc patches
>> ----------------------------------------------------------------
>
> Hi; this fails to build on some of my build hosts:
>
> OSX:
> [...]
> GEN target/sparc/trace.c
> GEN util/trace.c
> GEN hw/core/trace.c
> CC qga/main.o
> CC qemu-nbd.o
> CC qemu-img.o
> make: *** No rule to make target `config-all-devices.mak', needed by
> `aarch64-softmmu/all'. Stop.
> make: *** Waiting for unfinished jobs....
I unfortunately can't reproduce the issue, but if you've got some spare
minutes, could you please check whether this patch helps:
diff --git a/Makefile b/Makefile
index b437a346d7..23aae1437b 100644
--- a/Makefile
+++ b/Makefile
@@ -365,7 +365,8 @@ ifeq ($(SUBDIR_DEVICES_MAK),)
config-all-devices.mak: config-host.mak
$(call quiet-command,echo '# no devices' > $@,"GEN","$@")
else
-config-all-devices.mak: $(SUBDIR_DEVICES_MAK) config-host.mak
+config-all-devices.mak: $(SUBDIR_DEVICES_MAK) config-host.mak \
+ $(SRC_PATH)/hw/Kconfig
$(call quiet-command, sed -n \
's|^\([^=]*\)=\(.*\)$$|\1:=$$(findstring y,$$(\1)\2)|p' \
$(SUBDIR_DEVICES_MAK) | sort -u > $@, \
?
> ppc64 linux:
> CC chardev/char-mux.o
> CC chardev/char-null.o
> CC chardev/char-parallel.o
> CC chardev/char-pipe.o
> CC chardev/char-pty.o
> CC chardev/char-ringbuf.o
> CC chardev/char-serial.o
> CC chardev/char-socket.o
> CC chardev/char-stdio.o
> make: *** No rule to make target `/home/pm215/qemu/hw/bt/Kconfig',
> needed by `aarch64-softmmu/config-devices.mak'. Stop.
> make: *** Waiting for unfinished jobs....
> CC chardev/char-udp.o
> make: Leaving directory `/home/pm215/qemu/build/all'
>
>
> windows crossbuilds:
> CC chardev/char-serial.o
> CC chardev/char-socket.o
> CC chardev/char-stdio.o
> CC chardev/char-udp.o
> CC chardev/char-win.o
> make: *** No rule to make target
> '/home/petmay01/qemu-for-merges/hw/bt/Kconfig', needed by
> 'aarch64-softmmu/config-devices.mak'. Stop.
> make: *** Waiting for unfinished jobs....
> CC chardev/char-win-stdio.o
These ones are really surprising to me ... *-softmmu/config-devices.mak
should have a proper dependency on hw/Kconfig so they should get
properly rebuild as far as I can tell. Could you please check the
*-softmmu/config-devices.mak.d dependency files whether the dependency
on hw/Kconfig is really there? Hmm, maybe "make" also tries to check the
old dependency to hw/bt/Kconfig first, before regenerating the files ...
not sure how to fix this properly, maybe leave an empty hw/bt/Kconfig
around for a while 'til all build systems have regenerated the
dependency files?
Thomas
- [PULL 02/10] hw/usb: Remove the USB bluetooth dongle device, (continued)
- [PULL 02/10] hw/usb: Remove the USB bluetooth dongle device, Thomas Huth, 2019/12/12
- [PULL 04/10] Remove libbluetooth / bluez from the CI tests, Thomas Huth, 2019/12/12
- [PULL 05/10] tests: fix modules-test 'duplicate test case' error, Thomas Huth, 2019/12/12
- [PULL 06/10] pseries: disable migration-test if /dev/kvm cannot be used, Thomas Huth, 2019/12/12
- [PULL 07/10] hw/misc/ivshmem: Bury dead legacy INTx code, Thomas Huth, 2019/12/12
- [PULL 08/10] glib: use portable g_setenv(), Thomas Huth, 2019/12/12
- [PULL 09/10] tests/Makefile: Fix check-report.* targets shown in check-help, Thomas Huth, 2019/12/12
- [PULL 10/10] tests: use g_test_rand_int, Thomas Huth, 2019/12/12
- [PULL 03/10] Remove the core bluetooth code, Thomas Huth, 2019/12/12
- Re: [PULL 00/10] Bluetooth removal, and qtest & misc patches, Peter Maydell, 2019/12/16
- Re: [PULL 00/10] Bluetooth removal, and qtest & misc patches,
Thomas Huth <=