From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 24 Sep 2012 16:12:05 GMT
Subject: Re: [PATCH] Teach git-completion about git p4
Message-ID: <7vbogv1ju2.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1348264271-99395-1-git-send-email-rcumming@ea.com>

Ryan Cumming <etaoins@gmail.com> writes:

> From: Ryan Cumming <etaoins@gmail.com>
>
> git p4 was moved out of contrib in 1.7.11 but it git-completion didn't
> know about it. Add git p4 completion based on the existing SVN
> completion. It covers all known subcommands and options except for the
> -/ option for clone which doesn't use the standard -- prefix.
>
> Signed-off-by: Ryan Cumming <etaoins@gmail.com>
> ---

Sounds sensible; thanks.

> +_git_p4 ()
> +{
> +...
> +		local submit_opts="
> +			--origin= -M --preserve-user --export-labels
> +			$common_opts
> +			"
> +
> +		case "$subcommand,$cur" in
> ...
> +		submit,--*)
> +			__gitcomp "$submit_opts"

This is taken when $cur begins with --, but $submit_opts includes
"-M" which doesn't begin with it.  Is that a problem?

> +			;;
> +		submit,*)
> +			__gitcomp "$(__git_refs)"
> +			;;
> +		esac
> +	fi
> +}