Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Bugfix #19

Merged
merged 1 commit into from

2 participants

@jimparis

No description provided.

@kislyuk
Owner

Thanks, but I'm confused. Where are we reading in COMP_WORDS? I specifically avoid using that (and parse COMP_LINE instead) because I want the flexibility of parsing COMP_LINE ourselves.

@kislyuk
Owner

Ah I see. I always work with the global shellcode hook, but you're working with the per-app shellcode hook. I forgot to update the per-app shellcode hook, and you confused COMP_WORDS with COMP_WORDBREAKS. The latter is what needs to be set in the per-app hook.

@jimparis

Oops, I mixed those up, you're right. I put COMP_WORDBREAKS in my own script and pushed something different here.

@kislyuk kislyuk merged commit d5cba62 into kislyuk:master

1 check was pending

Details default The Travis build is in progress
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 15, 2013
  1. @jimparis
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 0 deletions.
  1. +1 −0  scripts/register-python-argcomplete
View
1  scripts/register-python-argcomplete
@@ -22,6 +22,7 @@ _python_argcomplete() {
local IFS='\013'
COMPREPLY=( $(IFS="$IFS" \
COMP_LINE="$COMP_LINE" \
+ COMP_WORDBREAKS="$COMP_WORDBREAKS" \
COMP_POINT="$COMP_POINT" \
_ARGCOMPLETE=1 \
"$1" 8>&1 9>&2 1>/dev/null 2>/dev/null) )
Something went wrong with that request. Please try again.