- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] use consistent time notation for the log
by Andrew Gregory 20 Jun '19

20 Jun '19

%X is locale-dependent, making it impossible to reliably parse and potentially overflowing the buffer. %T is consistent across locales. Also fixes some adjacent whitespace. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- lib/libalpm/log.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/libalpm/log.c b/lib/libalpm/log.c index d8842a55..2ff5544b 100644 --- a/lib/libalpm/log.c +++ b/lib/libalpm/log.c @@ -39,11 +39,11 @@ static int _alpm_log_leader(FILE *f, const char *prefix) { time_t t = time(NULL); struct tm *tm = localtime(&t); - int length = 32; - char timestamp[length]; + int length = 32; + char timestamp[length]; /* Use ISO-8601 date format */ - strftime(timestamp,length,"%FT%X%z", tm); + strftime(timestamp,length,"%FT%T%z", tm); return fprintf(f, "[%s] [%s] ", timestamp, prefix); } -- 2.21.0

2 1

[pacman-dev] [PATCH] pacman: correctly free lists
by morganamilo 20 Jun '19

20 Jun '19

diff --git a/src/pacman/database.c b/src/pacman/database.c index f3b890ce..7a792bad 100644 --- a/src/pacman/database.c +++ b/src/pacman/database.c @@ -105,7 +105,8 @@ static int check_db_missing_deps(alpm_list_t *pkglist) free(depstring); ret++; } - FREELIST(data); + alpm_list_free_inner(data, (alpm_list_fn_free)alpm_depmissing_free); + alpm_list_free(data); return ret; } @@ -159,7 +160,8 @@ static int check_db_local_package_conflicts(alpm_list_t *pkglist) conflict->package1, conflict->package2); ret++; } - FREELIST(data); + alpm_list_free_inner(data, (alpm_list_fn_free)alpm_conflict_free); + alpm_list_free(data); return ret; } -- 2.21.0

2 1

[pacman-dev] [PATCH] Fix compiler warnings with gcc-9.1
by Allan McRae 20 Jun '19

20 Jun '19

Signed-off-by: Allan McRae <allan(a)archlinux.org> --- I think these are all legimite ways of fixing the warings... callback.c: filenamelen >= 50-33, so i is always positive. No need for that check pacman.c: this is the same condition, it just changes the compiler optimisation of the loop to avoid the warning query.c: this gave a dumb warning... it is also the only place in the codebase we used strncpy vs strcpy. src/pacman/callback.c | 2 +- src/pacman/pacman.c | 2 +- src/pacman/query.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/pacman/callback.c b/src/pacman/callback.c index 22865614..aa5f521e 100644 --- a/src/pacman/callback.c +++ b/src/pacman/callback.c @@ -804,7 +804,7 @@ void cb_dl_progress(const char *filename, off_t file_xfered, off_t file_total) int i = filenamelen - 3; wchar_t *wcp = wcfname; /* grab the max number of char columns we can fill */ - while(i > 0 && wcwidth(*wcp) < i) { + while(wcwidth(*wcp) < i) { i -= wcwidth(*wcp); wcp++; } diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 93fb98a4..b7406cea 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -1079,7 +1079,7 @@ static void cl_to_log(int argc, char *argv[]) return; } char *p = cl_text; - for(i = 0; i < argc - 1; i++) { + for(i = 0; i + 1 < argc; i++) { strcpy(p, argv[i]); p += strlen(argv[i]); *p++ = ' '; diff --git a/src/pacman/query.c b/src/pacman/query.c index 3e728257..de82553c 100644 --- a/src/pacman/query.c +++ b/src/pacman/query.c @@ -189,7 +189,7 @@ static int query_fileowner(alpm_list_t *targets) if(!lrealpath(filename, rpath)) { /* Can't canonicalize path, try to proceed anyway */ - strncpy(rpath, filename, PATH_MAX); + strcpy(rpath, filename); } if(strncmp(rpath, root, rootlen) != 0) { -- 2.22.0

1 0

[pacman-dev] [PATCH] free makedepends/checkdepends when freeing packages
by Dave Reisner 20 Jun '19

20 Jun '19

Credit to Andrew for identifying source of the leak. --- lib/libalpm/package.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c index 94716666..dde32175 100644 --- a/lib/libalpm/package.c +++ b/lib/libalpm/package.c @@ -683,6 +683,8 @@ void _alpm_pkg_free(alpm_pkg_t *pkg) alpm_list_free(pkg->backup); free_deplist(pkg->depends); free_deplist(pkg->optdepends); + free_deplist(pkg->checkdepends); + free_deplist(pkg->makedepends); free_deplist(pkg->conflicts); free_deplist(pkg->provides); alpm_list_free(pkg->removes); -- 2.22.0

2 1

[pacman-dev] [PATCH] pacman: fix error during -Fy
by morganamilo 07 Jun '19

07 Jun '19

Signed-off-by: morganamilo <morganamilo(a)gmail.com> diff --git a/src/pacman/files.c b/src/pacman/files.c index 74d06815..8e518486 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -300,7 +300,7 @@ int pacman_files(alpm_list_t *targets) return files_list(files_dbs, targets); } - if(targets == NULL) { + if(targets == NULL && !config->op_s_sync) { pm_printf(ALPM_LOG_ERROR, _("no targets specified (use -h for help)\n")); return 1; } -- 2.21.0

2 1

[pacman-dev] [PATCH v2 1/2] libmakepkg: fix reporting of invalid archive extensions in compress.sh
by Eli Schwartz 06 Jun '19

06 Jun '19

In commit 1825bd6716c2a51c92642e8b96beac0101e83805 this was split out from makepkg, but the warning was not properly migrated; $ext did not ever exist. As a result, no matter what you did, the only possible warning was: ==> WARNING: '' is not a valid archive extension. Fix to filter based on the presence of .tar in the argument, and building the $ext variable for all checking and messaging purposes within the function. Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> --- scripts/libmakepkg/util/compress.sh.in | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/scripts/libmakepkg/util/compress.sh.in b/scripts/libmakepkg/util/compress.sh.in index 3d4d88fd..9e3150f7 100644 --- a/scripts/libmakepkg/util/compress.sh.in +++ b/scripts/libmakepkg/util/compress.sh.in @@ -29,21 +29,21 @@ source "$LIBRARY/util/message.sh" # Wrapper around many stream compression formats, for use in the middle of a # pipeline. A tar archive is passed on stdin and compressed to stdout. compress_as() { - # $1: final archive filename extension for compression type detection + # $1: final archive filename extension for compression type detection - local filename="$1" + local ext=".tar${1##*.tar}" - case "$filename" in - *tar.gz) ${COMPRESSGZ[@]:-gzip -c -f -n} ;; - *tar.bz2) ${COMPRESSBZ2[@]:-bzip2 -c -f} ;; - *tar.xz) ${COMPRESSXZ[@]:-xz -c -z -} ;; - *tar.zst) ${COMPRESSZST[@]:-zstd -c -z -q -} ;; - *tar.lrz) ${COMPRESSLRZ[@]:-lrzip -q} ;; - *tar.lzo) ${COMPRESSLZO[@]:-lzop -q} ;; - *tar.Z) ${COMPRESSZ[@]:-compress -c -f} ;; - *tar.lz4) ${COMPRESSLZ4[@]:-lz4 -q} ;; - *tar.lz) ${COMPRESSLZ[@]:-lzip -c -f} ;; - *tar) cat ;; + case "$ext" in + *.tar.gz) ${COMPRESSGZ[@]:-gzip -c -f -n} ;; + *.tar.bz2) ${COMPRESSBZ2[@]:-bzip2 -c -f} ;; + *.tar.xz) ${COMPRESSXZ[@]:-xz -c -z -} ;; + *.tar.zst) ${COMPRESSZST[@]:-zstd -c -z -q -} ;; + *.tar.lrz) ${COMPRESSLRZ[@]:-lrzip -q} ;; + *.tar.lzo) ${COMPRESSLZO[@]:-lzop -q} ;; + *.tar.Z) ${COMPRESSZ[@]:-compress -c -f} ;; + *.tar.lz4) ${COMPRESSLZ4[@]:-lz4 -q} ;; + *.tar.lz) ${COMPRESSLZ[@]:-lzip -c -f} ;; + *.tar) cat ;; *) warning "$(gettext "'%s' is not a valid archive extension.")" \ "$ext"; cat ;; esac -- 2.21.0

2 3

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v5.1.1-180-g39c20ad4
by Allan McRae 06 Jun '19

06 Jun '19

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The official pacman repository". The branch, master has been updated via 39c20ad4f1d5f6e915b5be8976b6a94885ca3b0c (commit) via 55a65aaf904a32bcf59c652cbc03fc510c265142 (commit) via ff1ae94c102cab487444bcdb0c76ee489c11dfe8 (commit) via eb92bcb089a19dd456ce244732415660cf393a89 (commit) from 27e80ca7f61d0eb9861d36c24520a3e081bcb2ec (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 39c20ad4f1d5f6e915b5be8976b6a94885ca3b0c Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Mon Jun 3 00:32:36 2019 -0700 hooks: rename type File to Path Make it clearer that the targets are matched against both directories and regular files and free up File to potentially refer specifically to regular files in the future. File is retained as a deprecated alias for Path for the time being to avoid breaking existing hooks and will be removed in a future release. See FS#53136. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 55a65aaf904a32bcf59c652cbc03fc510c265142 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Mon Jun 3 00:32:35 2019 -0700 makepkg: restrict pkgname and pkgver to ascii pkgname and pkgver are used as directory names within database files. libarchive does not provide a reliable locale-independent method for reading archive file names, causing errors when archive paths include non-ascii characters. This is a first step toward dealing with FS#49342, by hopefully reducing the number of packages with non-ascii data in the wild before updating libalpm to reject them outright. See https://github.com/libarchive/libarchive/wiki/Filenames and https://github.com/libarchive/libarchive/issues/587 Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit ff1ae94c102cab487444bcdb0c76ee489c11dfe8 Author: morganamilo <morganamilo(a)gmail.com> Date: Wed May 29 23:36:34 2019 +0100 pacman: rework the UI of -F Reworks the UI of -F according to FS#47949 In short -F replaces both -Fs and -Fo. Searching for an exact path (target contains "/"), causes the output to switch to the old -Fo output. Otherwise the old -Fs output is used. Also strip the leading "/" from targets like how -Qo does. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit eb92bcb089a19dd456ce244732415660cf393a89 Author: morganamilo <morganamilo(a)gmail.com> Date: Tue May 28 22:30:07 2019 +0100 pacman: refactor file match printing to their own functions Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/alpm-hooks.5.asciidoc | 17 +- doc/pacman.8.asciidoc | 14 +- lib/libalpm/hook.c | 8 +- scripts/libmakepkg/lint_pkgbuild/pkgname.sh.in | 4 + scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in | 5 + src/pacman/files.c | 174 +++++++++------------ src/pacman/pacman.c | 19 +-- test/pacman/tests/hook-file-change-packages.py | 2 +- .../pacman/tests/hook-file-remove-trigger-match.py | 2 +- test/pacman/tests/hook-file-upgrade-nomatch.py | 2 +- test/pacman/tests/hook-target-list.py | 2 +- test/pacman/tests/hook-type-reused.py | 2 +- 12 files changed, 112 insertions(+), 139 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] makepkg: restrict pkgname and pkgver to ascii
by Andrew Gregory 06 Jun '19

06 Jun '19

pkgname and pkgver are used as directory names within database files. libarchive does not provide a reliable locale-independent method for reading archive file names, causing errors when archive paths include non-ascii characters. This is a first step toward dealing with FS#49342, by hopefully reducing the number of packages with non-ascii data in the wild before updating libalpm to reject them outright. See https://github.com/libarchive/libarchive/wiki/Filenames and https://github.com/libarchive/libarchive/issues/587 Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- scripts/libmakepkg/lint_pkgbuild/pkgname.sh.in | 4 ++++ scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in | 5 +++++ 2 files changed, 9 insertions(+) diff --git a/scripts/libmakepkg/lint_pkgbuild/pkgname.sh.in b/scripts/libmakepkg/lint_pkgbuild/pkgname.sh.in index 84666a29..51a25d2c 100644 --- a/scripts/libmakepkg/lint_pkgbuild/pkgname.sh.in +++ b/scripts/libmakepkg/lint_pkgbuild/pkgname.sh.in @@ -45,6 +45,10 @@ lint_one_pkgname() { error "$(gettext "%s is not allowed to start with a dot.")" "$type" ret=1 fi + if [[ $name = *[![:ascii:]]* ]]; then + error "$(gettext "%s may only contain ascii characters.")" "$type" + return 1 + fi if [[ $name = *[^[:alnum:]+_.@-]* ]]; then error "$(gettext "%s contains invalid characters: '%s'")" \ "$type" "${name//[[:alnum:]+_.@-]}" diff --git a/scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in b/scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in index 8d5d04f1..b0f45027 100644 --- a/scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in +++ b/scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in @@ -41,6 +41,11 @@ check_pkgver() { error "$(gettext "%s is not allowed to contain colons, forward slashes, hyphens or whitespace.")" "pkgver${type:+ in $type}" return 1 fi + + if [[ $ver = *[![:ascii:]]* ]]; then + error "$(gettext "%s may only contain ascii characters.")" "pkgver${type:+ in $type}" + return 1 + fi } lint_pkgver() { -- 2.21.0

2 1

[pacman-dev] [PATCH] hooks: rename type File to Path
by Andrew Gregory 03 Jun '19

03 Jun '19

Make it clearer that the targets are matched against both directories and regular files and free up File to potentially refer specifically to regular files in the future. File is retained as a deprecated alias for Path for the time being to avoid breaking existing hooks and will be removed in a future release. See FS#53136. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- doc/alpm-hooks.5.asciidoc | 17 +++++++++-------- lib/libalpm/hook.c | 8 ++++++-- test/pacman/tests/hook-file-change-packages.py | 2 +- .../tests/hook-file-remove-trigger-match.py | 2 +- test/pacman/tests/hook-file-upgrade-nomatch.py | 2 +- test/pacman/tests/hook-target-list.py | 2 +- test/pacman/tests/hook-type-reused.py | 2 +- 7 files changed, 20 insertions(+), 15 deletions(-) diff --git a/doc/alpm-hooks.5.asciidoc b/doc/alpm-hooks.5.asciidoc index 0f6a1057..916d43bb 100644 --- a/doc/alpm-hooks.5.asciidoc +++ b/doc/alpm-hooks.5.asciidoc @@ -12,7 +12,7 @@ SYNOPSIS -------- [Trigger] (Required, Repeatable) Operation = Install|Upgrade|Remove (Required, Repeatable) -Type = File|Package (Required) +Type = Path|Package (Required) Target = <Path|PkgName> (Required, Repeatable) [Action] (Required) @@ -49,19 +49,20 @@ defined the hook will run if the transaction matches *any* of the triggers. Select the type of operation to match targets against. May be specified multiple times. Installations are considered an upgrade if the package or file is already present on the system regardless of whether the new package - version is actually greater than the currently installed version. For File + version is actually greater than the currently installed version. For Path triggers, this is true even if the file changes ownership from one package to another. Required. -*Type =* File|Package:: +*Type =* Path|Package:: Select whether targets are matched against transaction packages or files. - See CAVEATS for special notes regarding File triggers. Required. + See CAVEATS for special notes regarding Path triggers. 'File' is a deprecated + alias for 'Path' and will be removed in a future release. Required. *Target =* <path|package>:: - The file path or package name to match against the active transaction. - File paths refer to the files in the package archive; the installation root + The path or package name to match against the active transaction. + Paths refer to the files in the package archive; the installation root should *not* be included in the path. Shell-style glob patterns are - allowed. It is possible to invert matches by prepending a file with an + allowed. It is possible to invert matches by prepending a target with an exclamation mark. May be specified multiple times. Required. ACTIONS @@ -119,7 +120,7 @@ Exec = /usr/bin/sync CAVEATS ------- -There are situations when file triggers may act in unexpected ways. Hooks are +There are situations when path triggers may act in unexpected ways. Hooks are triggered using the file list of the installed, upgraded, or removed package. When installing or upgrading a file that is extracted with a '.pacnew' extension, the original file name is used in triggering the hook. When diff --git a/lib/libalpm/hook.c b/lib/libalpm/hook.c index d90ed2da..6143ea0f 100644 --- a/lib/libalpm/hook.c +++ b/lib/libalpm/hook.c @@ -38,7 +38,7 @@ enum _alpm_hook_op_t { enum _alpm_trigger_type_t { ALPM_HOOK_TYPE_PACKAGE = 1, - ALPM_HOOK_TYPE_FILE, + ALPM_HOOK_TYPE_PATH, }; struct _alpm_trigger_t { @@ -303,7 +303,11 @@ static int _alpm_hook_parse_cb(const char *file, int line, if(strcmp(value, "Package") == 0) { t->type = ALPM_HOOK_TYPE_PACKAGE; } else if(strcmp(value, "File") == 0) { - t->type = ALPM_HOOK_TYPE_FILE; + _alpm_log(handle, ALPM_LOG_DEBUG, + "File targets are deprecated, use Path instead\n"); + t->type = ALPM_HOOK_TYPE_PATH; + } else if(strcmp(value, "Path") == 0) { + t->type = ALPM_HOOK_TYPE_PATH; } else { error(_("hook %s line %d: invalid value %s\n"), file, line, value); } diff --git a/test/pacman/tests/hook-file-change-packages.py b/test/pacman/tests/hook-file-change-packages.py index ad96fc14..4671dbe8 100644 --- a/test/pacman/tests/hook-file-change-packages.py +++ b/test/pacman/tests/hook-file-change-packages.py @@ -4,7 +4,7 @@ self.add_hook("hook", """ [Trigger] - Type = File + Type = Path Operation = Upgrade Target = bin/foo diff --git a/test/pacman/tests/hook-file-remove-trigger-match.py b/test/pacman/tests/hook-file-remove-trigger-match.py index 6c9375c0..ac50014c 100644 --- a/test/pacman/tests/hook-file-remove-trigger-match.py +++ b/test/pacman/tests/hook-file-remove-trigger-match.py @@ -4,7 +4,7 @@ self.add_hook("hook", """ [Trigger] - Type = File + Type = Path Operation = Remove Target = bin/foo diff --git a/test/pacman/tests/hook-file-upgrade-nomatch.py b/test/pacman/tests/hook-file-upgrade-nomatch.py index e984d376..891b8164 100644 --- a/test/pacman/tests/hook-file-upgrade-nomatch.py +++ b/test/pacman/tests/hook-file-upgrade-nomatch.py @@ -4,7 +4,7 @@ self.add_hook("hook", """ [Trigger] - Type = File + Type = Path Operation = Upgrade Target = bin/?* diff --git a/test/pacman/tests/hook-target-list.py b/test/pacman/tests/hook-target-list.py index 6dd6c4d8..b932c491 100644 --- a/test/pacman/tests/hook-target-list.py +++ b/test/pacman/tests/hook-target-list.py @@ -14,7 +14,7 @@ Target = foo [Trigger] - Type = File + Type = Path Operation = Install # matches files in 'file/' but not 'file/' itself Target = file/?* diff --git a/test/pacman/tests/hook-type-reused.py b/test/pacman/tests/hook-type-reused.py index 472c8caf..0ea1c0ba 100644 --- a/test/pacman/tests/hook-type-reused.py +++ b/test/pacman/tests/hook-type-reused.py @@ -4,7 +4,7 @@ """ [Trigger] Type = Package - Type = File + Type = Path Operation = Install Target = foo -- 2.21.0

1 0

[pacman-dev] [PATCH 1/2] pacman: refactor file match printing to its own function
by morganamilo 29 May '19

29 May '19

Signed-off-by: morganamilo <morganamilo(a)gmail.com> --- src/pacman/files.c | 56 +++++++++++++++++++++++++--------------------- 1 file changed, 31 insertions(+), 25 deletions(-) diff --git a/src/pacman/files.c b/src/pacman/files.c index fa4170bd..3ebd9b9b 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -99,11 +99,39 @@ static int files_fileowner(alpm_list_t *syncs, alpm_list_t *targets) { return 0; } +static void print_match(alpm_list_t *match, alpm_db_t *repo, alpm_pkg_t *pkg) +{ + alpm_db_t *db_local = alpm_get_localdb(config->handle); + const colstr_t *colstr = &config->colstr; + + if(config->op_f_machinereadable) { + alpm_list_t *ml; + for(ml = match; ml; ml = alpm_list_next(ml)) { + char *filename = ml->data; + print_line_machinereadable(repo, pkg, filename); + } + } else if(config->quiet) { + printf("%s/%s\n", alpm_db_get_name(repo), alpm_pkg_get_name(pkg)); + } else { + alpm_list_t *ml; + printf("%s%s/%s%s %s%s%s", colstr->repo, alpm_db_get_name(repo), + colstr->title, alpm_pkg_get_name(pkg), + colstr->version, alpm_pkg_get_version(pkg), colstr->nocolor); + + print_groups(pkg); + print_installed(db_local, pkg); + printf("\n"); + + for(ml = match; ml; ml = alpm_list_next(ml)) { + char *filename = ml->data; + printf(" %s\n", filename); + } + } +} + static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { int ret = 0; - alpm_db_t *db_local = alpm_get_localdb(config->handle); alpm_list_t *t; - const colstr_t *colstr = &config->colstr; for(t = targets; t; t = alpm_list_next(t)) { char *targ = t->data; @@ -148,29 +176,7 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { } if(match != NULL) { - if(config->op_f_machinereadable) { - alpm_list_t *ml; - for(ml = match; ml; ml = alpm_list_next(ml)) { - char *filename = ml->data; - print_line_machinereadable(repo, pkg, filename); - } - } else if(config->quiet) { - printf("%s/%s\n", alpm_db_get_name(repo), alpm_pkg_get_name(pkg)); - } else { - alpm_list_t *ml; - printf("%s%s/%s%s %s%s%s", colstr->repo, alpm_db_get_name(repo), - colstr->title, alpm_pkg_get_name(pkg), - colstr->version, alpm_pkg_get_version(pkg), colstr->nocolor); - - print_groups(pkg); - print_installed(db_local, pkg); - printf("\n"); - - for(ml = match; ml; ml = alpm_list_next(ml)) { - c = ml->data; - printf(" %s\n", c); - } - } + print_match(match, repo, pkg); alpm_list_free(match); } } -- 2.20.1

4 14