Skip to content

Add bash completion script#55

Open
qrpnxz wants to merge 2 commits into
martanne:masterfrom
qrpnxz:master
Open

Add bash completion script#55
qrpnxz wants to merge 2 commits into
martanne:masterfrom
qrpnxz:master

Conversation

@qrpnxz

@qrpnxz qrpnxz commented Nov 15, 2021

Copy link
Copy Markdown

No description provided.

legionus added a commit to legionus/abduco that referenced this pull request Dec 2, 2021
Link: martanne#55
Signed-off-by: Alexey Gladkov <gladkov.alexey@gmail.com>
esac
}

command -F _abduco abduco

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
command -F _abduco abduco
complete -F _abduco abduco

' -- $2))
;;
*) # Session
local sessions=$(abduco | tail -n+2 | cut -f 3)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The session name is in the third column for the latest release (v0.6) but on the master branch it's in the fourth column. awk '{print $NF}' (=last column) works for both versions:

Suggested change
local sessions=$(abduco | tail -n+2 | cut -f 3)
local sessions=$(abduco | tail -n+2 | awk '{print $NF}')

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants