summaryrefslogtreecommitdiffstats
path: root/contrib
diff options
context:
space:
mode:
authorPierre Neidhardt <ambrevar@gmail.com>2014-02-14 13:18:55 +0100
committerAllan McRae <allan@archlinux.org>2014-03-03 11:25:55 +1000
commitcfde337b7b5f4a7f56ff0a4e6142804f6e638f69 (patch)
tree09369a73498e145c3daaee6a8b7275570ca8cc84 /contrib
parentcfaff6e0c14d29f07246386695bce0188ce6f44b (diff)
downloadpacman-cfde337b7b5f4a7f56ff0a4e6142804f6e638f69.tar.xz
pacsearch: pattern arguments work as for pacman
Previously only one pattern was allowed. $ pacsearch foo bar Search for packages containing 'foo bar'. $ pacman -Ss foo bar Search for packages containing both 'foo' and 'bar'. Note that removing the quotes from the call was not enough since $ pacsearch 'foo|bar' would then fail. Note the use of '--' to indicate the end of option parsing. This way we ensure that input will always be valid and we need not input checks anymore. Signed-off-by: Pierre Neidhardt <ambrevar@gmail.com> Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'contrib')
-rw-r--r--contrib/pacsearch.in41
1 files changed, 13 insertions, 28 deletions
diff --git a/contrib/pacsearch.in b/contrib/pacsearch.in
index 0ab840e1..18d4641b 100644
--- a/contrib/pacsearch.in
+++ b/contrib/pacsearch.in
@@ -92,61 +92,46 @@ sub print_pkg {
print "$MAGENTA";
}
print "$v[0]/$RESET$BOLD$v[1] $GREEN$v[2]$BLUE$v[3]$CYAN$v[4]$RESET\n";
- print "$v[5]\n";
+ print "$v[5]";
}
my %allpkgs = ();
my @pkglist = ();
-my $syncout = `pacman -Ss '@ARGV'`;
-# split each sync search entry into its own array entry
-my @syncpkgs = split(/\n^(?=\w)/m, $syncout);
-# remove the extra \n from the last desc entry
-if ($#syncpkgs >= 0) {
- chomp($syncpkgs[$#syncpkgs]);
-}
-
-foreach $_ (@syncpkgs) {
+open (my $syncout, '-|', 'pacman', '-Ss', '--', @ARGV) or exit 1;
+while ( readline($syncout) ) {
# We grab the following fields: repo, name, ver, group, installed, and
# desc. We grab leading space for 'group' and 'installed' so that we do not
# need to test if non-empty when printing.
- my @pkgfields = /^(.*?)\/(.*?) (.*?)( \(.*?\))?( \[.*\])?\n(.*)$/s;
- if(not @pkgfields) {
- # skip any non-matching line and just print it for the user
- print $_, "\n";
- next;
- }
+ my @pkgfields = /^(.*?)\/(.*?) (.*?)( \(.*?\))?( \[.*\])?$/s;
+ my $desc = readline($syncout);
# since 'group' and 'installed' are optional, we should fill it in if necessary
$pkgfields[3] = "" if not defined $pkgfields[3];
$pkgfields[4] = "" if not defined $pkgfields[4];
+ $pkgfields[5] = $desc;
# Add each sync pkg by name/ver to a hash table.
# Any value is good since we only check for existence.
$allpkgs{$pkgfields[1] . $pkgfields[2]} = 1;
push (@pkglist, \@pkgfields);
}
+close ($syncout);
-my $queryout = `pacman -Qs '@ARGV'`;
-# split each querysearch entry into its own array entry
-my @querypkgs = split(/\n^(?=\w)/m, $queryout);
-# remove the extra \n from the last desc entry
-if ($#querypkgs >= 0) {
- chomp ($querypkgs[$#querypkgs]);
-}
-
-foreach $_ (@querypkgs) {
+open (my $queryout, '-|', 'pacman', '-Qs', '--', @ARGV) or exit 1;
+while ( readline($queryout) ) {
# We grab the same field as before, even the "installed" which is always
# empty for local searches. This allows us to reserve a cell in @pkgfields.
- my @pkgfields = /^(.*?)\/(.*?) (.*?)( \(.*?\))?( \[.*\])?\n(.*)$/s;
- # skip any non-matching line
- next if not defined $pkgfields[1];
+ my @pkgfields = /^(.*?)\/(.*?) (.*?)( \(.*?\))?( \[.*\])?$/s;
+ my $desc = readline($queryout);
# check if the package was listed in the sync out
if (not exists $allpkgs{$pkgfields[1] . $pkgfields[2]}) {
# since 'group' is optional, we should fill it in if necessary
$pkgfields[3] = "" if not defined $pkgfields[3];
$pkgfields[4] = " [$LC_INSTALLED]";
+ $pkgfields[5] = $desc;
push (@pkglist, \@pkgfields);
}
}
+close ($queryout);
foreach (@pkglist) {
print_pkg (@{$_});