Re: [PATCH v3 2/2] submodule update: continue when a checkout fails

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 9, 2011, 17:21 UTC
Message-ID: <7vlixayl7d.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1307605623-2831-3-git-send-email-iveqy@iveqy.com>

Fredrik Gustafsson <iveqy@iveqy.com> writes:

Show 17 quoted lines

> +				*)
> +					err="${err} \"Failed to $action in submodule path '$path'\""
> ...
> +					err="${err} \"Failed to recurse into submodule path '$path'\""
> +					continue
> +				else
> +					die_with_status $res "Failed to recurse into submodule path '$path'"
> +				fi
> +			fi
>  		fi
>  	done
> +
> +	if test -n "$err"
> +	then
> +		OIFS=$IFS
> +		IFS=';'
> +		for e in $err

Hmm, I do not see anybody concatenating things with semicolons in between... Am I reading a right patch?

Also I am puzzled what the double-quotes around each error message are doing. Are they meant to be shown to the end user?

> +		do
> +			echo $e

Should this go to the standard error output?

> +		done
> +		IFS=$OIFS
> +		die

Do we want to give an extra empty line here with "die"?