From: Mateusz Karbowy <mateusz.karbowy@gmail.com>
To: Daniel Shahaf <d.s@daniel.shahaf.name>
Cc: Bart Schaefer <schaefer@brasslantern.com>, zsh-workers@zsh.org
Subject: Re: PATCH: 3.0.8: git completion update for cherry-pick
Date: Sat, 29 Aug 2015 17:53:57 +0100 [thread overview]
Message-ID: <CAFiR=Jt0msq78gUyitXViLO4vRmA1b=WKUs4mTp01DLgLheoGg@mail.gmail.com> (raw)
In-Reply-To: <20150827232856.GF21994@tarsus.local2>
[-- Attachment #1: Type: text/plain, Size: 662 bytes --]
Daniel,
I've fixed the no-quotes issue with $@ and now I'm passing both
compadd and git commit options using one -O parameter (the array names
are separated with colon, as you suggested).
Regards,
Obszczymucha
On 28 August 2015 at 00:28, Daniel Shahaf <d.s@daniel.shahaf.name> wrote:
> Bart Schaefer wrote on Thu, Aug 27, 2015 at 16:19:52 -0700:
>> On Aug 27, 11:11pm, Daniel Shahaf wrote:
>> }
>> } Apart from these two issues, the patch seems ready for commit to me.
>>
>> Let's get this (and 36304) in soon, so PWS can close -test-3 and
>> release 5.1.
>
> I've already pushed 36304.
>
> I don't think this patch needs to delay 5.1; it's not a regression.
[-- Attachment #2: _git.patch.txt --]
[-- Type: text/plain, Size: 3799 bytes --]
diff --git a/Completion/Unix/Command/_git b/Completion/Unix/Command/_git
index b8edc10..ef125e4 100644
--- a/Completion/Unix/Command/_git
+++ b/Completion/Unix/Command/_git
@@ -495,6 +495,8 @@ _git-checkout () {
(( $+functions[_git-cherry-pick] )) ||
_git-cherry-pick () {
+ local -a git_commit_opts
+ git_commit_opts=(--all --not HEAD --not)
_arguments \
'(- :)--quit[end revert or cherry-pick sequence]' \
'(- :)--continue[resume revert or cherry-pick sequence]' \
@@ -511,7 +513,7 @@ _git-cherry-pick () {
'*'{-s,--strategy=}'[use given merge strategy]:merge strategy:__git_merge_strategies' \
'*'{-X,--strategy-option=}'[pass merge-strategy-specific option to merge strategy]' \
'(-e --edit -x -n --no-commit -s --signoff)--ff[fast forward, if possible]' \
- ': :__git_commit_ranges'
+ ': : __git_commit_ranges -O expl:git_commit_opts'
}
(( $+functions[_git-citool] )) ||
@@ -5602,20 +5604,28 @@ __git_remote_branch_names_noprefix () {
(( $+functions[__git_commit_objects_prefer_recent] )) ||
__git_commit_objects_prefer_recent () {
- __git_recent_commits || __git_commit_objects
+ local -a argument_array_names
+ zparseopts -D -E O:=argument_array_names
+
+ __git_recent_commits $argument_array_names || __git_commit_objects
}
(( $+functions[__git_commits] )) ||
__git_commits () {
+ local -a argument_array_names
+ zparseopts -D -E O:=argument_array_names
+ (( $#argument_array_names )) && argument_array_names=( "${(@s/:/)argument_array_names[2]}" ) && set -- "${(@P)argument_array_names[1]}"
+
# TODO: deal with things that __git_heads and __git_tags has in common (i.e.,
# if both exists, they need to be completed to heads/x and tags/x.
- local -a sopts ropt
+ local -a sopts ropt expl
zparseopts -E -a sopts S: r:=ropt R: q
sopts+=( $ropt:q )
+ expl=( "$@" )
_alternative \
"heads::__git_heads $sopts" \
"commit-tags::__git_commit_tags $sopts" \
- 'commit-objects::__git_commit_objects_prefer_recent'
+ 'commit-objects:: __git_commit_objects_prefer_recent -O expl:$argument_array_names[2]'
}
(( $+functions[__git_heads] )) ||
@@ -5670,13 +5680,17 @@ __git_commit_objects () {
(( $+functions[__git_recent_commits] )) ||
__git_recent_commits () {
local gitdir expl start
- declare -a descr tags heads commits
+ declare -a descr tags heads commits argument_array_names commit_opts
local i j k ret
integer distance_from_head
+ zparseopts -D -E O:=argument_array_names
+ (( $#argument_array_names )) && argument_array_names=( "${(@s/:/)argument_array_names[2]}" )
+ (( $#argument_array_names > 1 )) && commit_opts=( "${(@P)argument_array_names[2]}" )
+
# Careful: most %d will expand to the empty string. Quote properly!
# NOTE: we could use %D directly, but it's not available in git 1.9.1 at least.
- commits=("${(f)"$(_call_program commits git --no-pager log -20 --format='%h%n%d%n%s\ \(%cr\)')"}")
+ commits=("${(f)"$(_call_program commits git --no-pager log $commit_opts -20 --format='%h%n%d%n%s\ \(%cr\)')"}")
__git_command_successful $pipestatus || return 1
for i j k in "$commits[@]" ; do
@@ -5759,13 +5773,19 @@ __git_commits2 () {
(( $+functions[__git_commit_ranges] )) ||
__git_commit_ranges () {
- local -a suf
+ local -a argument_array_names
+ zparseopts -D -E O:=argument_array_names
+ (( $#argument_array_names )) && argument_array_names=( "${(@s/:/)argument_array_names[2]}" ) && set -- "${(@P)argument_array_names[1]}"
+
+ local -a expl suf
if compset -P '*..(.|)'; then
- __git_commits $*
+ expl=( $* )
else
compset -S '..*' || suf=( -S .. -r '.@~ ^:\t\n\-' )
- __git_commits $* $suf
+ expl=( $* $suf )
fi
+
+ __git_commits -O expl:$argument_array_names[2]
}
(( $+functions[__git_commit_ranges2] )) ||
next prev parent reply other threads:[~2015-08-29 16:54 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-18 22:19 Mateusz Karbowy
2015-07-22 11:53 ` Daniel Shahaf
2015-08-10 21:31 ` Mateusz Karbowy
2015-08-11 22:06 ` Daniel Shahaf
2015-08-19 17:10 ` Mateusz Karbowy
2015-08-21 19:55 ` Mateusz Karbowy
2015-08-23 6:42 ` Daniel Shahaf
2015-08-25 22:26 ` Mateusz Karbowy
2015-08-27 23:11 ` Daniel Shahaf
2015-08-27 23:19 ` Bart Schaefer
2015-08-27 23:28 ` Daniel Shahaf
2015-08-29 16:53 ` Mateusz Karbowy [this message]
2015-08-30 22:39 ` Daniel Shahaf
2015-08-31 7:50 ` Mateusz Karbowy
2015-09-01 4:19 ` Daniel Shahaf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAFiR=Jt0msq78gUyitXViLO4vRmA1b=WKUs4mTp01DLgLheoGg@mail.gmail.com' \
--to=mateusz.karbowy@gmail.com \
--cc=d.s@daniel.shahaf.name \
--cc=schaefer@brasslantern.com \
--cc=zsh-workers@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).