From b6e058985be524ba94544d541abc1ca9c4817133 Mon Sep 17 00:00:00 2001 From: Fabian Homborg Date: Wed, 10 Feb 2016 15:35:01 +0100 Subject: [PATCH] Fix "replacement string too large" error in pacman completion Turns out we can skip using string altogether, thanks to the magic of the cartesian product. --- share/completions/pacman.fish | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/share/completions/pacman.fish b/share/completions/pacman.fish index 2a5a44f4d..d01367f79 100644 --- a/share/completions/pacman.fish +++ b/share/completions/pacman.fish @@ -8,8 +8,7 @@ set -l listinstalled "(pacman -Q | string replace ' ' \t)" # This might be an issue if another package manager is also installed (e.g. for containers) set -l listall "(__fish_print_packages)" set -l listrepos "(__fish_print_pacman_repos)" -# Mask $1 so it won't be taken for a variable -set -l listgroups "(pacman -Sg | string replace -r '(.+)' '\$1\tPackage group')" +set -l listgroups "(pacman -Sg)\t'Package Group'" set -l noopt 'not __fish_contains_opt -s S -s D -s Q -s R -s U -s T -s F database query sync remove upgrade deptest files' set -l database '__fish_contains_opt -s D database'