On 03/06/13 13:45, Andrew Gregory wrote: > Brings pacman -Sl behavior in line with other listing operations. > > Signed-off-by: Andrew Gregory <[email protected]> > --- > src/pacman/sync.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/src/pacman/sync.c b/src/pacman/sync.c > index 2093029..f8af176 100644 > --- a/src/pacman/sync.c > +++ b/src/pacman/sync.c > @@ -503,12 +503,15 @@ static int sync_list(alpm_list_t *syncs, alpm_list_t > *targets) > if(db == NULL) { > pm_printf(ALPM_LOG_ERROR, > _("repository \"%s\" was not > found.\n"), repo); > - alpm_list_free(ls); > - return 1; > } > > ls = alpm_list_add(ls, db); > } > + > + if(!ls) { > + /* all of our targets were invalid */ > + return 1; > + } > } else { > ls = syncs; > } >
This needs a bit more. "pacman -Ql afsljk glibc" returns 1. So with "pacman -Sl asjkladfjk core", pacman should print the error about the unknown repo and then the list of core packages and return 1. Allan
