[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 5/5] make: Make "src-tarball" target actually make a source tarball



George Dunlap writes ("[PATCH 5/5] make: Make "src-tarball" target actually 
make a source tarball"):
> At the moment, making a release tarball is an annoyingly manual
> process that involves running "git archive" into a temporary directory.
...
> +src-tarball: subtree-update
> +     bash ./tools/misc/mktarball $(XEN_ROOT) $$(git describe)

Does this absolutely have to depend on subtree-update ?  Perhaps we
need a new "make the subtrees exist" target ?

> diff --git a/tools/misc/mktarball b/tools/misc/mktarball
...
> +function finish {
> +    [[ -n "$tdir" ]] && rm -rf $tdir
> +}

The use of the POSIX syntax
   finish() {
seems much more prevalent in-tree, than the use of `function'.

> +trap finish EXIT

The EXIT trap handler should probably `set +e'.

> +function git_archive_into {
> +    mkdir "$2"
> +
> +    git --git-dir="$1"/.git \
> +     archive --format=tar HEAD | \
> +     tar Cxf "$2" - 
> +}
> +
> +if [[ -z "$1" || -z "$2" ]] ; then
> +  echo "usage: $0 path-to-XEN_ROOT xen-version"
> +  exit 1
> +fi
> +
> +xen_root="$1"
> +desc="$2"
> +
> +mkdir -p $xen_root/dist/
> +
> +tdir="$(mktemp -d $xen_root/dist/xen.XXXXXXXX)"

Why not use a fixed filename ?  This script isn't safe for concurrent
invocation anyway, because the output filename is (mostly) fixed.

If you do that you can do away with the trap handler entirely.  You
should probably make git_archive_into delete the destination
directory.

> +GZIP=-9v tar cz -f $xen_root/dist/xen-$desc.tar.gz -C $tdir xen-$desc
> +
> +echo "Source tarball in $xen_root/dist/xen-$desc.tar.gz"
> \ No newline at end of file

No newline at end of file.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.