diff options
| author | Aurelien Foret <aurelien@archlinux.org> | 2006-01-17 22:19:40 +0100 |
|---|---|---|
| committer | Aurelien Foret <aurelien@archlinux.org> | 2006-01-17 22:19:40 +0100 |
| commit | 4857012be7c5970bd2fe1c7aed37fb2e1071a802 (patch) | |
| tree | b10f97619639e07245b93a77ea0a4c5b2ad4fc3d | |
| parent | 510d57b5774b80b9aa89c28a2ad1bc77304d0f38 (diff) | |
| download | pacman-4857012be7c5970bd2fe1c7aed37fb2e1071a802.tar.gz pacman-4857012be7c5970bd2fe1c7aed37fb2e1071a802.tar.xz | |
code cleanup
| -rw-r--r-- | src/pacman/add.c | 2 | ||||
| -rw-r--r-- | src/pacman/remove.c | 3 | ||||
| -rw-r--r-- | src/pacman/sync.c | 13 |
3 files changed, 12 insertions, 6 deletions
diff --git a/src/pacman/add.c b/src/pacman/add.c index aa65a139..70ff1abd 100644 --- a/src/pacman/add.c +++ b/src/pacman/add.c @@ -108,7 +108,7 @@ int pacman_add(list_t *targets) break; case PM_ERR_FILE_CONFLICTS: for(i = alpm_list_first(data); i; i = alpm_list_next(i)) { - MSG(NL, ":: %s\n", (char *)alpm_list_getdata(i)); + MSG(NL, ":: %s", (char *)alpm_list_getdata(i)); } alpm_list_free(data); MSG(NL, "\nerrors occurred, no packages were upgraded.\n"); diff --git a/src/pacman/remove.c b/src/pacman/remove.c index 11dc2f8d..495a7253 100644 --- a/src/pacman/remove.c +++ b/src/pacman/remove.c @@ -96,7 +96,8 @@ int pacman_remove(list_t *targets) case PM_ERR_UNSATISFIED_DEPS: for(lp = alpm_list_first(data); lp; lp = alpm_list_next(lp)) { PM_DEPMISS *miss = alpm_list_getdata(lp); - MSG(NL, " %s: is required by %s\n", alpm_dep_getinfo(miss, PM_DEP_TARGET), alpm_dep_getinfo(miss, PM_DEP_NAME)); + MSG(NL, " %s: is required by %s\n", alpm_dep_getinfo(miss, PM_DEP_TARGET), + alpm_dep_getinfo(miss, PM_DEP_NAME)); } alpm_list_free(data); break; diff --git a/src/pacman/sync.c b/src/pacman/sync.c index c6a8fa3d..6e8ba5c8 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -515,7 +515,8 @@ int pacman_sync(list_t *targets) } } - /* Step 2: "compute" the transaction based on targets and flags */ + /* Step 2: "compute" the transaction based on targets and flags + */ if(alpm_trans_prepare(&data) == -1) { ERR(NL, "failed to prepare transaction (%s)\n", alpm_strerror(pm_errno)); switch(pm_errno) { @@ -523,8 +524,9 @@ int pacman_sync(list_t *targets) for(lp = alpm_list_first(data); lp; lp = alpm_list_next(lp)) { PM_DEPMISS *miss = alpm_list_getdata(lp); - MSG(NL, ":: %s: requires %s", alpm_dep_getinfo(miss, PM_DEP_TARGET), - alpm_dep_getinfo(miss, PM_DEP_NAME)); + MSG(NL, ":: %s: %s %s", alpm_dep_getinfo(miss, PM_DEP_TARGET), + alpm_dep_getinfo(miss, PM_DEP_TYPE) == PM_DEP_TYPE_DEPEND ? "requires" : "is required by", + alpm_dep_getinfo(miss, PM_DEP_NAME)); switch((int)alpm_dep_getinfo(miss, PM_DEP_MOD)) { case PM_DEP_MOD_EQ: MSG(CL, "=%s", alpm_dep_getinfo(miss, PM_DEP_VERSION)); break; case PM_DEP_MOD_GE: MSG(CL, ">=%s", alpm_dep_getinfo(miss, PM_DEP_VERSION)); break; @@ -543,6 +545,8 @@ int pacman_sync(list_t *targets) } alpm_list_free(data); break; + alpm_list_free(data); + break; default: break; } @@ -753,7 +757,8 @@ int pacman_sync(list_t *targets) goto cleanup; } - /* Step 3: actually perform the installation */ + /* Step 3: actually perform the installation + */ if(alpm_trans_commit(&data) == -1) { ERR(NL, "failed to commit transaction (%s)\n", alpm_strerror(pm_errno)); switch(pm_errno) { |