[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mo
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode |
Date: |
Tue, 22 Dec 2015 16:54:06 +0100 |
On Tue, 22 Dec 2015 17:28:42 +0200
"Michael S. Tsirkin" <address@hidden> wrote:
> On Thu, Dec 10, 2015 at 12:40:55AM +0100, Igor Mammedov wrote:
> > Signed-off-by: Igor Mammedov <address@hidden>
> > ---
> > 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
> >
- [Qemu-devel] [PATCH 00/74] pc: acpi: convert DSDT to AML API and drop ASL templates support, Igor Mammedov, 2015/12/09
- [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode, Igor Mammedov, 2015/12/09
- Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode, Marcel Apfelbaum, 2015/12/10
- Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode, Michael S. Tsirkin, 2015/12/22
- Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode, Michael S. Tsirkin, 2015/12/22
- Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode, Igor Mammedov, 2015/12/22
- Re: [Qemu-devel] [PATCH 01/74] tests: acpi: print ASL diff in verbose mode, Michael S. Tsirkin, 2015/12/22
[Qemu-devel] [PATCH 02/74] acpi: add aml_derefof, Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 05/74] acpi: add aml_mutex(), aml_acquire(), aml_release(), Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 06/74] acpi: add aml_create_qword_field(), Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 07/74] acpi: aml: add helper for Opcode Arg2 Arg2 [Dst] AML pattern, Igor Mammedov, 2015/12/09