From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:55205) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TtHsQ-00079g-Ex for qemu-devel@nongnu.org; Thu, 10 Jan 2013 08:09:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TtHsO-00044v-M3 for qemu-devel@nongnu.org; Thu, 10 Jan 2013 08:08:58 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58121) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TtHsO-00044p-Ea for qemu-devel@nongnu.org; Thu, 10 Jan 2013 08:08:56 -0500 Message-ID: <50EEBD5F.2000007@redhat.com> Date: Thu, 10 Jan 2013 14:08:47 +0100 From: Gerd Hoffmann MIME-Version: 1.0 References: <20130108000619.9EA7245754@buildbot.b1-systems.de> <20130109085357.GE30476@stefanha-thinkpad.redhat.com> <495746080bc52c7893ee32b86f105d4f.squirrel@weilnetz.de> <20130110124422.GF30946@stefanha-thinkpad.redhat.com> In-Reply-To: <20130110124422.GF30946@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] buildbot failure in qemu on block_mingw32 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Stefan Weil , Daniel Gollub , qemu-devel@nongnu.org, Stefan Weil Hi, >> I don't get this error in line 2097 with my native and cross builds, >> nor do I get the warnings. Which version of MinGW is used on the buildbot? >> >> Anyway, it will be fixed with the pending patch >> http://patchwork.ozlabs.org/patch/210240/ > > I've added Gerd on CC if he has time to check the buildbot. But since > this will be fixed anyway it probably doesn't matter. http://buildbot.b1-systems.de/qemu/buildslaves/kraxel_rhel61 has package version lists. And, yes, mingw64 builds it fine, both 32 and 64 bit versions. While talking about the buildbots (adding Daniel to Cc): kraxel_fedora16 has mingw64 installed and can so both 32bit and 64bit mingw builds. The cross prefixes are "i686-w64-mingw32-" and "x86_64-w64-mingw32-". Both kraxel_fedora16 and kraxel_rhel61 have clang installed, which can be configured like this: clang.addStep(Configure(command=["./configure", "--cc=clang", "--host-cc=clang", "--disable-werror", "--disable-debug-info"], logfiles={"config.log": "config.log"})) Finally, a custom message formater which adds the log tail directly to the fail messages would be cool, so you don't have to dig out the web browser to figure what the error is. /me runs this one: def kraxelMessageFormatter(mode, name, build, results, master_status): result = Results[results] defmsg = mail.defaultMessage(mode, name, build, results, master_status); text = list(); text.append(defmsg['body']); # get log for last step logs = build.getLogs() for log in reversed(logs): if log.getName() == 'stdio': break content = log.getText().splitlines() # Note: can be VERY LARGE url = "%s/steps/%s/logs/%s" % (master_status.getURLForThing(build), log.getStep().getName(), log.getName()) # append log info to standard message text.append("========== log tail ==========") for line in content[-32:]: text.append(unicode(line,'utf8')) text.append("") text.append("========== full log ==========") text.append(url); return { 'body' : "\n".join(text), 'type' : 'plain' } ... hook up this way ... mn = mail.MailNotifier($args, messageFormatter=kraxelMessageFormatter); cheers, Gerd