actionless / pikaur

AUR helper with minimal dependencies. Review PKGBUILDs all in once, next build them all without user interaction.
GNU General Public License v3.0
868 stars 87 forks source link

Fish completion and PKGEXT #49

Closed sboukortt closed 6 years ago

sboukortt commented 6 years ago

Hi, I just noticed that the fish completion for pikaur does not consider filenames ending in .pkg.tar when completing pikaur -U. Looking at /usr/share/fish/completions/pikaur.fish, I noticed this segment:

# Upgrade options
# Theoretically, pacman reads packages in all formats that libarchive supports
# In practice, it's going to be tar.xz or tar.gz
# Using "pkg.tar.*" here would change __fish_complete_suffix's descriptions to "unknown"
complete -c $progname -n "$upgrade" -xa '(__fish_complete_suffix pkg.tar.xz)' -d 'Package file'
complete -c $progname -n "$upgrade" -xa '(__fish_complete_suffix pkg.tar.gz)' -d 'Package file'

I understand not wanting to list every single possibility (though I just tried "pkg.tar*" and did not notice the difference mentioned in the comment), but .pkg.tar seems to be quite a popular option, since it bypasses compression entirely, which makes it very fast.

See for example:

Could we perhaps add it to the fish completion?

Thanks.

actionless commented 6 years ago

yup, feel free to submit a PR with such change

actionless commented 6 years ago

closed in #50