From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58775) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WQiOi-000097-KS for qemu-devel@nongnu.org; Thu, 20 Mar 2014 15:13:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WQiOc-0008LE-KJ for qemu-devel@nongnu.org; Thu, 20 Mar 2014 15:13:00 -0400 Received: from v220110690675601.yourvserver.net ([37.221.199.173]:45483) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WQiOc-0008L5-AD for qemu-devel@nongnu.org; Thu, 20 Mar 2014 15:12:54 -0400 Message-ID: <532B3DA8.7070000@weilnetz.de> Date: Thu, 20 Mar 2014 20:12:40 +0100 From: Stefan Weil MIME-Version: 1.0 References: <1395296712-25274-1-git-send-email-sw@weilnetz.de> <532A8AE4.5060304@weilnetz.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL for 2.0 0/5] MinGW related patches List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers Am 20.03.2014 13:45, schrieb Peter Maydell: > On 20 March 2014 06:29, Stefan Weil wrote: >> Am 20.03.2014 07:25, schrieb Stefan Weil: >>> This includes a patch for GTK (needed for MinGW, but not restricted t= o it) >>> and most of the patches which restructure the includes for the Win AP= I. >>> I ommitted the last optional block patch. >>> >>> Stefan >>> >>> [PULL for 2.0 1/5] gtk: Support GTK without VTE >>> [PULL for 2.0 2/5] w32: Add and use intermediate include file for >>> [PULL for 2.0 3/5] w32: Move inline function from header file to C >>> [PULL for 2.0 4/5] w32: Reduce dependencies in sysemu/os-win32.h >>> [PULL for 2.0 5/5] w32: Replace Windows specific data types in >>> >> >> The most important part was missing, sorry: >> >> The following changes since commit 06c1bee85a7def8d0139ee6829728a891ef= e623f: >> >> Merge remote-tracking branch 'remotes/afaerber/tags/prep-for-2.0' in= to >> staging (2014-03-19 23:34:43 +0000) >> >> are available in the git repository at: >> >> >> git://qemu.weilnetz.de/qemu.git qemu-2.0 >=20 > I'm afraid this doesn't build on my mingw32 setup: >=20 > CC util/rfifolock.o > In file included from > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread.h:14, > from > /home/petmay01/linaro/qemu-for-merges/include/qemu/rfifolock.h:17, > from /home/petmay01/linaro/qemu-for-merges/util/rfifol= ock.c:15: > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread-win32.h:8: > error: expected specifier-qualifier-list before =E2=80=98WinHandle=E2=80= =99 > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread-win32.h:18: > error: expected specifier-qualifier-list before =E2=80=98WinLong=E2=80=99 > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread-win32.h:22: > error: expected specifier-qualifier-list before =E2=80=98WinLong=E2=80=99 > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread-win32.h:28: > error: expected specifier-qualifier-list before =E2=80=98WinHandle=E2=80= =99 > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread-win32.h:32: > error: expected specifier-qualifier-list before =E2=80=98WinHandle=E2=80= =99 > /home/petmay01/linaro/qemu-for-merges/include/qemu/thread-win32.h:42: > error: expected =E2=80=98=3D=E2=80=99, =E2=80=98,=E2=80=99, =E2=80=98;=E2= =80=99, =E2=80=98asm=E2=80=99 or =E2=80=98__attribute__=E2=80=99 before > =E2=80=98qemu_thread_get_handle=E2=80=99 > make: *** [util/rfifolock.o] Error 1 >=20 > thanks > -- PMM >=20 Unfortunately a recent commit added new code in util/, and that code needs a modification of patch 5/5. I really should have re-run the compile test :-( Peter, is it possible to pull only patches 1-4? They compile and are fine, and we'd save mail bandwidth if I don't have to send them again. You could also apply only patch 1. It is the only really important one because it adds GTK for Windows users. The other ones can also be applied after QEMU 2.0, if you prefer that. I'll send an updated patch 5, and because it is modified, it will need a new review. Cheers, Stefan