diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2017-05-13 11:26:55 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2017-05-19 14:24:03 -0400 |
| commit | 25f027c5efb2a26b2c7cb9a9608eb2bd49cc5ffc (patch) | |
| tree | a06ce3e12ccf6ada4027bd2f2994ad5727b3658f /src/shared/install.c | |
| parent | 35bca925f9bf78df3f64e321ab4830936fcef662 (diff) | |
| download | systemd-25f027c5efb2a26b2c7cb9a9608eb2bd49cc5ffc.tar.gz | |
tree-wide: when %m is used in log_*, always specify errno explicitly
All those uses were correct, but I think it's better to be explicit.
Using implicit errno is too error prone, and with this change we can require
(in the sense of a style guideline) that the code is always specified.
Helpful query: git grep -n -P 'log_[^s][a-z]+\(.*%m'
Diffstat (limited to 'src/shared/install.c')
| -rw-r--r-- | src/shared/install.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/shared/install.c b/src/shared/install.c index 58c8e852b2..7f3bc0d813 100644 --- a/src/shared/install.c +++ b/src/shared/install.c @@ -3045,7 +3045,7 @@ int unit_file_get_list( if (errno == ENOENT) continue; if (IN_SET(errno, ENOTDIR, EACCES)) { - log_debug("Failed to open \"%s\": %m", *i); + log_debug_errno(errno, "Failed to open \"%s\": %m", *i); continue; } |