From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38882) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aBPGM-0000EU-Hl for qemu-devel@nongnu.org; Tue, 22 Dec 2015 10:54:11 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aBPGL-0007la-Kg for qemu-devel@nongnu.org; Tue, 22 Dec 2015 10:54:10 -0500 Received: from mx1.redhat.com ([209.132.183.28]:49565) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aBPGL-0007lW-FE for qemu-devel@nongnu.org; Tue, 22 Dec 2015 10:54:09 -0500 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 016A6345B32 for ; Tue, 22 Dec 2015 15:54:08 +0000 (UTC) Date: Tue, 22 Dec 2015 16:54:06 +0100 From: Igor Mammedov Message-ID: <20151222165406.14e8af8a@nial.brq.redhat.com> In-Reply-To: <20151222172647-mutt-send-email-mst@redhat.com> References: <1449704528-289297-1-git-send-email-imammedo@redhat.com> <1449704528-289297-2-git-send-email-imammedo@redhat.com> <20151222172647-mutt-send-email-mst@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: qemu-devel@nongnu.org On Tue, 22 Dec 2015 17:28:42 +0200 "Michael S. Tsirkin" wrote: > On Thu, Dec 10, 2015 at 12:40:55AM +0100, Igor Mammedov wrote: > > Signed-off-by: Igor Mammedov > > --- > > tests/bios-tables-test.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c > > index 6d37332..50678b5 100644 > > --- a/tests/bios-tables-test.c > > +++ b/tests/bios-tables-test.c > > @@ -580,6 +580,13 @@ static void test_acpi_asl(test_data *data) > > (gchar *)&signature, > > sdt->asl_file, sdt->aml_file, > > exp_sdt->asl_file, exp_sdt->aml_file); > > + if (getenv("V")) { > > How do you test? make check V=1? yep make V=1 check > > > + int ret G_GNUC_UNUSED; > > + char *diff = g_strdup_printf("git diff --exit-code %s %s", > > + exp_sdt->asl_file, sdt->asl_file); > > This assumes running under git which is generally wrong e.g. > people build from tarballs. > git is also not a build dependency. usually only developers use V=1 as they care about verbose output and they probably use/have git so it doesn't have to builddep, if they don't they won't get any useful output here. my choice of 'git diff' was dictated by the fact that it produces nice colored diff of ASL while usual diff doesn't. And that helps a lot (to me) as that diff is intended for human consumption. > > > > + ret = system(diff) ; > > + g_free(diff); > > + } > > } > > } > > g_string_free(asl, true); > > -- > > 1.8.3.1 > >