summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorAaron Griffin <aaron@archlinux.org>2007-03-09 06:33:06 +0100
committerAaron Griffin <aaron@archlinux.org>2007-03-09 06:33:06 +0100
commit6f870968edd09479410f3a6a52bb0be75e327366 (patch)
treeda0b2c83dc754b68e3fadb338a8723936b5d381c /src
parent68876e4eb029386dffee5c5084551bcc8cfe626d (diff)
downloadpacman-6f870968edd09479410f3a6a52bb0be75e327366.tar.gz
pacman-6f870968edd09479410f3a6a52bb0be75e327366.tar.xz
* Two fixes when running under a new root (-r|--root)
- Check EACCES in can_remove_file, to verify - we're only checking for a read-only FS here... if write permissions fail, we can still remove it - Switched the message when creating DB directories from an error to a debug message * Fix the dep tests once and for all (it wasn't working for provides=() packages)
Diffstat (limited to 'src')
-rw-r--r--src/pacman/deptest.c31
1 files changed, 15 insertions, 16 deletions
diff --git a/src/pacman/deptest.c b/src/pacman/deptest.c
index abc9ae99..dc968f25 100644
--- a/src/pacman/deptest.c
+++ b/src/pacman/deptest.c
@@ -38,17 +38,6 @@
extern config_t *config;
-static int chk_package(const char *pkgname, pmdepend_t *dep)
-{
- pmpkg_t *pkg;
- pkg = alpm_db_get_pkg(alpm_option_get_localdb(), pkgname);
-
- if(!pkg || !alpm_depcmp(pkg, dep)) {
- return(1);
- }
- return(0);
-}
-
int pacman_deptest(alpm_list_t *targets)
{
int retval = 0;
@@ -60,32 +49,42 @@ int pacman_deptest(alpm_list_t *targets)
for(i = targets; i; i = alpm_list_next(i)) {
int found = 0;
+ pmpkg_t *pkg;
pmdepend_t *dep;
const char *target;
alpm_list_t *j, *provides;
target = alpm_list_getdata(i);
+
+ /* splitdep modifies the string... we'll compensate for now */
+ char *saved_target = NULL;
+ saved_target = calloc(strlen(target)+1, sizeof(char));
+ strncpy(saved_target, target, strlen(target));
+
dep = alpm_splitdep(target);
- if(chk_package(target, dep) == 0) {
+ pkg = alpm_db_get_pkg(alpm_option_get_localdb(), target);
+ if(pkg && alpm_depcmp(pkg, dep)) {
found = 1;
} else {
/* not found, can we find anything that provides this in the local DB? */
provides = alpm_db_whatprovides(alpm_option_get_localdb(), target);
for(j = provides; j; j = alpm_list_next(j)) {
- const char *provide;
- provide = alpm_list_getdata(j);
+ pmpkg_t *pkg;
+ pkg = alpm_list_getdata(j);
- if(chk_package(provide, dep) == 0) {
+ if(pkg && alpm_depcmp(pkg, dep)) {
found = 1;
+ break;
}
}
}
if(!found) {
- MSG(NL, _("requires: %s"), target);
+ MSG(NL, _("requires: %s"), saved_target);
retval = 1;
}
+ free(saved_target);
}
return(retval);
}