|
From: | Thomas Huth |
Subject: | Re: [PATCH-for-5.1 v3 0/2] stubs: Fix notify-event stub linkage error on MinGW |
Date: | Wed, 5 Aug 2020 13:47:48 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 05/08/2020 12.01, Philippe Mathieu-Daudé wrote: > 2 trivial patches to fix the link error reported by Thomas: > > LINK tests/test-timed-average.exe > libqemuutil.a(main-loop.o): In function `qemu_notify_event': > util/main-loop.c:139: multiple definition of `qemu_notify_event' > > tests/test-timed-average.o:/builds/huth/qemu/tests/../stubs/notify-event.c:5: > first defined here > collect2: error: ld returned 1 exit status > rules.mak:124: recipe for target 'tests/test-timed-average.exe' failed > > Since v2: > - Remove incorrectly added abort() in qemu_timer_notify_cb() Works for me (tested here: https://gitlab.com/huth/qemu/-/pipelines/174503835 ) ... but I wonder whether there is a code path that uses the stub from stubs/cpu-get-icount.c but the qemu_notify_event() from util/main-loop.c ? Thomas
[Prev in Thread] | Current Thread | [Next in Thread] |