[pacman-dev] [PATCH] Don't print "local/" with -Qs

[Dan McGee]

On Sun, May 10, 2009 at 6:16 AM, Nagy Gabor <ngaba at bibl.u-szeged.hu> wrote:
> >From 1572535efe01dfae1f28b74862dec21f2102f165 Mon Sep 17 00:00:00 2001
> From: Nagy Gabor <ngaba at bibl.u-szeged.hu>
> Date: Sun, 10 May 2009 14:08:04 +0200
> Subject: [PATCH] Don't print "local/" with -Qs
>
> See FS#14642.
>
> Signed-off-by: Nagy Gabor <ngaba at bibl.u-szeged.hu>

Any objections to this? It is a longstanding "tradition" that we've
had the local prefix, so I want to make sure this isn't a mistake, as
this will definitely affect parsing of -Qs information as it can no
longer be treated the same as -Ss information.

I expressed my thoughts in the bug report that we keep it as is; I'll
await a few more opinions before taking this route. If we do take it,
I expect someone to fix the pacsearch script in contrib. :)

> ---
>  src/pacman/query.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/pacman/query.c b/src/pacman/query.c
> index 0d48638..2cb86a9 100644
> --- a/src/pacman/query.c
> +++ b/src/pacman/query.c
> @@ -166,7 +166,7 @@ static int query_search(alpm_list_t *targets)
>                pmpkg_t *pkg = alpm_list_getdata(i);
>
>                if (!config->quiet) {
> -                       printf("local/%s %s", alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg));
> +                       printf("%s %s", alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg));
>                } else {
>                        printf("%s", alpm_pkg_get_name(pkg));
>                }
> --
> 1.6.0.3