Brings pacman -Sl behavior in line with other listing operations. Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com> --- v2: * return 1 if any repo names are invalid * remove the explicit return when all names are invalid, the for loop will take care of that on its own src/pacman/sync.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/pacman/sync.c b/src/pacman/sync.c index 2093029..fc1314b 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -485,6 +485,7 @@ static int sync_list(alpm_list_t *syncs, alpm_list_t *targets) { alpm_list_t *i, *j, *ls = NULL; alpm_db_t *db_local = alpm_get_localdb(config->handle); + int ret = 0; if(targets) { for(i = targets; i; i = alpm_list_next(i)) { @@ -503,8 +504,7 @@ 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; + ret = 1; } ls = alpm_list_add(ls, db); @@ -536,7 +536,7 @@ static int sync_list(alpm_list_t *syncs, alpm_list_t *targets) alpm_list_free(ls); } - return 0; + return ret; } static alpm_db_t *get_db(const char *dbname) -- 1.8.3.1