summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGuillem Jover <guillem@debian.org>2017-10-16 21:37:42 +0200
committerGuillem Jover <guillem@debian.org>2017-10-17 02:28:08 +0200
commitdb880e67e10a6b1b33102955027c5d79c49df6d2 (patch)
tree651dfef2737239a502f5e13bd42dcd37f5361545
parent0f5d107191fa58a4092d9ce17d079a50c82c4332 (diff)
downloaddpkg-db880e67e10a6b1b33102955027c5d79c49df6d2.tar.gz
libdpkg: Clarify subprocess error message by shuffling it around
For command-names the current message was more or less fine. But for command descriptions the message made little sense.
-rw-r--r--debian/changelog1
-rw-r--r--lib/dpkg/subproc.c10
2 files changed, 6 insertions, 5 deletions
diff --git a/debian/changelog b/debian/changelog
index aad3a4dcf..1997b3846 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -45,6 +45,7 @@ dpkg (1.19.0) UNRELEASED; urgency=medium
default from default.mk.
* Make --uniform-compression the new default dpkg-deb behavior. Add support
for negating the option via --no-uniform-compression.
+ * Clarify subprocess error message by shuffling words around.
* Perl modules:
- Switch from Dpkg::Util to List::Util, now that the module in the
new required Perl contains the needed functions.
diff --git a/lib/dpkg/subproc.c b/lib/dpkg/subproc.c
index 8eb01854d..b2f36ec2e 100644
--- a/lib/dpkg/subproc.c
+++ b/lib/dpkg/subproc.c
@@ -136,7 +136,7 @@ subproc_check(int status, const char *desc, enum subproc_flags flags)
if (flags & SUBPROC_RETERROR)
return n;
- out(_("subprocess %s returned error exit status %d"), desc, n);
+ out(_("%s subprocess returned error exit status %d"), desc, n);
} else if (WIFSIGNALED(status)) {
n = WTERMSIG(status);
if (!n)
@@ -147,16 +147,16 @@ subproc_check(int status, const char *desc, enum subproc_flags flags)
return n;
if (n == SIGINT)
- out(_("subprocess %s was interrupted"), desc);
+ out(_("%s subprocess was interrupted"), desc);
else
- out(_("subprocess %s was killed by signal (%s)%s"),
+ out(_("%s subprocess was killed by signal (%s)%s"),
desc, strsignal(n),
WCOREDUMP(status) ? _(", core dumped") : "");
} else {
if (flags & SUBPROC_RETERROR)
return -1;
- out(_("subprocess %s failed with wait status code %d"), desc,
+ out(_("%s subprocess failed with wait status code %d"), desc,
status);
}
@@ -173,7 +173,7 @@ subproc_wait(pid_t pid, const char *desc)
if (dead_pid != pid) {
onerr_abort++;
- ohshite(_("wait for subprocess %s failed"), desc);
+ ohshite(_("wait for %s subprocess failed"), desc);
}
return status;