From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:40688) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S1KrB-0005Kq-Iy for qemu-devel@nongnu.org; Sat, 25 Feb 2012 11:52:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S1Kr6-0007J4-83 for qemu-devel@nongnu.org; Sat, 25 Feb 2012 11:52:25 -0500 Received: from cantor2.suse.de ([195.135.220.15]:38579 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S1Kr5-0007J0-Uv for qemu-devel@nongnu.org; Sat, 25 Feb 2012 11:52:20 -0500 Message-ID: <4F4911C1.4020009@suse.de> Date: Sat, 25 Feb 2012 17:52:17 +0100 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1329940095-378-1-git-send-email-sw@weilnetz.de> <4F490CCF.40101@suse.de> <4F490EA7.3000702@weilnetz.de> In-Reply-To: <4F490EA7.3000702@weilnetz.de> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v2] w32: Support tests (make check) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Weil Cc: Anthony Liguori , qemu-devel@nongnu.org Am 25.02.2012 17:39, schrieb Stefan Weil: > Am 25.02.2012 17:31, schrieb Andreas F=E4rber: >> Am 22.02.2012 20:48, schrieb Stefan Weil: >>> Adding $(EXESUF) is needed to make those tests work on w32 hosts, too= . >>> >>> v2: >>> Rebased, added new tests, tests sorted alphabetically. >>> >>> Signed-off-by: Stefan Weil >>> --- >>> tests/Makefile | 38 +++++++++++++++++++++++--------------- >>> 1 files changed, 23 insertions(+), 15 deletions(-) >>> >>> diff --git a/tests/Makefile b/tests/Makefile >>> index 74b29dc..09f2b13 100644 >>> --- a/tests/Makefile >>> +++ b/tests/Makefile >>> @@ -1,16 +1,24 @@ >>> -CHECKS =3D check-qdict check-qfloat check-qint check-qstring check-q= list >>> -CHECKS +=3D check-qjson test-qmp-output-visitor test-qmp-input-visit= or >>> -CHECKS +=3D test-string-input-visitor test-string-output-visitor >>> test-coroutine >>> +CHECKS =3D check-qdict$(EXESUF) >>> +CHECKS +=3D check-qfloat$(EXESUF) >>> +CHECKS +=3D check-qint$(EXESUF) >>> +CHECKS +=3D check-qjson$(EXESUF) >>> +CHECKS +=3D check-qlist$(EXESUF) >>> +CHECKS +=3D check-qstring$(EXESUF) >>> +CHECKS +=3D test-coroutine$(EXESUF) >>> +CHECKS +=3D test-qmp-input-visitor$(EXESUF) >>> +CHECKS +=3D test-qmp-output-visitor$(EXESUF) >>> +CHECKS +=3D test-string-input-visitor$(EXESUF) >>> +CHECKS +=3D test-string-output-visitor$(EXESUF) >> >> Cute. (one-per-line allows to easily comment individual ones out) >> >> Some lines below were already way over 80 chars, so >> >> Reviewed-by: Andreas F=E4rber >> >> Andreas >=20 > And as long as they remain sorted, they also reduce the risk > of merge conflicts: >=20 > When everybody just adds to the end,two different commits > will always conflict. >=20 > They won't conflictif both commits add new lines at different > locations. I'd rather not strictly keep it in alphabetical order since having one test pass might be a prerequisite for another test to be meaningful (e.g., qfloat before pi). IMO we should have groups of tests, in which we can order them alphabetically. Anyway, my line of thinking was that no such interdependency is documented here so it should be permissible to rearrange the order now. The q* and the visitors are still together, input before output, so I like it. We could add lines before test-coroutine and test-qmp-input-visitors to group them by topic if we wanted, but the list is still small, so no real need. Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg