diff options
author | Michal Schmidt <mschmidt@redhat.com> | 2014-11-28 11:58:34 +0100 |
---|---|---|
committer | Michal Schmidt <mschmidt@redhat.com> | 2014-11-28 12:04:41 +0100 |
commit | 0a1beeb64207eaa88ab9236787b1cbc2f704ae14 (patch) | |
tree | 50117277be2e2078d0373b944a30b4f09bb94443 /src/path | |
parent | b4d23205f238e06aaa31264628e20669e714acad (diff) | |
download | systemd-0a1beeb64207eaa88ab9236787b1cbc2f704ae14.tar.gz |
treewide: auto-convert the simple cases to log_*_errno()
As a followup to 086891e5c1 "log: add an "error" parameter to all
low-level logging calls and intrdouce log_error_errno() as log calls
that take error numbers", use sed to convert the simple cases to use
the new macros:
find . -name '*.[ch]' | xargs sed -r -i -e \
's/log_(debug|info|notice|warning|error|emergency)\("(.*)%s"(.*), strerror\(-([a-zA-Z_]+)\)\);/log_\1_errno(-\4, "\2%m"\3);/'
Multi-line log_*() invocations are not covered.
And we also should add log_unit_*_errno().
Diffstat (limited to 'src/path')
-rw-r--r-- | src/path/path.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/path/path.c b/src/path/path.c index 37f2571fab..e4d8d1d53c 100644 --- a/src/path/path.c +++ b/src/path/path.c @@ -88,7 +88,7 @@ static int list_homes(void) { if (q == -ENXIO) continue; if (q < 0) { - log_error("Failed to query %s: %s", path_table[i], strerror(-r)); + log_error_errno(-r, "Failed to query %s: %m", path_table[i]); r = q; continue; } @@ -109,7 +109,7 @@ static int print_home(const char *n) { r = sd_path_home(i, arg_suffix, &p); if (r < 0) { - log_error("Failed to query %s: %s", n, strerror(-r)); + log_error_errno(-r, "Failed to query %s: %m", n); return r; } |