Re: [PATCH spice-gtk] Update submodules using the --merge option

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

 



Hi

----- Original Message -----
> From: Christophe de Dinechin <dinechin@xxxxxxxxxx>
> 
> By default, subdmodules will be checked out in detached state.
> This means that you may lose some work in progress.

Lose is a bit strong here.

If you have uncommitted changes, submodule update will fail.

If it's committed, it's in your reflog, and in which case you should have created a branch for your work.

> 
> Using the --merge option will also ensure that if there
> are conflicts between your current submodule and the
> version referenced by the parent, you get an opportunity
> to resolve the conflicts instead of having your changes
> silently wiped out.
> 
> Signed-off-by: Christophe de Dinechin <dinechin@xxxxxxxxxx>
> ---
>  autogen.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/autogen.sh b/autogen.sh
> index cc7bda3..3fbd5b3 100755
> --- a/autogen.sh
> +++ b/autogen.sh
> @@ -7,7 +7,7 @@ test -z "$srcdir" && srcdir=.
>  
>  (
>      cd "$srcdir"
> -    git submodule update --init --recursive
> +    git submodule update --init --recursive --merge

I would rather use --rebase (to avoid accidental push of those update merges).

>      gtkdocize
>      autoreconf -v --force --install
>  )
> --
> 2.11.0 (Apple Git-81)
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]