diff options
| author | Dan McGee <dan@archlinux.org> | 2007-11-13 07:37:11 +0100 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2007-11-15 05:52:26 +0100 |
| commit | 2f55733be35e7ec4188ddab17a92ed67faa7210d (patch) | |
| tree | 68dda9780e98fd3400f154f24fb535641e58b489 | |
| parent | c8243bb8ed2908e92d311a32d15e797dc83ba633 (diff) | |
| download | pacman-2f55733be35e7ec4188ddab17a92ed67faa7210d.tar.gz pacman-2f55733be35e7ec4188ddab17a92ed67faa7210d.tar.xz | |
Ensure -Si and -Qi output show correct dependencies
Because alpm_pkg_get_depends() no longer returns strings as the data, we
need to first convert the returned structures to printable strings before
we can print the list.
Signed-off-by: Dan McGee <dan@archlinux.org>
| -rw-r--r-- | src/pacman/package.c | 24 |
1 files changed, 22 insertions, 2 deletions
diff --git a/src/pacman/package.c b/src/pacman/package.c index 84cdb204..5c2abcf4 100644 --- a/src/pacman/package.c +++ b/src/pacman/package.c @@ -46,6 +46,8 @@ void dump_pkg_full(pmpkg_t *pkg, int level) const char *reason, *descheader; time_t bdate, idate; char bdatestr[50], idatestr[50]; + const alpm_list_t *i; + alpm_list_t *depstrings = NULL; if(pkg == NULL) { return; @@ -69,6 +71,12 @@ void dump_pkg_full(pmpkg_t *pkg, int level) break; } + /* turn depends list into a text list */ + for(i = alpm_pkg_get_depends(pkg); i; i = alpm_list_next(i)) { + pmdepend_t *dep = (pmdepend_t*)alpm_list_getdata(i); + depstrings = alpm_list_add(depstrings, alpm_dep_get_string(dep)); + } + descheader = _("Description : "); /* actual output */ @@ -78,7 +86,7 @@ void dump_pkg_full(pmpkg_t *pkg, int level) list_display(_("License :"), alpm_pkg_get_licenses(pkg)); list_display(_("Groups :"), alpm_pkg_get_groups(pkg)); list_display(_("Provides :"), alpm_pkg_get_provides(pkg)); - list_display(_("Depends On :"), alpm_pkg_get_depends(pkg)); + list_display(_("Depends On :"), depstrings); list_display(_("Optional Deps :"), alpm_pkg_get_optdepends(pkg)); /* Only applicable if installed */ if(level > 0) { @@ -112,6 +120,8 @@ void dump_pkg_full(pmpkg_t *pkg, int level) dump_pkg_backups(pkg); } printf("\n"); + + FREELIST(depstrings); } /* Display the content of a sync package @@ -119,10 +129,18 @@ void dump_pkg_full(pmpkg_t *pkg, int level) void dump_pkg_sync(pmpkg_t *pkg, const char *treename) { const char *descheader, *md5sum; + const alpm_list_t *i; + alpm_list_t *depstrings = NULL; if(pkg == NULL) { return; } + /* turn depends list into a text list */ + for(i = alpm_pkg_get_depends(pkg); i; i = alpm_list_next(i)) { + pmdepend_t *dep = (pmdepend_t*)alpm_list_getdata(i); + depstrings = alpm_list_add(depstrings, alpm_dep_get_string(dep)); + } + descheader = _("Description : "); md5sum = alpm_pkg_get_md5sum(pkg); @@ -132,7 +150,7 @@ void dump_pkg_sync(pmpkg_t *pkg, const char *treename) printf(_("Version : %s\n"), (char *)alpm_pkg_get_version(pkg)); list_display(_("Groups :"), alpm_pkg_get_groups(pkg)); list_display(_("Provides :"), alpm_pkg_get_provides(pkg)); - list_display(_("Depends On :"), alpm_pkg_get_depends(pkg)); + list_display(_("Depends On :"), depstrings); list_display(_("Conflicts With :"), alpm_pkg_get_conflicts(pkg)); list_display(_("Replaces :"), alpm_pkg_get_replaces(pkg)); printf(_("Download Size : %6.2f K\n"), (float)alpm_pkg_get_size(pkg) / 1024.0); @@ -147,6 +165,8 @@ void dump_pkg_sync(pmpkg_t *pkg, const char *treename) printf(_("MD5 Sum : %s"), md5sum); } printf("\n"); + + FREELIST(depstrings); } /* Display list of backup files and their modification states |