[PATCH weston 3/3] compositor: print usage to stdout on success (not stderr)

Peter Hutterer peter.hutterer at who-t.net
Sun Jun 10 23:57:06 UTC 2018


Triggered by weston --help, the usage() output should not look like an error.

Note that there is only one caller of usage() at the moment, but let's handle
this here based on the status in case we add other cases.

https://gitlab.freedesktop.org/wayland/weston/issues/112

Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
 compositor/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/compositor/main.c b/compositor/main.c
index e84857f7..93f92fdc 100644
--- a/compositor/main.c
+++ b/compositor/main.c
@@ -438,7 +438,7 @@ verify_xdg_runtime_dir(void)
 static int
 usage(int error_code)
 {
-	fprintf(stderr,
+	fprintf(error_code == EXIT_SUCCESS ? stdout : stderr,
 		"Usage: weston [OPTIONS]\n\n"
 		"This is weston version " VERSION ", the Wayland reference compositor.\n"
 		"Weston supports multiple backends, and depending on which backend is in use\n"
-- 
2.14.4



More information about the wayland-devel mailing list