Re: [PATCH kvm-unit-tests 00/12] scripts/mkstandalone: rewrite

[Andrew Jones <drjones@redhat.com>]

On Tue, Jan 19, 2016 at 02:23:59PM +0100, Radim Krčmář wrote:
> Drew and I have shelved series' where mkstandalone was duplicating the
> effort.  This patchset should make future changes much simpler by
> reusing code from other files in standalone tests.

Thanks Radim! This definitely makes mkstandalone much, much more
maintainable. Long-live Bash :-)

I'll rework my return code and timeout support patches on top of this,
and post them soon.

Paolo,

I noticed you wrote 'applied', but I guess you haven't yet pushed.

Thanks,
drew

> 
> 
> Radim Krčmář (12):
>   scripts/mkstandalone: simplify redirection
>   scripts/mkstandalone: fix ARCH
>   scripts/mkstandalone: refactor temp file creation
>   scripts/mkstandalone: use bash in standalone test
>   scripts/mkstandalone: reuse $TEST_DIR/run
>   run_tests: move reusable code to scripts/runtime
>   scripts/runtime: change callsite of $TEST_DIR/run
>   scripts/mkstandalone: refactor check for test kernel
>   scripts/mkstandalone: use scripts/runtime
>   scripts/mkstandalone: simplifiy redirection, again
>   scripts/mkstandalone: sanitize one_kernel handling
>   scripts/mkstandalone: return the exit value of a test
> 
>  run_tests.sh            | 101 +----------------------------
>  scripts/mkstandalone.sh | 164 +++++++++++++++++++-----------------------------
>  scripts/runtime.bash    | 103 ++++++++++++++++++++++++++++++
>  3 files changed, 169 insertions(+), 199 deletions(-)
>  create mode 100644 scripts/runtime.bash
> 
> -- 
> 2.7.0
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html