diff options
| author | Nagy Gabor <ngaba@bibl.u-szeged.hu> | 2008-07-01 21:06:32 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2008-07-08 04:03:52 +0200 |
| commit | 2122eb1428ac754e9f5d675b4c4fa62b0286d3b6 (patch) | |
| tree | 06d4fe8bbeb2f43099aa557c1e2fee98518d939b /lib/libalpm/package.c | |
| parent | 5f701005ed70fe08bb9932d1a1fe02c59cbc5bb0 (diff) | |
| download | pacman-2122eb1428ac754e9f5d675b4c4fa62b0286d3b6.tar.gz pacman-2122eb1428ac754e9f5d675b4c4fa62b0286d3b6.tar.xz | |
Don't duplicate packages in requiredby list
This is a "fix" for FS#10226. I think that multiple versioned dependencies
are quite common now, and the old behavior is quite annoying there. This
patch won't cause any slow-down.
Signed-off-by: Nagy Gabor <ngaba@bibl.u-szeged.hu>
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/package.c')
| -rw-r--r-- | lib/libalpm/package.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c index 3708a58a..79cc64e5 100644 --- a/lib/libalpm/package.c +++ b/lib/libalpm/package.c @@ -537,9 +537,7 @@ unsigned short SYMEXPORT alpm_pkg_has_scriptlet(pmpkg_t *pkg) * @brief Compute the packages requiring a given package. * @param pkg a package * @return the list of packages requiring pkg - * - * A depends on B through n depends <=> A listed in B's requiredby n times - * n == 0 or 1 in almost all cases */ + */ alpm_list_t SYMEXPORT *alpm_pkg_compute_requiredby(pmpkg_t *pkg) { const alpm_list_t *i, *j; @@ -560,6 +558,7 @@ alpm_list_t SYMEXPORT *alpm_pkg_compute_requiredby(pmpkg_t *pkg) _alpm_log(PM_LOG_DEBUG, "adding '%s' in requiredby field for '%s'\n", cachepkgname, pkg->name); reqs = alpm_list_add(reqs, strdup(cachepkgname)); + break; } } } |