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
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
