coredumpctl: Don't treat no coredumps as failure

Having to deal with a process that fails or doesn't fail depending on
whether there are coredumps or not is incredibly annoying for users.

diff --git a/man/coredumpctl.xml b/man/coredumpctl.xml
index 6179d6b760d95e69d6ed4e95641dcdd07d00227e..6db9bca0681a24d1298fc8870e4e2cce09fc415e 100644 (file)
--- a/man/coredumpctl.xml
+++ b/man/coredumpctl.xml
@@ -389,10 +389,7 @@
 
   <refsect1>
     <title>Exit status</title>
-    <para>On success, 0 is returned; otherwise, a non-zero failure
-    code is returned. Not finding any matching core dumps is treated as
-    failure.
-    </para>
+    <para>On success, 0 is returned; otherwise, a non-zero failure code is returned.</para>
   </refsect1>
 
   <refsect1>

diff --git a/src/coredump/coredumpctl.c b/src/coredump/coredumpctl.c
index e7d0dd34c0d104d88f8db615c1ce41af41dd750a..4a25a2fb703a1f78316bd677e66fcdd4adb9e41e 100644 (file)
--- a/src/coredump/coredumpctl.c
+++ b/src/coredump/coredumpctl.c
@@ -963,9 +963,10 @@ static int dump_list(int argc, char **argv, void *userdata) {
                 }
 
                 if (!arg_field && n_found <= 0) {
-                        if (!arg_quiet)
+                        if (!arg_quiet && !sd_json_format_enabled(arg_json_format_flags))
                                 log_notice("No coredumps found.");
-                        return -ESRCH;
+
+                        return 0;
                 }
         }