diff options
| author | Brian Ashworth <bosrsf04@gmail.com> | 2019-02-06 13:18:46 -0500 |
|---|---|---|
| committer | Drew DeVault <sir@cmpwn.com> | 2019-02-11 10:22:53 -0500 |
| commit | a2f661dceb5baa20bd5d329fbb6fb2d75a4a09f8 (patch) | |
| tree | 58168d2c3e464d556b1be8d76208f4f82258eabc | |
| parent | 6ade4bd7dd1cc5eed68842b407aa3f690b1f8e21 (diff) | |
| download | sway-a2f661dceb5baa20bd5d329fbb6fb2d75a4a09f8.zip sway-a2f661dceb5baa20bd5d329fbb6fb2d75a4a09f8.tar.gz sway-a2f661dceb5baa20bd5d329fbb6fb2d75a4a09f8.tar.bz2 | |
Clarify error for options and positional args
When both options and positional arguments are given, sway would print
the error `Don't use options with the IPC client`. Over the past
several months, it seems like users are including this error message in
issues instead of a debug log due to not understanding that the error
message means there is an issue with their command.
This makes the error message more verbose and will hopefully make it so
more users understand that the message is not a bug in sway, but with
the command used.
| -rw-r--r-- | sway/main.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/sway/main.c b/sway/main.c index 67d0f79..12f92bd 100644 --- a/sway/main.c +++ b/sway/main.c @@ -320,7 +320,13 @@ int main(int argc, char **argv) { if (optind < argc) { // Behave as IPC client if (optind != 1) { - sway_log(SWAY_ERROR, "Don't use options with the IPC client"); + sway_log(SWAY_ERROR, + "Detected both options and positional arguments. If you " + "are trying to use the IPC client, options are not " + "supported. Otherwise, check the provided arguments for " + "issues. See `man 1 sway` or `sway -h` for usage. If you " + "are trying to generate a debug log, use " + "`sway -d 2>sway.log`."); exit(EXIT_FAILURE); } if (!drop_permissions()) { |