From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37536) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xu5Nu-0005bQ-U7 for Qemu-devel@nongnu.org; Thu, 27 Nov 2014 15:09:56 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xu5Np-0007ly-Jg for Qemu-devel@nongnu.org; Thu, 27 Nov 2014 15:09:50 -0500 Received: from mail-r83.rz.uni-mannheim.de ([134.155.96.83]:44523) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xu5Np-0007lT-D1 for Qemu-devel@nongnu.org; Thu, 27 Nov 2014 15:09:45 -0500 Message-ID: <54778506.5060200@weilnetz.de> Date: Thu, 27 Nov 2014 21:09:42 +0100 From: Stefan Weil MIME-Version: 1.0 References: <93259ECD-066A-413E-B647-2BB240DBF7EC@livius.net> <2E7B730D-C409-4800-9BEC-8E302812C7E8@livius.net> In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] MinGW build List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Liviu Ionescu Cc: Peter Maydell , QEMU Developers Am 27.11.2014 um 20:57 schrieb Liviu Ionescu: >=20 > On 27 Nov 2014, at 21:34, Peter Maydell wrot= e: >=20 >> On 27 November 2014 at 16:43, Liviu Ionescu wrote: >>> in qemu-common.h: >>> >>> #ifdef _WIN32 >>> #include "sysemu/os-win32.h" >>> #endif >>> >>> #ifdef __MINGW32__ >>> #include "sysemu/os-win32.h" >>> #endif >> >> Wait, so your mingw toolchain doesn't define _WIN32 ?? >> That seems likely to break a lot of other code that does >> ifdef checks on _WIN32. >=20 > good point!=20 >=20 > I removed the second #ifdef, and the build passed. >=20 > but the countless warnings are still there :-( >=20 >=20 > regards, >=20 > Liviu The countless warnings are "normal" because you use MinGW which is unmaintained since several years. There is a better alternative: use Mingw-w64. It compiles QEMU (*nearly) without warnings, solves the ffs problem and has the additional benefit that it supports both 32 bit and 64 bit builds. Maybe we should drop MinGW support completely... Cheers Stefan *nearly means that there are still warnings for format strings in C++ code. You won't see them if you don't install a C++ compiler, but as you are interested in ARM and the C++ code is needed for the ARM disassembler, you might want it.