[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [kvm-unit-tests PATCH v6 2/3] run_tests: put logs into
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [kvm-unit-tests PATCH v6 2/3] run_tests: put logs into per-test file |
Date: |
Thu, 12 Jan 2017 22:14:00 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, Jan 12, 2017 at 01:55:48PM +0100, Paolo Bonzini wrote:
[...]
> > diff --git a/scripts/functions.bash b/scripts/common.bash
> > similarity index 75%
> > rename from scripts/functions.bash
> > rename to scripts/common.bash
> > index ee9143c..2dd7360 100644
> > --- a/scripts/functions.bash
> > +++ b/scripts/common.bash
> > @@ -1,3 +1,12 @@
> > +: ${unittest_log_dir:=logs}
> > +
> > +function run_task()
> > +{
> > + local testname="$2"
> > +
> > + RUNTIME_log_file="${unittest_log_dir}/${testname}.log"
> > + "$@"
> > +}
>
> This should go in run_tests.sh since that's the only place that uses
> unittest_log_dir. (The last line then becomes 'run "$@"' and the
> for_each_unittest call must be adjusted). Done and queued.
Yes, this should be cleaner. Thanks,
-- peterx