[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 03/13] scripts: Add archive-source.sh
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v11 03/13] scripts: Add archive-source.sh |
Date: |
Fri, 22 Sep 2017 10:42:35 +0800 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Thu, 09/21 10:22, Eric Blake wrote:
> On 09/21/2017 10:07 AM, Fam Zheng wrote:
> > Signed-off-by: Fam Zheng <address@hidden>
> > ---
> > scripts/archive-source.sh | 51
> > +++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 51 insertions(+)
> > create mode 100755 scripts/archive-source.sh
> >
> > diff --git a/scripts/archive-source.sh b/scripts/archive-source.sh
> > new file mode 100755
> > index 0000000000..8aa363d3a0
> > --- /dev/null
> > +++ b/scripts/archive-source.sh
> > @@ -0,0 +1,51 @@
> > +#!/bin/bash
> > +#
> > +# Author: Fam Zheng <address@hidden>
> > +#
> > +# Archive source tree, including submodules. This is created for test code
> > to
> > +# export the source files, in order to be built in a different enviornment,
>
> s/enviornment/environment/
>
> > +# such as in a docker instance or VM.
> > +#
> > +# This code is licensed under the GPL version 2 or later. See
> > +# the COPYING file in the top-level directory.
> > +
> > +error() {
> > + echo "$@" >&2
>
> slightly risky if $@ contains \ - using 'printf %s\\n "$*"' is nicer.
> But this particular script can get away with it, as I don't see use of \
> (other than if you have a weird $0).
>
> > +
> > +if test -n "$submodules"; then
> > + {
> > + git ls-files || error "git ls-files failed"
> > + for sm in $submodules; do
> > + (cd $sm; git ls-files) | sed "s:^:$sm/:"
> > + if test "${PIPESTATUS[0]} ${PIPESTATUS[1]}" != "0 0"; then
>
> Could shorten to "${PIPESTATUS[*]}", as discussed in another mail, but
> this works too.
>
> With the typo fixed, and your choice of how to handle my other two nits
> (including doing nothing),
> Reviewed-by: Eric Blake <address@hidden>
Fixing all of them and adding your r-b line.
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org
>
Fam
- [Qemu-devel] [PATCH v11 00/13] tests: Add VM based build tests (for non-x86_64 and/or non-Linux), Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 01/13] gitignore: Ignore vm test images, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 02/13] qemu.py: Add "wait()" method, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 03/13] scripts: Add archive-source.sh, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 04/13] tests: Add a test key pair, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 05/13] tests: Add vm test lib, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 06/13] tests: Add ubuntu.i386 image, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 07/13] tests: Add FreeBSD image, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 08/13] tests: Add NetBSD image, Fam Zheng, 2017/09/21
- [Qemu-devel] [PATCH v11 09/13] tests: Add OpenBSD image, Fam Zheng, 2017/09/21