[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] archive-source: also create a stash for submodu
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH] archive-source: also create a stash for submodules |
Date: |
Mon, 15 Jul 2019 15:36:54 +0100 |
User-agent: |
mu4e 1.3.3; emacs 26.1 |
Marc-André Lureau <address@hidden> writes:
> "git archive" fails when a submodule has a modification, because "git
> stash create" doesn't handle submodules. Let's teach our
> archive-source.sh to handle modifications in submodules the same way
> as qemu tree, by creating a stash.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
Queued to testing/next, thanks.
> ---
> scripts/archive-source.sh | 18 ++++++++++--------
> 1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/scripts/archive-source.sh b/scripts/archive-source.sh
> index ca94e49978..fb5d6b3918 100755
> --- a/scripts/archive-source.sh
> +++ b/scripts/archive-source.sh
> @@ -39,14 +39,16 @@ function cleanup() {
> }
> trap "cleanup" 0 1 2 3 15
>
> -if git diff-index --quiet HEAD -- &>/dev/null
> -then
> - HEAD=HEAD
> -else
> - HEAD=$(git stash create)
> -fi
> +function tree_ish() {
> + local retval='HEAD'
> + if ! git diff-index --quiet --ignore-submodules=all HEAD -- &>/dev/null
> + then
> + retval=$(git stash create)
> + fi
> + echo "$retval"
> +}
>
> -git archive --format tar $HEAD > "$tar_file"
> +git archive --format tar "$(tree_ish)" > "$tar_file"
> test $? -ne 0 && error "failed to archive qemu"
> for sm in $submodules; do
> status="$(git submodule status "$sm")"
> @@ -62,7 +64,7 @@ for sm in $submodules; do
> echo "WARNING: submodule $sm is out of sync"
> ;;
> esac
> - (cd $sm; git archive --format tar --prefix "$sm/" $smhash) > "$sub_file"
> + (cd $sm; git archive --format tar --prefix "$sm/" $(tree_ish)) >
> "$sub_file"
> test $? -ne 0 && error "failed to archive submodule $sm ($smhash)"
> tar --concatenate --file "$tar_file" "$sub_file"
> test $? -ne 0 && error "failed append submodule $sm to $tar_file"
--
Alex Bennée