[pacman-dev] [PATCH] clarify broken dependency error messages

Andrew Gregory andrew.gregory.8 at gmail.com
Sun Jul 9 00:33:18 UTC 2017


Signed-off-by: Andrew Gregory <andrew.gregory.8 at gmail.com>
---
 src/pacman/sync.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/pacman/sync.c b/src/pacman/sync.c
index 680d7b3e..67bec8fd 100644
--- a/src/pacman/sync.c
+++ b/src/pacman/sync.c
@@ -717,15 +717,16 @@ static void print_broken_dep(alpm_depmissing_t *miss)
 	alpm_pkg_t *pkg;
 	if(miss->causingpkg == NULL) {
 		/* package being installed/upgraded has unresolved dependency */
-		colon_printf(_("%s: requires %s\n"), miss->target, depstring);
+		colon_printf(_("unable to satisfy dependency '%s' required by %s\n"),
+				depstring, miss->target);
 	} else if((pkg = alpm_pkg_find(trans_add, miss->causingpkg))) {
 		/* upgrading a package breaks a local dependency */
-		colon_printf(_("%s: installing %s (%s) breaks dependency '%s'\n"),
-				miss->target, miss->causingpkg, alpm_pkg_get_version(pkg), depstring);
+		colon_printf(_("installing %s (%s) breaks dependency '%s' required by %s\n"),
+				miss->causingpkg, alpm_pkg_get_version(pkg), depstring, miss->target);
 	} else {
 		/* removing a package breaks a local dependency */
-		colon_printf(_("%s: removing %s breaks dependency '%s'\n"),
-				miss->target, miss->causingpkg, depstring);
+		colon_printf(_("removing %s breaks dependency '%s' required by %s\n"),
+				miss->causingpkg, depstring, miss->target);
 	}
 	free(depstring);
 }
-- 
2.13.2


More information about the pacman-dev mailing list