Please disregard, I now see that I missed the fact that there is a third section in the man page for run-time only commands. And that the first `gaps` spec is actually a part of it.

In the process of obtaining the package source to make a patch to at least improve the clarity, I also noticed that this man page comes from upstream and this isn't a Debian specific "issue".

Sorry for any wasted time!
Jon W


On Mon, 26 Aug 2024 21:18:08 -0700 Jonathan Wallace <jonmon6...@gmail.com> wrote:

> Package: sway
> Version: 1.9-2
> Severity: minor
>
> Dear Maintainer,
>
> One of the command specs for the `gaps` commands is in the
> config-file-only section of the man page, but is actually allowed at
> runtime with the `swaymsg` command.
>
> Here's how it appears in the man page for sway(5):
>
> COMMANDS
> This section only lists general commands. For input and output
> commands, refer to sway-input(5)
> and sway-output(5).
> The following commands may only be used in the configuration file.
>
> ...
>
> gaps inner|outer|horizontal|vertical|top|right|bottom|left
> all|current set|plus|minus|toggle <amount>
>
> ...
>
> The following commands may be used either in the configuration
> file or at runtime.
>
> ...
>
> gaps inner|outer|horizontal|vertical|top|right|bottom|left <amount>
>
>
> This is incorrect, for example running `$ swaymsg gaps inner all plus 5`
> works as you would expect and increases the gaps of all open windows at
> runtime. However one reading the man page would not expect this to work.
>
> This is my first week running sid and my first bug report. I'm also
> looking for mentorship on contributing and I think this would be a nice
> soft start to get the tools set up. Let me know if you are open to a
> patch for this issue. I would appreciate some help on getting oriented
> with how to do it.
>
> Thanks,
> Jon W
>
>
>
>
> -- System Information:
> Debian Release: trixie/sid
> APT prefers unstable
> APT policy: (500, 'unstable')
> Architecture: amd64 (x86_64)
>
> Kernel: Linux 6.10.6-amd64 (SMP w/8 CPU threads; PREEMPT)
> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE not set
> Shell: /bin/sh linked to /usr/bin/dash
> Init: systemd (via /run/systemd/system)
> LSM: AppArmor: enabled

Reply via email to