[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh |
Date: |
Sat, 9 Sep 2017 13:07:12 +0100 |
On 9 September 2017 at 06:45, Fam Zheng <address@hidden> wrote:
> Signed-off-by: Fam Zheng <address@hidden>
> Message-Id: <address@hidden>
> ---
> scripts/archive-source.sh | 31 +++++++++++++++++++++++++++++++
> 1 file changed, 31 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..1de369532e
> --- /dev/null
> +++ b/scripts/archive-source.sh
> @@ -0,0 +1,31 @@
> +#!/bin/sh
> +#
> +# Author: Fam Zheng <address@hidden>
> +#
> +# Create archive of source tree, including submodules
> +#
> +# This code is licensed under the GPL version 2 or later. See
> +# the COPYING file in the top-level directory.
So is this the script that for instance Mike Roth would use
to create the release tarballs? If it isn't, should it be?
Is it intended for end users to create tarballs with, or
is it really just a helper script for the docker stuff?
If the latter, it would be helpful to say so. If the former,
it could really use more usage information/documentation...
> +
> +set -e
> +
> +if test $# -lt 1; then
> + echo "Usage: $0 <output>"
> + exit 1
> +fi
> +
> +submodules=$(git submodule foreach --recursive --quiet 'echo $name')
> +
> +if test -n "$submodules"; then
> + {
> + git ls-files
> + for sm in $submodules; do
> + (cd $sm; git ls-files) | sed "s:^:$sm/:"
> + done
> + } | grep -x -v $(for sm in $submodules; do echo "-e $sm"; done) > $1.list
Supporting '-e something' in a tar -T listfile seems to
be GNU tar specific. Do we care?
> +else
> + git ls-files > $1.list
> +fi
This will blow up if we ever have a file in the repo
that starts with a '-'. Do we care?
> +
> +tar -cf $1 -T $1.list
> +rm $1.list
This is missing a lot of quoting for $1, so it will go wrong
if there's a space in that filename argument.
thanks
-- PMM
- [Qemu-devel] [PULL v2 02/37] docker: docker.py make --no-cache skip checksum test, (continued)
- [Qemu-devel] [PULL v2 02/37] docker: docker.py make --no-cache skip checksum test, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 04/37] docker: reduce noise when building travis.docker, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 01/37] docker: ensure NOUSER for travis images, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 03/37] docker: don't install device-tree-compiler build-deps in travis.docker, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 05/37] docker: Update ubuntu image, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 06/37] docker: Enable features explicitly in test-full, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 07/37] tests/docker: Clean up paths, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 08/37] gitignore: Ignore vm test images, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 09/37] qemu.py: Add "wait()" method, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh, Fam Zheng, 2017/09/09
- Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh,
Peter Maydell <=
- Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh, Peter Maydell, 2017/09/09
[Qemu-devel] [PULL v2 11/37] tests: Add vm test lib, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 12/37] tests: Add ubuntu.i386 image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 13/37] tests: Add FreeBSD image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 14/37] tests: Add NetBSD image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 16/37] Makefile: Add rules to run vm tests, Fam Zheng, 2017/09/09