From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59718) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YSg6Z-0000nd-6s for qemu-devel@nongnu.org; Tue, 03 Mar 2015 01:14:56 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YSg6V-0003xR-Bo for qemu-devel@nongnu.org; Tue, 03 Mar 2015 01:14:55 -0500 Received: from v220110690675601.yourvserver.net ([37.221.199.173]:47960) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YSg6V-0003xK-6B for qemu-devel@nongnu.org; Tue, 03 Mar 2015 01:14:51 -0500 Message-ID: <54F55155.80103@weilnetz.de> Date: Tue, 03 Mar 2015 07:14:45 +0100 From: Stefan Weil MIME-Version: 1.0 References: <20150228064712.GA15109@ad.nay.redhat.com> In-Reply-To: <20150228064712.GA15109@ad.nay.redhat.com> Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] Announcement: Patchew server is online now List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng , qemu-devel@nongnu.org Am 28.02.2015 um 07:47 schrieb Fam Zheng: > Hello, everyone > > I'm glad to announce the fresh Patchew server deployment: > > http://qemu.patchew.org/ > > The mission of this project is to help identify obvious defects (coding style, > compiling, etc.) of posted patches. > > Follow the green "passed" or red "failed" button in the page and you'll see the > testing log. You can also see which series have got reviews or comments. > > Q: How will this impact patch submission and merging process? > > Nothing is affected (at least for now), use this as an auxiliary. > > Q: What exactly are tested for each patch? > > * scripts/checkpatch.pl > * git am (on top of qemu.git) Obviously checkpatch.pl reports false positives because it tries to analyze quoted-printable UTF-8 e-mails. Here an example: http://qemu.patchew.org/testing/log/%3C1425214326-25326-1-git-send-email-sw@weilnetz.de%3E Here is the message from Patchew: <<< WARNING >>>ERROR: spaces required around that '=' (ctx:WxV) <<< WARNING >>>#96: FILE: util/oslib-posix.c:404: <<< WARNING >>>+ for (i =3D 0; i < numpages; i++) { <<< WARNING >>> ^ It's wrong because "=3D" is simply a quoted "=", and it is surrounded by spaces here. Regards Stefan