summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAurelien Foret <aurelien@archlinux.org>2006-01-06 15:00:03 +0100
committerAurelien Foret <aurelien@archlinux.org>2006-01-06 15:00:03 +0100
commit2c22045ba85778dcdcedeee9ab11b8631c007cc5 (patch)
tree6d6fb84345f23686b826cba8031e46f475325deb
parent4585060a5a47e77ffb4bffef110b16505921a75d (diff)
downloadpacman-2c22045ba85778dcdcedeee9ab11b8631c007cc5.tar.gz
pacman-2c22045ba85778dcdcedeee9ab11b8631c007cc5.tar.xz
- fixed a regression with pacman 2.x (reworked patch from VMiklos <vmiklos@frugalware.org>):
if pacman is desiganted as a target in a sysupgrade operation, it should be able to upgrade itself first upon user request. - removed uneeded sanity checks in alpm interface calls - replaced printf by MSG in pacman
-rw-r--r--lib/libalpm/alpm.c9
-rw-r--r--lib/libalpm/alpm.h1
-rw-r--r--src/pacman/sync.c34
3 files changed, 32 insertions, 12 deletions
diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c
index 833737f4..7dca7ffd 100644
--- a/lib/libalpm/alpm.c
+++ b/lib/libalpm/alpm.c
@@ -719,8 +719,6 @@ int alpm_logaction(char *fmt, ...)
PMList *alpm_list_first(PMList *list)
{
- ASSERT(list != NULL, return(NULL));
-
return(list);
}
@@ -740,10 +738,17 @@ void *alpm_list_getdata(PMList *entry)
int alpm_list_free(PMList *entry)
{
+ ASSERT(entry != NULL, return(-1));
+
FREELIST(entry);
return(0);
}
+
+int alpm_list_count(PMList *list)
+{
+ return(_alpm_list_count(list));
+}
/** @} */
/** @defgroup alpm_misc Miscellaneous Functions
diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h
index 454fdc09..09034067 100644
--- a/lib/libalpm/alpm.h
+++ b/lib/libalpm/alpm.h
@@ -300,6 +300,7 @@ PM_LIST *alpm_list_first(PM_LIST *list);
PM_LIST *alpm_list_next(PM_LIST *entry);
void *alpm_list_getdata(PM_LIST *entry);
int alpm_list_free(PM_LIST *entry);
+int alpm_list_count(PM_LIST *list);
/* md5sums */
char *alpm_get_md5sum(char *name);
diff --git a/src/pacman/sync.c b/src/pacman/sync.c
index 273636ac..3e29568b 100644
--- a/src/pacman/sync.c
+++ b/src/pacman/sync.c
@@ -437,14 +437,28 @@ int pacman_sync(list_t *targets)
for(lp = alpm_list_first(data); lp; lp = alpm_list_next(lp)) {
PM_SYNCPKG *sync = alpm_list_getdata(lp);
PM_PKG *spkg = alpm_sync_getinfo(sync, PM_SYNC_PKG);
- if(!strcmp("pacman", alpm_pkg_getinfo(spkg, PM_PKG_NAME))) {
+ if(!strcmp("pacman", alpm_pkg_getinfo(spkg, PM_PKG_NAME)) && alpm_list_count(data) > 1) {
MSG(NL, "\n:: pacman has detected a newer version of the \"pacman\" package.\n");
MSG(NL, ":: It is recommended that you allow pacman to upgrade itself\n");
MSG(NL, ":: first, then you can re-run the operation with the newer version.\n");
MSG(NL, "::\n");
- if(!yesno(":: Upgrade anyway? [Y/n] ")) {
- retval = 0;
- goto cleanup;
+ if(yesno(":: Upgrade pacman first? [Y/n] ")) {
+ if(alpm_trans_release() == -1) {
+ ERR(NL, "failed to release transaction (%s)\n", alpm_strerror(pm_errno));
+ retval = 1;
+ goto cleanup;
+ }
+ if(alpm_trans_init(PM_TRANS_TYPE_SYNC, config->flags, cb_trans_evt, cb_trans_conv) == -1) {
+ ERR(NL, "failed to init transaction (%s)\n", alpm_strerror(pm_errno));
+ retval = 1;
+ goto cleanup;
+ }
+ if(alpm_trans_addtarget("pacman") == -1) {
+ ERR(NL, "could not add target '%s': %s\n", (char *)i->data, alpm_strerror(pm_errno));
+ retval = 1;
+ goto cleanup;
+ }
+ break;
}
}
}
@@ -510,7 +524,7 @@ int pacman_sync(list_t *targets)
packages = alpm_trans_getinfo(PM_TRANS_PACKAGES);
if(packages == NULL) {
- retval = 0;
+ /* nothing to do: just exit without complaining */
goto cleanup;
}
@@ -531,17 +545,17 @@ int pacman_sync(list_t *targets)
PM_PKG *p = alpm_list_getdata(j);
char *pkgname = alpm_pkg_getinfo(p, PM_PKG_NAME);
if(!list_is_strin(pkgname, list_remove)) {
- list_remove = list_add(list_remove, pkgname);
+ list_remove = list_add(list_remove, strdup(pkgname));
}
}
}
}
if(list_remove) {
- printf("\nRemove: ");
+ MSG(NL, "\nRemove: ");
str = buildstring(list_remove);
indentprint(str, 9);
- printf("\n");
- FREELISTPTR(list_remove);
+ MSG(CL, "\n");
+ FREELIST(list_remove);
FREE(str);
}
for(lp = alpm_list_first(packages); lp; lp = alpm_list_next(lp)) {
@@ -564,7 +578,7 @@ int pacman_sync(list_t *targets)
MSG(NL, "\nTargets: ");
str = buildstring(list_install);
indentprint(str, 9);
- MSG(NL, "\nTotal Package Size: %.1f MB\n", mb);
+ MSG(NL, "Total Package Size: %.1f MB\n", mb);
FREELIST(list_install);
FREE(str);